*** yangyapeng has quit IRC | 00:02 | |
*** vbellur has quit IRC | 00:58 | |
*** bswartz has joined #openstack-manila | 01:02 | |
*** bswartz has quit IRC | 01:13 | |
*** yangyapeng has joined #openstack-manila | 01:17 | |
*** vbellur has joined #openstack-manila | 01:31 | |
zengyingzhe_ | cknight, ping | 01:33 |
---|---|---|
*** cknight has left #openstack-manila | 01:46 | |
*** wangqun has joined #openstack-manila | 01:46 | |
*** yangyape_ has joined #openstack-manila | 02:11 | |
*** yangyapeng has quit IRC | 02:14 | |
*** yangyape_ has quit IRC | 02:23 | |
*** yangyapeng has joined #openstack-manila | 02:24 | |
*** yangyape_ has joined #openstack-manila | 02:24 | |
*** yangyapeng has quit IRC | 02:29 | |
*** dsariel has quit IRC | 03:45 | |
openstackgerrit | zhongjun proposed openstack/manila: Add IPv6 ACL support in Manila https://review.openstack.org/312321 | 03:54 |
*** chlong has joined #openstack-manila | 04:21 | |
*** gouthamr has quit IRC | 04:33 | |
*** bswartz has joined #openstack-manila | 05:07 | |
*** senk_ has joined #openstack-manila | 05:16 | |
*** senk_ has quit IRC | 06:35 | |
*** david-lyle has joined #openstack-manila | 06:36 | |
*** pcaruana has joined #openstack-manila | 06:36 | |
*** houming has joined #openstack-manila | 06:38 | |
*** david-lyle_ has quit IRC | 06:39 | |
*** wangqun has quit IRC | 06:54 | |
*** chlong has quit IRC | 07:08 | |
*** senk_ has joined #openstack-manila | 07:11 | |
*** chlong has joined #openstack-manila | 07:25 | |
*** Guest30635 has quit IRC | 07:29 | |
*** Guest30635 has joined #openstack-manila | 07:29 | |
*** Guest30635 is now known as assassin | 07:29 | |
*** lpetrut has joined #openstack-manila | 07:34 | |
*** wangqun has joined #openstack-manila | 07:53 | |
*** yangyape_ has quit IRC | 07:54 | |
*** yangyapeng has joined #openstack-manila | 07:57 | |
*** wangqun has quit IRC | 07:59 | |
*** yangyapeng has quit IRC | 08:06 | |
*** yangyapeng has joined #openstack-manila | 08:06 | |
*** chlong has quit IRC | 08:07 | |
*** aovchinnikov has joined #openstack-manila | 08:07 | |
openstackgerrit | zhongjun proposed openstack/python-manilaclient: Implement share backup https://review.openstack.org/344671 | 08:19 |
openstackgerrit | yuyafei proposed openstack/python-manilaclient: Add __ne__ built-in function https://review.openstack.org/355157 | 08:28 |
*** houming has quit IRC | 08:30 | |
*** houming has joined #openstack-manila | 08:40 | |
*** lpetrut has quit IRC | 08:47 | |
*** houming has quit IRC | 08:50 | |
*** houming has joined #openstack-manila | 08:57 | |
*** lpetrut has joined #openstack-manila | 08:58 | |
openstackgerrit | zhongjun proposed openstack/manila: Implement share backup https://review.openstack.org/343980 | 09:10 |
*** lpetrut has quit IRC | 09:30 | |
openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 09:52 |
*** wangqun has joined #openstack-manila | 09:57 | |
*** wangqun has quit IRC | 10:05 | |
*** yangyapeng has quit IRC | 10:20 | |
openstackgerrit | Erno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic https://review.openstack.org/353928 | 10:27 |
openstackgerrit | Dai Dang Van proposed openstack/manila: Implement extend and shrink features for glusterfs https://review.openstack.org/331389 | 10:34 |
*** lpetrut has joined #openstack-manila | 10:45 | |
*** lpetrut has quit IRC | 11:06 | |
*** yangyapeng has joined #openstack-manila | 11:23 | |
*** furlongm has joined #openstack-manila | 11:51 | |
*** furlongm_ has quit IRC | 11:51 | |
*** JoseMello has joined #openstack-manila | 12:09 | |
*** gouthamr has joined #openstack-manila | 12:10 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: Container driver https://review.openstack.org/308930 | 12:31 |
*** mtanino has joined #openstack-manila | 12:35 | |
mkoderer__ | tbarron: ping | 12:49 |
*** xyang1 has joined #openstack-manila | 12:56 | |
tbarron | mkoderer__: pong | 12:58 |
mkoderer__ | tbarron: just commented on https://review.openstack.org/#/c/308930/78 | 13:03 |
tbarron | mkoderer__: looking | 13:03 |
mkoderer__ | tbarron: IMO we need to state that the driver is not for production usage and let it merge | 13:03 |
mkoderer__ | currently we are blocking features that are network releated since all wait for this driver | 13:04 |
*** houming has quit IRC | 13:04 | |
tbarron | mkoderer__: yeah, that fits with my understanding, i just want more or less what you said put in the devref for this driver. aovchinnikov already has gotten most of what I asked for. Then it can merge IMO. | 13:06 |
mkoderer__ | tbarron: +1 | 13:07 |
tbarron | mkoderer__: aovchinnikov downstream i want to be able to point to the devref when a product manager or customer wants to go a kilo too far with it | 13:07 |
aovchinnikov | tbarron: ok, let's remove the gate part from known issues then | 13:09 |
tbarron | mkoderer__: aovchinnikov sure, and would you open a bug for the concurrency issues as mkoderer__ suggests and put it in the commit msg as a related bug? | 13:10 |
mkoderer__ | aovchinnikov: like here: https://review.openstack.org/#/c/349506/ | 13:12 |
mkoderer__ | this fix also waits for the docker driver btw :P | 13:12 |
tbarron | mkoderer__: point taken :) | 13:12 |
aovchinnikov | tbarron: somehow I don't like the idea of registering bugs for non-existent features | 13:13 |
tbarron | aovchinnikov: yeah, i just meant for the concurrency issue | 13:13 |
tbarron | aovchinnikov: it's not a "feature" that one can run concurrently w/o breakage | 13:13 |
aovchinnikov | tbarron: yes, but since there is no container driver in the main tree yet how one should refer to it in the bug description? | 13:14 |
aovchinnikov | that is what I am concerned about | 13:14 |
aovchinnikov | it seems a bit like "when we merge this feature we'll have this bug" to me | 13:15 |
aovchinnikov | which probably means that we should not merge this feature in the first place, right? | 13:15 |
tbarron | aovchinnikov: i see, well i was just trying to log technical debt as we commit. | 13:16 |
tbarron | aovchinnikov: no, we should merge this but very clearly state its limits currently. | 13:16 |
mkoderer__ | aovchinnikov: tbarron: was the same for the "MTU feature" - I think the idea is that the bug is already assigned and clear whos doing it | 13:16 |
aovchinnikov | on the other hand when the driver is in the tree then it is the right time to file it's bugs | 13:17 |
tbarron | aovchinnikov: well I don't want to get stuck on process here. | 13:17 |
tbarron | aovchinnikov: mkoderer__ does the devref now state exactly the intended scope for this driver and the known limitations? (I'm in the waiting room for a doc appt and have intermittent connectivity) | 13:18 |
tbarron | aovchinnikov: mkoderer__ when you are confident we've scoped it right in the doc, let me know, i'll take one more look and very likely +W | 13:19 |
tbarron | trying work together here - back in 10-20m | 13:19 |
mkoderer__ | tbarron: thanks! | 13:21 |
aovchinnikov | also I don't think it is a good idea to label it 'dev-only' in the docs | 13:23 |
*** dustins has joined #openstack-manila | 13:23 | |
mkoderer__ | aovchinnikov: would you say it's production ready? | 13:23 |
gouthamr | I don't see anything wrong with labeling it dev only.. i like the idea until we iron out the concurrency issues that you note.. | 13:24 |
aovchinnikov | no, it is not, but suppose we label it 'dev-only' now and improve it some time later | 13:24 |
aovchinnikov | how are we going to undo the first impression? | 13:25 |
gouthamr | release specific configuration reference, release notes can convey that as we go.. | 13:25 |
aovchinnikov | so I believe it boils down to choosing of accurate wording | 13:25 |
mkoderer__ | aovchinnikov: yes it's just about documentation and that's all | 13:26 |
*** porrua has joined #openstack-manila | 13:27 | |
mkoderer__ | gouthamr: did you test https://review.openstack.org/#/c/355254/1 with a jumbo frame setup? | 13:27 |
mkoderer__ | gouthamr: I added a user-story for our team to verify it end2end.. | 13:28 |
gouthamr | mkoderer__: i've verified it sets the MTU value on the VLAN ports; haven't done end to end testing yet.. ONTAP would lie? :) | 13:28 |
gouthamr | mkoderer__: just kidding.. nice! i actually dunno how to verify jumbo frames are supported. how would you do that? | 13:29 |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: Container driver https://review.openstack.org/308930 | 13:30 |
aovchinnikov | ok, here it goes | 13:30 |
mkoderer__ | gouthamr: yeah I think there are way's with pings to verfify packages are not fragmeneted somewhere | 13:30 |
mkoderer__ | gouthamr: need to ask my network guru's | 13:31 |
openstackgerrit | Accela Zhao proposed openstack/python-manilaclient: Add driver private storage admin API https://review.openstack.org/336581 | 13:32 |
mkoderer__ | gouthamr: iperf would be also a solution | 13:32 |
gouthamr | mkoderer__: thanks, will take a look.. | 13:33 |
*** vbellur has quit IRC | 13:36 | |
*** akerr has joined #openstack-manila | 13:37 | |
mkoderer__ | gouthamr: ping -M do -s 9000 <NetApp box> | 13:45 |
*** vbellur has joined #openstack-manila | 13:46 | |
*** vbellur has quit IRC | 13:51 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests https://review.openstack.org/340236 | 13:51 |
*** chlong has joined #openstack-manila | 13:52 | |
tbarron | bswartz: https://review.openstack.org/#/c/308930/79 can get your +2 back and prob +W now | 13:59 |
*** aovchinnikov has quit IRC | 14:00 | |
bswartz | tbarron: ty | 14:02 |
*** mtanino has quit IRC | 14:09 | |
openstackgerrit | Andrew Kerr proposed openstack/manila: [WIP] Add tests for share groups https://review.openstack.org/355264 | 14:13 |
openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/puppet-manila: Add authtoken class https://review.openstack.org/351792 | 14:18 |
*** lpetrut has joined #openstack-manila | 14:24 | |
openstackgerrit | Accela Zhao proposed openstack/manila: Driver private data admin API https://review.openstack.org/315346 | 14:27 |
*** cknight has joined #openstack-manila | 14:28 | |
*** pcaruana has quit IRC | 14:30 | |
*** rraja has quit IRC | 14:32 | |
*** JoseMello has quit IRC | 14:40 | |
*** vbellur has joined #openstack-manila | 14:40 | |
mkoderer__ | xyang1: ping | 14:49 |
xyang1 | mkoderer__: hi | 14:53 |
mkoderer__ | xyang1: I am not sure about https://review.openstack.org/#/c/315346/ | 14:54 |
mkoderer__ | xyang1: do we really want to extend the api with driver specific extra specs? | 14:54 |
xyang1 | mkoderer__: I think it is useful to have a way to validate those data | 14:55 |
*** ociuhandu has joined #openstack-manila | 14:55 | |
mkoderer__ | xyang1: IMHO the openstack API should be seen as abstraction layer for different vendors/drivers | 14:56 |
xyang1 | mkoderer__: this is an abstraction, one api to retrieve the data | 14:56 |
mkoderer__ | xyang1: yeah that's the typical vendor trick to lock you in.. just definde the interface broad engough ;) | 14:57 |
xyang1 | mkoderer__: I don't get what you meant. this is just a generic api to read data | 14:58 |
mkoderer__ | xyang1: 1.) the data is named "private" and is now availible trough api | 14:59 |
xyang1 | mkoderer__: it is admin api | 14:59 |
xyang1 | mkoderer__: admin should be allowed to read private data | 15:00 |
*** akerr is now known as akerr_away | 15:00 | |
*** akerr_away is now known as akerr | 15:00 | |
mkoderer__ | xyang1: but it's "private" driver data nodody should really care about | 15:00 |
mkoderer__ | xyang1: 2.) for me changing the api just because of 3rd CI testing isn't a good enough reason | 15:01 |
xyang1 | mkoderer__: I don't agree nobody should care about it. if it is saved, it is useful | 15:01 |
mkoderer__ | xyang1: e.g. there is alot in neutron that isn't publish in the api. And all vendor specific tables aren't readable api-wise | 15:03 |
xyang1 | mkoderer__: I don't know hiw neutron makes decisions, but this api is useful for manila | 15:05 |
mkoderer__ | xyang1: ok I just wanted to raise my doubts - I am fine if others do see it differently | 15:05 |
xyang1 | mkoderer__: ok, thanks for reviewing and raises your concerns | 15:05 |
*** senk_ has quit IRC | 15:06 | |
*** lpetrut has quit IRC | 15:06 | |
openstackgerrit | Thomas Bechtold proposed openstack/manila: Fix sample config generation for cinder,nova and neutron opts https://review.openstack.org/355517 | 15:06 |
mkoderer__ | yeah docker driver is merging :) | 15:09 |
*** Yogi1 has joined #openstack-manila | 15:10 | |
openstackgerrit | Alex O'Rourke proposed openstack/manila: 3PAR: Add update_access support https://review.openstack.org/339795 | 15:32 |
*** dsariel has joined #openstack-manila | 15:54 | |
*** mtanino has joined #openstack-manila | 15:57 | |
*** Suyi has joined #openstack-manila | 16:22 | |
openstackgerrit | Merged openstack/manila: NetApp cDOT: Apply network MTU to VLAN ports https://review.openstack.org/355254 | 16:26 |
*** yangyapeng has quit IRC | 16:31 | |
openstackgerrit | Alexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests https://review.openstack.org/340236 | 16:43 |
openstackgerrit | Tatiana Ovchinnikova proposed openstack/manila-ui: Additional fields for Create/Update Share Type forms https://review.openstack.org/338504 | 16:46 |
*** aovchinnikov has joined #openstack-manila | 16:50 | |
*** harlowja has joined #openstack-manila | 17:08 | |
*** senk_ has joined #openstack-manila | 17:45 | |
*** JoseMello has joined #openstack-manila | 17:47 | |
openstackgerrit | Dai Dang Van proposed openstack/manila: Implement extend and shrink features for glusterfs https://review.openstack.org/331389 | 18:08 |
*** dsariel has quit IRC | 18:09 | |
*** xyang1 has quit IRC | 18:22 | |
*** xyang1 has joined #openstack-manila | 18:26 | |
*** senk_ has quit IRC | 18:27 | |
*** aovchinnikov has quit IRC | 18:30 | |
*** dsariel has joined #openstack-manila | 18:30 | |
bswartz | tbarron: do you know what phusion is? | 19:30 |
*** vbellur has quit IRC | 19:31 | |
tbarron | bswartz: no, guessing is it this? https://github.com/phusion/baseimage-docker | 19:33 |
bswartz | tbarron: yeah it | 19:33 |
bswartz | tbarron: yeah it's an off-the-shelf container image which has an init system with systemd built in, which we need for the container driver | 19:34 |
bswartz | it happens to be ubuntu based | 19:34 |
tbarron | yeah | 19:34 |
bswartz | I'd love to know if a fedora-based container exists which has an init system like phusion | 19:34 |
bswartz | or centos even | 19:35 |
tbarron | yeah, I'll see what I can find out | 19:36 |
tbarron | so phusion does both upstart and systemd type init? | 19:37 |
tbarron | I ask b/c in the docker driver review I asked if it works with xenial and that hasn't been checked yet. | 19:37 |
bswartz | well I don't know that detail, and honestly it's not relevant | 19:37 |
bswartz | what matters is that there's an init system | 19:37 |
tbarron | Not a blocker, but when infra wants to convert to Xenial it will be relevant, no? | 19:37 |
bswartz | it's quite common for containers to be single-app and to have no init system | 19:37 |
tbarron | right | 19:38 |
bswartz | but we require an init system for the container behind our manila container driver | 19:38 |
bswartz | I seriously doubt xenial will cause any issues | 19:38 |
tbarron | i guess if the init system is inside the contaiiner it's insulaated from what infra does w.r.t. 14.04 vs xenial anyways | 19:38 |
bswartz | yes | 19:38 |
tbarron | ok | 19:39 |
bswartz | aside from the kernel version, very little matters outside the container | 19:39 |
tbarron | right | 19:39 |
bswartz | Honestly I'd be more concerned about changes in docker itself between trusty->xenial | 19:39 |
bswartz | docker is rapidly evolving and could have broken something we depend on | 19:39 |
tbarron | that makes sense | 19:45 |
tbarron | so I just put in a request to a guy who left our storage group at redhat to work on containers what he knows and will let you know what I find out. | 19:45 |
tbarron | It would be cool to be able to run a devstack in fedora or centos or rhel with the container driver in order to be able to work on DHSS=true code w/o having a netapp backend :) but atm that's not a high prio/necessity from my POV. Gate is ubuntu and the main point of the container driver is to enable upstream testing in gate. | 19:48 |
tbarron | and we don't have any hard dependency issues introduced. | 19:48 |
*** senk_ has joined #openstack-manila | 19:49 | |
*** senk_ has quit IRC | 19:50 | |
*** porrua has quit IRC | 20:16 | |
*** Yogi1 has quit IRC | 20:19 | |
*** cargonza has joined #openstack-manila | 20:20 | |
*** akerr has quit IRC | 20:23 | |
*** xyang1 has quit IRC | 20:29 | |
*** xyang1 has joined #openstack-manila | 20:29 | |
*** ociuhandu has quit IRC | 20:31 | |
*** yangyapeng has joined #openstack-manila | 20:33 | |
*** gouthamr has quit IRC | 20:51 | |
*** JoseMello has quit IRC | 21:00 | |
*** lpetrut has joined #openstack-manila | 21:03 | |
openstackgerrit | Merged openstack/manila: Container driver https://review.openstack.org/308930 | 21:04 |
openstackgerrit | Merged openstack/puppet-manila: Add authtoken class https://review.openstack.org/351792 | 21:05 |
*** dsariel has quit IRC | 21:06 | |
*** lpetrut has quit IRC | 21:12 | |
*** dustins has quit IRC | 21:15 | |
*** lpetrut has joined #openstack-manila | 21:23 | |
*** harlowja has quit IRC | 21:55 | |
*** mtanino has quit IRC | 22:03 | |
*** lpetrut has quit IRC | 22:11 | |
*** cknight has quit IRC | 22:14 | |
openstackgerrit | Merged openstack/manila: Tox Upper Constraints - strip out reinstalls for remaining jobs https://review.openstack.org/355255 | 22:14 |
*** bswartz has quit IRC | 22:15 | |
*** xyang1 has quit IRC | 23:08 | |
*** sticker has joined #openstack-manila | 23:08 | |
*** harlowja has joined #openstack-manila | 23:36 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!