tommylikehu | time | 00:58 |
---|---|---|
tommylikehu | hey~ any manila guys could take a look at this patch ? https://review.openstack.org/#/c/308212/ | 00:59 |
*** tuanluong has joined #openstack-manila | 01:12 | |
*** ganso has quit IRC | 01:13 | |
*** mtanino has quit IRC | 01:41 | |
tommylikehu | ping zengyingzhe | 01:56 |
zengyingzhe | tommylikehu, hi | 01:57 |
tommylikehu | thank, zengyingzhe, this patch is working in progess, so the driver's part havn't been added https://review.openstack.org/#/c/312321/ | 01:58 |
tommylikehu | could you take a look at this patch : https://review.openstack.org/#/c/362786/ | 01:59 |
zengyingzhe | tommylikehu, I see, thanks for your explanation. | 02:00 |
openstackgerrit | Victoria Martinez de la Cruz proposed openstack/manila: Setting up a development env with Devstack instructions https://review.openstack.org/408275 | 02:01 |
*** gcb has joined #openstack-manila | 02:12 | |
*** xinyanzhang has joined #openstack-manila | 02:13 | |
*** mtanino has joined #openstack-manila | 02:37 | |
*** cdelatte has quit IRC | 02:43 | |
openstackgerrit | zhongjun proposed openstack/manila: [WIP]Enable IPv6 in manila https://review.openstack.org/312321 | 02:44 |
openstackgerrit | Cedric Zhuang proposed openstack/manila: Add "update_access" interface support for VNX. https://review.openstack.org/395404 | 02:51 |
openstackgerrit | Akira KAMIO proposed openstack/manila: Fix nfs server service name for rhel,suse https://review.openstack.org/407846 | 02:53 |
*** kaisers_ has quit IRC | 02:56 | |
openstackgerrit | chenaidong1 proposed openstack/manila: Correct manila reraising of exception https://review.openstack.org/408390 | 03:01 |
*** mtanino has quit IRC | 03:02 | |
openstackgerrit | Akira KAMIO proposed openstack/manila: Fix nfs server service name for rhel, suse https://review.openstack.org/407846 | 03:33 |
*** kaisers_ has joined #openstack-manila | 03:46 | |
*** absubram_ has joined #openstack-manila | 03:50 | |
*** absubram_ is now known as absubram | 03:52 | |
*** tuanluong has quit IRC | 04:27 | |
*** harlowja has joined #openstack-manila | 04:37 | |
*** kaisers_ has quit IRC | 05:01 | |
*** kaisers_ has joined #openstack-manila | 05:01 | |
*** narayrak has joined #openstack-manila | 05:17 | |
*** wiebalck has joined #openstack-manila | 05:29 | |
*** shausy has joined #openstack-manila | 05:39 | |
*** wiebalck has quit IRC | 05:43 | |
*** harlowja has quit IRC | 05:46 | |
*** wiebalck has joined #openstack-manila | 06:00 | |
*** rraja has joined #openstack-manila | 06:03 | |
*** wiebalck has quit IRC | 06:06 | |
*** gouthamr has joined #openstack-manila | 06:16 | |
*** lpetrut has joined #openstack-manila | 06:32 | |
openstackgerrit | jeck proposed openstack/manila: Fix doc typo error https://review.openstack.org/408428 | 06:36 |
openstackgerrit | zhongjun proposed openstack/python-manilaclient: Enable IPv6 in manila https://review.openstack.org/328932 | 06:48 |
*** xinyanzhang has quit IRC | 07:00 | |
*** nkrinner_afk is now known as nkrinner | 07:00 | |
*** shausy has quit IRC | 07:05 | |
openstackgerrit | Tina Tang proposed openstack/manila: Support create 1 GB share https://review.openstack.org/408448 | 07:12 |
*** lpetrut has quit IRC | 07:13 | |
*** xinyanzhang has joined #openstack-manila | 07:15 | |
*** gouthamr_ has joined #openstack-manila | 07:28 | |
*** magic has joined #openstack-manila | 07:29 | |
*** gouthamr has quit IRC | 07:29 | |
*** lpetrut has joined #openstack-manila | 07:30 | |
*** xiaoyang has quit IRC | 07:32 | |
*** xiaoyang has joined #openstack-manila | 07:32 | |
*** magic has quit IRC | 07:33 | |
openstackgerrit | Tina Tang proposed openstack/manila: Change network allocation of Unity drvier to 1 https://review.openstack.org/408463 | 07:41 |
*** jprovazn has joined #openstack-manila | 07:42 | |
*** nherciu has joined #openstack-manila | 07:48 | |
*** xiaoyang has quit IRC | 08:00 | |
*** lpetrut has quit IRC | 08:02 | |
openstackgerrit | zhongjun proposed openstack/manila-specs: Add spec for enable IPv6 in manila https://review.openstack.org/362786 | 08:08 |
*** gouthamr_ has quit IRC | 08:08 | |
*** tommy_ has joined #openstack-manila | 08:11 | |
*** tpatzig_ has joined #openstack-manila | 08:11 | |
*** tommy_ is now known as Guest61919 | 08:12 | |
*** Guest61919 has quit IRC | 08:13 | |
*** tpatzig_ has quit IRC | 08:13 | |
*** jprovazn has quit IRC | 08:17 | |
*** lpetrut has joined #openstack-manila | 08:17 | |
*** jprovazn has joined #openstack-manila | 08:20 | |
*** nkrinner has quit IRC | 08:20 | |
*** nkrinner has joined #openstack-manila | 08:22 | |
*** lpetrut has quit IRC | 08:28 | |
*** nkrinner has quit IRC | 08:28 | |
*** lpetrut has joined #openstack-manila | 08:31 | |
*** gouthamr has joined #openstack-manila | 08:33 | |
*** lpetrut has quit IRC | 08:41 | |
*** lpetrut has joined #openstack-manila | 08:41 | |
*** nkrinner has joined #openstack-manila | 08:42 | |
*** mkoderer has joined #openstack-manila | 08:45 | |
*** lpetrut has quit IRC | 08:46 | |
openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-specs: Add a spec to fix and improve Access Rules https://review.openstack.org/399049 | 08:51 |
*** pcaruana has joined #openstack-manila | 08:58 | |
*** gouthamr has quit IRC | 09:11 | |
*** absubram_ has joined #openstack-manila | 09:18 | |
*** absubram has quit IRC | 09:19 | |
*** absubram_ is now known as absubram | 09:19 | |
*** gouthamr has joined #openstack-manila | 09:24 | |
*** gouthamr has quit IRC | 09:29 | |
*** sapcc-bot has quit IRC | 09:40 | |
*** carthaca_ has quit IRC | 09:40 | |
*** carthaca_ has joined #openstack-manila | 09:40 | |
*** sapcc-bot has joined #openstack-manila | 09:40 | |
*** mkoderer_ has joined #openstack-manila | 09:40 | |
*** tpatzig_ has joined #openstack-manila | 09:40 | |
*** dgonzalez_ has joined #openstack-manila | 09:40 | |
*** tommy_ has joined #openstack-manila | 09:40 | |
*** databus23_ has joined #openstack-manila | 09:40 | |
*** tommy_ is now known as Guest55964 | 09:41 | |
*** mkoderer_ has quit IRC | 09:42 | |
*** tpatzig_ has quit IRC | 09:42 | |
*** dgonzalez_ has quit IRC | 09:42 | |
*** Guest55964 has quit IRC | 09:42 | |
*** databus23_ has quit IRC | 09:42 | |
*** jprovazn has quit IRC | 09:56 | |
*** tpatzig_ has joined #openstack-manila | 09:58 | |
*** databus23_ has joined #openstack-manila | 09:58 | |
*** david_1 has joined #openstack-manila | 09:58 | |
*** dgonzalez_ has joined #openstack-manila | 09:58 | |
*** mkoderer_ has joined #openstack-manila | 09:58 | |
*** tpatzig_ has quit IRC | 10:00 | |
*** databus23_ has quit IRC | 10:00 | |
*** dgonzalez_ has quit IRC | 10:00 | |
*** mkoderer_ has quit IRC | 10:00 | |
*** david_1 has quit IRC | 10:00 | |
*** carthaca_ has quit IRC | 10:02 | |
*** carthaca_ has joined #openstack-manila | 10:02 | |
*** jprovazn has joined #openstack-manila | 10:10 | |
*** gouthamr has joined #openstack-manila | 10:14 | |
*** sandanar has joined #openstack-manila | 10:15 | |
*** dsariel has joined #openstack-manila | 10:18 | |
*** ganso has joined #openstack-manila | 10:23 | |
*** gouthamr_ has joined #openstack-manila | 10:23 | |
*** gouthamr has quit IRC | 10:25 | |
*** gouthamr has joined #openstack-manila | 10:28 | |
*** gouthamr_ has quit IRC | 10:30 | |
*** lpetrut has joined #openstack-manila | 10:33 | |
*** carthaca_ has quit IRC | 10:55 | |
*** sapcc-bot has quit IRC | 10:55 | |
openstackgerrit | Merged openstack/manila: Updated from global requirements https://review.openstack.org/408097 | 10:55 |
*** tpsilva has joined #openstack-manila | 11:02 | |
openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Add mountable snapshots support https://review.openstack.org/345526 | 11:02 |
*** zengyingzhe has quit IRC | 11:12 | |
*** zengyingzhe has joined #openstack-manila | 11:13 | |
*** xinyanzhang has quit IRC | 11:31 | |
openstackgerrit | zhongjun proposed openstack/manila: Add share_type filter support to pool_list https://review.openstack.org/365617 | 11:46 |
openstackgerrit | Akira KAMIO proposed openstack/manila: Fix nfs server service name for rhel, suse https://review.openstack.org/407846 | 11:51 |
*** gouthamr has quit IRC | 11:55 | |
openstackgerrit | zhongjun proposed openstack/manila: Add share_type filter support to pool_list https://review.openstack.org/365617 | 11:56 |
*** sapcc-bot has joined #openstack-manila | 11:59 | |
*** tpatzig_ has joined #openstack-manila | 11:59 | |
*** carthaca_ has joined #openstack-manila | 11:59 | |
*** databus23_ has joined #openstack-manila | 11:59 | |
*** dgonzalez_ has joined #openstack-manila | 11:59 | |
*** carthaca_ has quit IRC | 12:01 | |
*** databus23_ has quit IRC | 12:01 | |
*** tpatzig_ has quit IRC | 12:01 | |
*** dgonzalez_ has quit IRC | 12:01 | |
*** jcsp has quit IRC | 12:02 | |
*** carthaca_ has joined #openstack-manila | 12:05 | |
*** databus23_ has joined #openstack-manila | 12:05 | |
*** mkoderer_ has joined #openstack-manila | 12:05 | |
*** tommy_ has joined #openstack-manila | 12:05 | |
*** dgonzalez_ has joined #openstack-manila | 12:05 | |
*** tpatzig_ has joined #openstack-manila | 12:05 | |
*** tommy_ is now known as Guest42909 | 12:05 | |
openstackgerrit | Akira KAMIO proposed openstack/manila: Fix nfs server service name for rhel, suse https://review.openstack.org/407846 | 12:06 |
*** carthaca_ has quit IRC | 12:07 | |
*** databus23_ has quit IRC | 12:07 | |
*** dgonzalez_ has quit IRC | 12:07 | |
*** Guest42909 has quit IRC | 12:07 | |
*** mkoderer_ has quit IRC | 12:07 | |
*** tpatzig_ has quit IRC | 12:07 | |
*** jcsp has joined #openstack-manila | 12:17 | |
*** erlon-airlong has joined #openstack-manila | 12:18 | |
ganso | vponomaryov: you listed 4 things | 12:26 |
vponomaryov | ganso: and? | 12:26 |
ganso | vponomaryov: "Two things: 1), 2), 3), 4)" | 12:26 |
vponomaryov | ganso; and? | 12:27 |
ganso | vponomaryov: just pointing out | 12:27 |
vponomaryov | does it make your mind blow up? )) | 12:27 |
ganso | vponomaryov: a bit :P | 12:27 |
vponomaryov | your +2 there makes mine blow up | 12:27 |
vponomaryov | we're even | 12:28 |
vponomaryov | ^_^ | 12:28 |
ganso | vponomaryov: indeed, I just looked at the code again and noticed I had confused it with another patch | 12:29 |
ganso | vponomaryov: I thought only the reno was missing | 12:29 |
*** surabujin has quit IRC | 12:43 | |
openstackgerrit | Akira KAMIO proposed openstack/manila: Fix nfs server service name for rhel, suse https://review.openstack.org/407846 | 12:54 |
*** jcsp has quit IRC | 12:56 | |
openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Add mountable snapshots support https://review.openstack.org/345526 | 12:59 |
*** surabujin has joined #openstack-manila | 13:03 | |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [LVM,Generic drivers] Fix relationships between parent and child shares https://review.openstack.org/407468 | 13:19 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [Tempest] Add scenario test creating share from snapshot https://review.openstack.org/400758 | 13:20 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [DNM] Test scenario tests on three drivers https://review.openstack.org/402404 | 13:20 |
openstackgerrit | Valeriy Ponomaryov proposed openstack/manila: [Generic driver] Fix generation of admin export location https://review.openstack.org/407501 | 13:27 |
*** narayrak has quit IRC | 13:37 | |
*** tommylikehu_ has joined #openstack-manila | 13:43 | |
tommylikehu_ | ping tommylikehu_ | 13:49 |
*** absubram has quit IRC | 13:51 | |
*** jprovazn has quit IRC | 13:52 | |
openstackgerrit | Akira KAMIO proposed openstack/manila: Fix nfs server service name for rhel, suse https://review.openstack.org/407846 | 13:53 |
*** porrua has joined #openstack-manila | 13:59 | |
bswartz | ganso vponomaryov tbarron cknight: ping | 14:08 |
openstackgerrit | Ben Swartzlander proposed openstack/manila-specs: Eliminate Race Conditions Spec https://review.openstack.org/396255 | 14:14 |
openstackgerrit | Ben Swartzlander proposed openstack/manila-specs: Eliminate Race Conditions Spec https://review.openstack.org/396255 | 14:14 |
*** porrua_ has joined #openstack-manila | 14:16 | |
*** porrua has quit IRC | 14:19 | |
*** gouthamr has joined #openstack-manila | 14:20 | |
*** StraubTW has joined #openstack-manila | 14:26 | |
*** eharney has joined #openstack-manila | 14:26 | |
*** eharney has quit IRC | 14:33 | |
*** dustins has joined #openstack-manila | 14:36 | |
*** dustins has quit IRC | 14:43 | |
*** dustins has joined #openstack-manila | 14:43 | |
vponomaryov | bswartz: pong | 14:44 |
*** eharney has joined #openstack-manila | 14:47 | |
bswartz | vponomaryov: nm | 14:48 |
*** xyang_ has joined #openstack-manila | 14:55 | |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 14:58 |
*** wiebalck has joined #openstack-manila | 14:58 | |
*** wiebalck has quit IRC | 15:03 | |
*** kaisers_ has quit IRC | 15:03 | |
*** cknight has joined #openstack-manila | 15:03 | |
*** nherciu has quit IRC | 15:03 | |
*** chlong has joined #openstack-manila | 15:13 | |
*** jprovazn has joined #openstack-manila | 15:14 | |
*** alyson_ has joined #openstack-manila | 15:20 | |
*** kaisers_ has joined #openstack-manila | 15:33 | |
*** cdelatte has joined #openstack-manila | 15:43 | |
*** delattec has joined #openstack-manila | 15:43 | |
*** delattec has quit IRC | 15:44 | |
*** cdelatte has quit IRC | 15:44 | |
*** cdelatte has joined #openstack-manila | 15:44 | |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 15:47 |
*** mtanino has joined #openstack-manila | 15:49 | |
tbarron | markstur: you have a life? | 16:01 |
markstur | tbarron: i have memories | 16:01 |
ganso | bswartz: btw: "openstack server create [--nic <net-id=net-uuid,v4-fixed-ip=ip-addr,v6-fixed-ip=ip-addr,port-id=port-uuid>]" | 16:01 |
bswartz | ganso: I'm still confused about the purpose of waiting | 16:02 |
ganso | bswartz: the gross syntax is still there in openstack client | 16:02 |
bswartz | merging your fix will speed up the work that needs to be done -- waiting just creates and excuse to delay the configref work | 16:02 |
ganso | bswartz: the purpose is giving a chance to move docs before they are wiped | 16:02 |
bswartz | ganso: sorry I'm on the devref thing still | 16:02 |
bswartz | ganso: they won't be wiped! | 16:02 |
bswartz | they'll just be missing from HEAD | 16:03 |
ganso | bswartz: indeed, I agree, I stated this yesterday while talking to tbarron | 16:03 |
*** rraja has quit IRC | 16:03 | |
ganso | bswartz: Initially I was up for removing right now, as it was decided | 16:03 |
ganso | bswartz: since it will create the urge to create the docs where they should be | 16:03 |
tbarron | I just don't want to field questions about where did they go and tell people to use git history. | 16:03 |
bswartz | so who argues for anything other than immediate removal? | 16:03 |
ganso | bswartz: but I agree it is fair to have a deadline | 16:03 |
bswartz | tbarron: tell them stable/newton | 16:03 |
tbarron | I will refer all questions to bswartz and ganso | 16:03 |
bswartz | tbarron: :-D | 16:03 |
ganso | tbarron: lol | 16:04 |
*** adrianofr_ has joined #openstack-manila | 16:04 | |
vponomaryov | bswartz: I have other variant | 16:04 |
tbarron | nothiinng I'm dug in about | 16:04 |
vponomaryov | bswartz; we do not remove it, but replace all inner data with reference to proper place | 16:04 |
ganso | tbarron: link them the removal patch and say "that fella over there removed them" | 16:04 |
*** nkrinner is now known as nkrinner_afk | 16:04 | |
tbarron | vponomaryov: I suggested that, the problem is that the proper places are not there yet | 16:04 |
ganso | vponomaryov: yes, reference to limbo in that case | 16:05 |
vponomaryov | tbarron: then, obviously, we make them first, then replace devref | 16:05 |
ganso | vponomaryov: I already addressed in the removel patch saying "look for the config ref" | 16:05 |
vponomaryov | ganso: should also be link | 16:05 |
gouthamr | bswartz ganso: they will be removed from the current place with no proper replacement.. | 16:06 |
gouthamr | bswartz ganso: not everyone goes through our git repo | 16:06 |
bswartz | I think removal will light a fire under anyone who cares | 16:07 |
tbarron | vponomaryov: I suggested that too, in the review | 16:07 |
gouthamr | bswartz ganso: http://docs.openstack.org/developer/manila may have some traffic we don't know about - s | 16:07 |
bswartz | this is a bandaid that needs to be ripped off | 16:07 |
vponomaryov | tbarron: good, because I kind of insist on such solution | 16:07 |
bswartz | I never dreamed there would be so much resistance to moving the docs to where they belong | 16:08 |
tbarron | vponomaryov: that doesn't help with ganso and bswartz's urge to rip the band aid | 16:08 |
vponomaryov | bswartz: itis ok to move, but in devref should stay reference "go there to find to you wanted to find here" | 16:08 |
tbarron | bswartz: ganso as I said, I'm not dug in, got other stuff to work on | 16:09 |
vponomaryov | s/to you/that you/ | 16:09 |
bswartz | vponomaryov: you mean a top level link right? not separate links for each driver | 16:09 |
vponomaryov | bswartz: yes, the one for page where we see list of all driver docs | 16:10 |
*** sandanar has quit IRC | 16:10 | |
bswartz | vponomaryov: link? | 16:10 |
tbarron | it's going to take a little time and thought that's not in my budget between now and PTG to do cephfs, ganesha, generic, zfs on linux, probably glusterfs drivers, etc. | 16:10 |
tbarron | none of it is hard | 16:10 |
tbarron | just not work that is scheduled for anybody | 16:11 |
vponomaryov | bswartz: http://docs.openstack.org/developer/manila/devref/index.html#share-backends | 16:11 |
vponomaryov | like this | 16:11 |
tbarron | I'd rather see us working on other things until O-3 | 16:11 |
bswartz | vponomaryov: k | 16:11 |
tommylikehu_ | hey all | 16:12 |
bswartz | tommylikehu_: for future reference, when you add topics to the meeting agenda, add them as top level bullets rather than under the open discussion bullet | 16:13 |
vponomaryov | bswartz: I guess, this one -> http://docs.openstack.org/newton/config-reference/shared-file-systems/drivers.html | 16:13 |
bswartz | just add them at the bottom of the list above the recurring topics | 16:13 |
tommylikehu_ | bswartz: sure, | 16:14 |
bswartz | vponomaryov: yeah that could easily be link to the adminref/configref | 16:14 |
tommylikehu_ | my topic is easy enough to put and discussed here :) | 16:15 |
tbarron | tommylikehu_: yeah, it's more interesting than this doc stuff too! | 16:15 |
tommylikehu_ | thanks tbarron :) | 16:16 |
tommylikehu_ | I think I get stuck at the network plugin on IPv6 spec, please take some time on the spec :https://review.openstack.org/#/c/362786/ | 16:16 |
tommylikehu_ | I uploaded an draft version, and any constructive comment would help me a lot:) | 16:17 |
tbarron | bswartz: vponomaryov do we just need to say here that the plugins were in general written to be address family agnostic, but that a good code inspectioin and lots of testing will need to be done as part of the implementation? | 16:18 |
tbarron | bswartz: vponomaryov there was some idea that validation would be done against the plugins for admin and for regular net allocations?? | 16:19 |
*** pcaruana has quit IRC | 16:19 | |
bswartz | tbarron: no it's not something we can handwave around | 16:19 |
bswartz | explicit support is needed for both v6-only and v4+v6 configs | 16:19 |
* bswartz grumbles about the effort required to move everything from trusty to xenial | 16:20 | |
tbarron | bswartz: config in manila.conf for the network plugins themselves? | 16:20 |
bswartz | tbarron in the manager code and uses them | 16:21 |
bswartz | the manager currently gets N v4 addresses from the plugin and gives them to the driver, where N is the value of get_network_allocations_number() | 16:22 |
vponomaryov | bswartz: actually N of addresses of network, without regards to version | 16:23 |
tbarron | does it get exclusively v4 addresses if neutron is running v6? | 16:23 |
tbarron | jinx | 16:23 |
tbarron | bswartz: yeah, that's what I'm seeing in the code | 16:23 |
bswartz | for v6-only, we need to make sure we're getting v6 addresses, and for v4+v6, the manager needs to get 2×N addresses | 16:23 |
tbarron | there are very few places that are v4 dependent that I see, and those look like easy bugs to fix | 16:24 |
bswartz | vponomaryov: right, so for v4+v6 you actually need twice the number of addresses | 16:24 |
tbarron | bswartz: so we would configure v6 only, for example, in the network api class or a spinoff config for it | 16:24 |
bswartz | get_network_allocations_number() is the number of new ports on the network | 16:24 |
vponomaryov | bswartz: no, we have such number per network | 16:24 |
vponomaryov | bswartz: not per Ip version | 16:25 |
tbarron | vponomaryov: so today in a v4/v6 net it will be indeterministic whether you allocate v4 or v6, right? | 16:25 |
vponomaryov | yes | 16:25 |
vponomaryov | v6 is not tested | 16:25 |
tbarron | vponomaryov: we probably ought to fix that I think | 16:25 |
vponomaryov | but design should be the same for it | 16:25 |
bswartz | the only thing we claim to support is v4-only | 16:25 |
bswartz | it might be buggy in the presence of v6, which is also something to fix | 16:26 |
bswartz | but we explicitly need to add v6-only and v4+v6 cases and test those | 16:26 |
bswartz | and I'm confident new code is needed in those 2 cases at least | 16:26 |
tbarron | bswartz: so how do we know if we need to allocat just v4, just v6, or both? /etc/manila.conf? | 16:26 |
vponomaryov | bswartz: we support only one net per net plugin | 16:26 |
vponomaryov | how can we consider v4 + v6? | 16:26 |
vponomaryov | network plugin just shoudl say "yes, I able to work with Ip version of THAT network | 16:27 |
vponomaryov | s/I Iam/ | 16:27 |
tbarron | vponomaryov: network_api_class = manila.network.neutron.neutron_network_plugin.NeutronNetworkPlugin doesn't say the neutron net can't be doing both v4 and v6, right? | 16:28 |
vponomaryov | v4 + v6 will take place only after we implement multi-subnet support in share-network | 16:28 |
tommylikehu_ | network can decide the ability to allocate IPv4 or IPv6 and the share network will decide whether we can get both IPv4 and IPv6 ? | 16:28 |
tommylikehu_ | network plugin | 16:28 |
tbarron | vponomaryov: so any given subnet will be one or the other and any share network will (till goutham gets there) only have one subnet | 16:28 |
vponomaryov | tbarron: my point si that we can have only ONE of them at once per network plugin right now | 16:29 |
vponomaryov | tbarron: so, spec should say which case does it really cover | 16:29 |
vponomaryov | cases* | 16:29 |
tbarron | vponomaryov: i'm missing why per network plugin vs why per share network. I see the poiint about one per share network. | 16:29 |
vponomaryov | and describe requried changes for them | 16:29 |
vponomaryov | tbarron: does not matter right now because we have 1to1 relation right now | 16:30 |
vponomaryov | tbarron: + admin network plugin that gets info from config | 16:31 |
vponomaryov | so, two network plugins for two nets | 16:31 |
vponomaryov | is current design | 16:31 |
tommylikehu_ | vponomaryov: should we consider admin network regarding IPv6? | 16:32 |
vponomaryov | tommylikehu_: how do you think? ) | 16:32 |
tommylikehu_ | only user network should considered | 16:32 |
vponomaryov | how then your service will talk to backend if network infra structure is IPv6 only? | 16:33 |
tommylikehu_ | how many driver will require a admin network? | 16:34 |
vponomaryov | I am not prophet | 16:35 |
vponomaryov | the most secure approach is to use separate network for service needs | 16:35 |
tbarron | tommylikehu_: for stuff like migration we need to be able to mount at admin export locations that connect to admin network | 16:35 |
tbarron | vponomaryov: correct as needed ^^^ | 16:35 |
tommylikehu_ | tbarron: ok | 16:36 |
tbarron | vponomaryov: what I don't understand is how when we have 'network_api_class = manila.network.neutron.neutron_network_plugin.NeutronNetworkPlugin' that implies that the underlying network is only ipv4 or only ipv6 | 16:37 |
tbarron | vponomaryov: but maybe that's not what you are saying and I misunderstand | 16:37 |
tbarron | vponomaryov: I get that we are doing allocations *as if* that were true. That's what bswartz is saying we should fix if i understand him correctly | 16:38 |
vponomaryov | tbarron: exactly this network plugin gets info from share-network where we sepcify neutron subnet that has explicit argument that we get via API saying which Ip version it is | 16:38 |
vponomaryov | I loooove complex sentenses ^_^ | 16:39 |
tbarron | vponomaryov: so what prevents me from making one share net with ipv4 subnet and another with ipv6 subnet using the same plugin | 16:39 |
vponomaryov | tbarron: nothing example bugs that can have a party there without any testing | 16:39 |
vponomaryov | s/example/except/ | 16:39 |
openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Add mountable snapshots support https://review.openstack.org/345526 | 16:40 |
tbarron | ok, so I think bswartz is arguing (now it's his turn to correct me) that we should add a bit of config, say, neutron_network_plugin_address_families = '4', '6', or '4,6' | 16:41 |
tbarron | and that we should revamp the allocation code accordingly | 16:41 |
*** tpatzig_ has joined #openstack-manila | 16:42 | |
tbarron | vponomaryov: bswartz and perhaps an incremental step would be that we only allow '4' or '6' today | 16:42 |
tbarron | vponomaryov: bswartz tommylikehu_ until the plugiin code is ready for both | 16:42 |
tbarron | both at the same time | 16:43 |
tbarron | default could be '4', backwards compatible | 16:43 |
vponomaryov | tbarron: spec is for such things, what is expected to be supported, what should be changed for it and what are the requirements for it | 16:43 |
*** tpatzig_ has quit IRC | 16:44 | |
tommylikehu_ | I add some words in the spec that driver should consider the network support ip version before he report the capability | 16:44 |
tbarron | vponomaryov: agree, and to the extent you and bswartz can help, that's great. Not many people know the net plugin area well yet I think. | 16:44 |
tommylikehu_ | does this make sense? | 16:44 |
vponomaryov | tbarron, in theory, this spec could say that DHSS=True case is not supported yet and describe changes only for DHSS=False mode | 16:44 |
tbarron | vponomaryov: interesting. | 16:45 |
vponomaryov | it is use case because we have too few DHSS=True drivers yet | 16:46 |
tbarron | vponomaryov: bswartz speaking of simplifying assumptions, can we assume nova networking is dead now and nova plugins can be ignored for this issue? | 16:46 |
vponomaryov | tbarron: yes | 16:46 |
vponomaryov | tbarron: actually we'd better to remove it | 16:47 |
tbarron | tommylikehu_: ^^ confirmed. | 16:47 |
tbarron | vponomaryov: agree | 16:47 |
tommylikehu_ | tbarron: great | 16:47 |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 16:48 |
tbarron | tommylikehu_: you say only IPv4 would be supported for Standalone plugin but I didn't see what that *has* to be true. | 16:48 |
vponomaryov | tbarron: just look at source code and its configuration ) | 16:49 |
vponomaryov | for that net plugin | 16:49 |
tbarron | vponomaryov: but it would be easy to change the gateway and netmasdk to allow ipv6 | 16:49 |
tbarron | vponomaryov: tommylikehu_ but that could be followup, not in scope for now. | 16:50 |
vponomaryov | tbarron: StandAloneNetwork plugin is designed to support both from the begining | 16:50 |
vponomaryov | tbarron: v6 is just not tested there | 16:50 |
tbarron | vponomaryov: and the config is written for v4, but that's not a big deal | 16:50 |
vponomaryov | tbarron: https://github.com/openstack/manila/blob/e3afcd75/manila/network/standalone_network_plugin.py#L63 | 16:51 |
tbarron | tommylikehu_: but that's probably good enough to say v6 is out of scope for StandAlone for the first iteration as described in this spec. | 16:51 |
tbarron | vponomaryov: I see, it's just the comments in the sample config that seem to imply v4 only | 16:52 |
vponomaryov | tbarron: it is not config, it is "help" messages | 16:52 |
tbarron | vponomaryov: +1 | 16:52 |
tbarron | tommylikehu_: it must be 1am for you? | 16:53 |
tommylikehu_ | tbarron: yes | 16:54 |
tommylikehu_ | almost 1am | 16:54 |
tbarron | tommylikehu_: you should get some sleep but I'll email you some ideas later, or put them in the review | 16:54 |
tbarron | tommylikehu_: we need vponomaryov and bswartz to help us out thought as both of us are new in the network plugin space | 16:55 |
tommylikehu_ | tbarron: : ) thanks for you help | 16:55 |
tommylikehu_ | I gonna sleep in few minutes | 16:55 |
tommylikehu_ | thanks vponomaryov and bswartz :) | 16:56 |
*** tommylikehu_ has quit IRC | 16:57 | |
*** lpetrut has quit IRC | 17:12 | |
*** tpatzig_ has joined #openstack-manila | 17:13 | |
*** tpatzig_ has quit IRC | 17:15 | |
vponomaryov | core team, lookign for help merging one-liner fix for scenario job -> https://review.openstack.org/#/c/408204 | 17:25 |
* gouthamr rereads valeriy's comment. realizes he already made the memoization comment, re-reads tbarron's comment about that, smacks himself in the head. needs sleep | 17:44 | |
* gouthamr #link https://review.openstack.org/#/c/407846 for anyone wondering ^ | 17:45 | |
tbarron | vponomaryov: sorry, thought I'd already done that one. | 17:49 |
vponomaryov | tbarron: thanks | 17:54 |
*** xyang_ has quit IRC | 18:04 | |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 18:07 |
*** xyang_ has joined #openstack-manila | 18:09 | |
*** StraubTW has quit IRC | 18:10 | |
*** StraubTW has joined #openstack-manila | 18:10 | |
*** harlowja has joined #openstack-manila | 18:17 | |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 18:19 |
*** harlowja_ has joined #openstack-manila | 18:20 | |
*** harlowja has quit IRC | 18:22 | |
*** lpetrut has joined #openstack-manila | 18:22 | |
*** xyang_ has quit IRC | 18:27 | |
*** xyang_ has joined #openstack-manila | 18:31 | |
*** nherciu has joined #openstack-manila | 18:31 | |
openstackgerrit | Merged openstack/manila: [Tempest] Fix concurrency issue in scenario test https://review.openstack.org/408204 | 18:31 |
*** mkoderer has quit IRC | 18:36 | |
*** xyang_ has quit IRC | 18:38 | |
*** xyang_ has joined #openstack-manila | 18:40 | |
*** xyang_ has quit IRC | 18:41 | |
*** xyang_ has joined #openstack-manila | 18:42 | |
*** xyang_ has quit IRC | 18:43 | |
openstackgerrit | Tiago Pasqualini da Silva proposed openstack/manila: Add DriverFilter and GoodnessWeigher documentation https://review.openstack.org/379801 | 18:46 |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 19:03 |
openstackgerrit | Goutham Pacha Ravi proposed openstack/manila: Add create_share_from_snapshot_support extra spec https://review.openstack.org/356682 | 19:11 |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 19:14 |
openstackgerrit | Goutham Pacha Ravi proposed openstack/manila-specs: Add a spec to fix and improve Access Rules https://review.openstack.org/399049 | 19:22 |
*** xyang_ has joined #openstack-manila | 19:28 | |
*** xyang_ has quit IRC | 19:29 | |
*** lpetrut has quit IRC | 19:35 | |
*** xyang_ has joined #openstack-manila | 19:44 | |
openstackgerrit | craig delatte proposed openstack/puppet-manila: Fix duplicate declaration between manila/tripleO https://review.openstack.org/408658 | 19:45 |
*** xyang_ has quit IRC | 20:10 | |
*** xyang_ has joined #openstack-manila | 20:12 | |
*** absubram has joined #openstack-manila | 20:16 | |
*** xyang_ has quit IRC | 20:22 | |
*** xyang_ has joined #openstack-manila | 20:28 | |
*** kaisers_ has quit IRC | 20:43 | |
*** kaisers_ has joined #openstack-manila | 20:48 | |
*** kaisers_ has quit IRC | 20:53 | |
*** xyang1 has joined #openstack-manila | 21:02 | |
*** alyson_ has quit IRC | 21:03 | |
*** xyang_ has quit IRC | 21:04 | |
*** kaisers_ has joined #openstack-manila | 21:04 | |
*** chlong has quit IRC | 21:10 | |
*** xyang_ has joined #openstack-manila | 21:16 | |
*** xyang_ has quit IRC | 21:17 | |
*** xyang_ has joined #openstack-manila | 21:20 | |
*** jprovazn has quit IRC | 21:28 | |
*** StraubTW has quit IRC | 21:33 | |
*** dsariel has quit IRC | 21:34 | |
xyang1 | vponomaryov gouthamr ping | 21:37 |
*** eharney has quit IRC | 21:54 | |
*** xyang_ has quit IRC | 22:11 | |
*** tpsilva has quit IRC | 22:11 | |
*** xyang_ has joined #openstack-manila | 22:12 | |
*** gouthamr has quit IRC | 22:15 | |
*** bwolfe_ has joined #openstack-manila | 22:17 | |
*** bwolfe_ has quit IRC | 22:17 | |
*** ganso has quit IRC | 22:23 | |
*** lpetrut has joined #openstack-manila | 22:26 | |
*** nherciu has quit IRC | 22:34 | |
*** xyang_ has quit IRC | 22:35 | |
*** xyang_ has joined #openstack-manila | 22:44 | |
*** gregsfortytwo has quit IRC | 22:44 | |
*** porrua_ has quit IRC | 22:46 | |
*** david-lyle_ has joined #openstack-manila | 23:05 | |
*** tommylikehu_ has joined #openstack-manila | 23:06 | |
*** lpetrut has quit IRC | 23:07 | |
*** david-lyle has quit IRC | 23:07 | |
*** tommylikehu_ has quit IRC | 23:07 | |
*** xyang1 has quit IRC | 23:19 | |
*** absubram has quit IRC | 23:20 | |
*** cknight has quit IRC | 23:24 | |
*** david-lyle_ is now known as david-lyle | 23:35 | |
*** erlon-airlong has quit IRC | 23:40 | |
*** dustins has quit IRC | 23:41 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!