Monday, 2016-08-15

*** yangyapeng has quit IRC00:02
*** vbellur has quit IRC00:58
*** bswartz has joined #openstack-manila01:02
*** bswartz has quit IRC01:13
*** yangyapeng has joined #openstack-manila01:17
*** vbellur has joined #openstack-manila01:31
zengyingzhe_cknight, ping01:33
*** cknight has left #openstack-manila01:46
*** wangqun has joined #openstack-manila01:46
*** yangyape_ has joined #openstack-manila02:11
*** yangyapeng has quit IRC02:14
*** yangyape_ has quit IRC02:23
*** yangyapeng has joined #openstack-manila02:24
*** yangyape_ has joined #openstack-manila02:24
*** yangyapeng has quit IRC02:29
*** dsariel has quit IRC03:45
openstackgerritzhongjun proposed openstack/manila: Add IPv6 ACL support in Manila  https://review.openstack.org/31232103:54
*** chlong has joined #openstack-manila04:21
*** gouthamr has quit IRC04:33
*** bswartz has joined #openstack-manila05:07
*** senk_ has joined #openstack-manila05:16
*** senk_ has quit IRC06:35
*** david-lyle has joined #openstack-manila06:36
*** pcaruana has joined #openstack-manila06:36
*** houming has joined #openstack-manila06:38
*** david-lyle_ has quit IRC06:39
*** wangqun has quit IRC06:54
*** chlong has quit IRC07:08
*** senk_ has joined #openstack-manila07:11
*** chlong has joined #openstack-manila07:25
*** Guest30635 has quit IRC07:29
*** Guest30635 has joined #openstack-manila07:29
*** Guest30635 is now known as assassin07:29
*** lpetrut has joined #openstack-manila07:34
*** wangqun has joined #openstack-manila07:53
*** yangyape_ has quit IRC07:54
*** yangyapeng has joined #openstack-manila07:57
*** wangqun has quit IRC07:59
*** yangyapeng has quit IRC08:06
*** yangyapeng has joined #openstack-manila08:06
*** chlong has quit IRC08:07
*** aovchinnikov has joined #openstack-manila08:07
openstackgerritzhongjun proposed openstack/python-manilaclient: Implement share backup  https://review.openstack.org/34467108:19
openstackgerrityuyafei proposed openstack/python-manilaclient: Add __ne__ built-in function  https://review.openstack.org/35515708:28
*** houming has quit IRC08:30
*** houming has joined #openstack-manila08:40
*** lpetrut has quit IRC08:47
*** houming has quit IRC08:50
*** houming has joined #openstack-manila08:57
*** lpetrut has joined #openstack-manila08:58
openstackgerritzhongjun proposed openstack/manila: Implement share backup  https://review.openstack.org/34398009:10
*** lpetrut has quit IRC09:30
openstackgerritErno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic  https://review.openstack.org/35392809:52
*** wangqun has joined #openstack-manila09:57
*** wangqun has quit IRC10:05
*** yangyapeng has quit IRC10:20
openstackgerritErno Kuvaja proposed openstack/puppet-manila: [WIP] Add CephFSNative driver logic  https://review.openstack.org/35392810:27
openstackgerritDai Dang Van proposed openstack/manila: Implement extend and shrink features for glusterfs  https://review.openstack.org/33138910:34
*** lpetrut has joined #openstack-manila10:45
*** lpetrut has quit IRC11:06
*** yangyapeng has joined #openstack-manila11:23
*** furlongm has joined #openstack-manila11:51
*** furlongm_ has quit IRC11:51
*** JoseMello has joined #openstack-manila12:09
*** gouthamr has joined #openstack-manila12:10
openstackgerritAlexey Ovchinnikov proposed openstack/manila: Container driver  https://review.openstack.org/30893012:31
*** mtanino has joined #openstack-manila12:35
mkoderer__tbarron: ping12:49
*** xyang1 has joined #openstack-manila12:56
tbarronmkoderer__: pong12:58
mkoderer__tbarron: just commented on https://review.openstack.org/#/c/308930/7813:03
tbarronmkoderer__: looking13:03
mkoderer__tbarron: IMO we need to state that the driver is not for production usage and let it merge13:03
mkoderer__currently we are blocking features that are network releated since all wait for this driver13:04
*** houming has quit IRC13:04
tbarronmkoderer__: 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: +113:07
tbarronmkoderer__: 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 it13:07
aovchinnikovtbarron: ok, let's remove the gate part from known issues then13:09
tbarronmkoderer__: 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 :P13:12
tbarronmkoderer__: point taken :)13:12
aovchinnikovtbarron: somehow I don't like the idea of registering bugs for non-existent features13:13
tbarronaovchinnikov: yeah, i just meant for the concurrency issue13:13
tbarronaovchinnikov: it's not a "feature" that one can run concurrently w/o breakage13:13
aovchinnikovtbarron: 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
aovchinnikovthat is what I am concerned about13:14
aovchinnikovit seems a bit like "when we merge this feature we'll have this bug" to me13:15
aovchinnikovwhich probably means that we should not merge this feature in the first place, right?13:15
tbarronaovchinnikov: i see, well i was just trying to log technical debt as we commit.13:16
tbarronaovchinnikov: 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 it13:16
aovchinnikovon the other hand when the driver is in the tree then it is the right time to file it's bugs13:17
tbarronaovchinnikov: well I don't want to get stuck on process here.13:17
tbarronaovchinnikov: 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
tbarronaovchinnikov: 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 +W13:19
tbarrontrying work together here - back in 10-20m13:19
mkoderer__tbarron: thanks!13:21
aovchinnikovalso I don't think it is a good idea to label it 'dev-only' in the docs13:23
*** dustins has joined #openstack-manila13:23
mkoderer__aovchinnikov: would you say it's production ready?13:23
gouthamrI 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
aovchinnikovno, it is not, but suppose we label it 'dev-only' now and improve it some time later13:24
aovchinnikovhow are we going to undo the first impression?13:25
gouthamrrelease specific configuration reference, release notes can convey that as we go..13:25
aovchinnikovso I believe it boils down to choosing of accurate wording13:25
mkoderer__aovchinnikov: yes it's just about documentation and that's all13:26
*** porrua has joined #openstack-manila13: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
gouthamrmkoderer__: 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
gouthamrmkoderer__: just kidding.. nice! i actually dunno how to verify jumbo frames are supported. how would you do that?13:29
openstackgerritAlexey Ovchinnikov proposed openstack/manila: Container driver  https://review.openstack.org/30893013:30
aovchinnikovok, here it goes13:30
mkoderer__gouthamr: yeah I think there are way's with pings to verfify packages are not fragmeneted somewhere13:30
mkoderer__gouthamr: need to ask my network guru's13:31
openstackgerritAccela Zhao proposed openstack/python-manilaclient: Add driver private storage admin API  https://review.openstack.org/33658113:32
mkoderer__gouthamr: iperf would be also a solution13:32
gouthamrmkoderer__: thanks, will take a look..13:33
*** vbellur has quit IRC13:36
*** akerr has joined #openstack-manila13:37
mkoderer__gouthamr: ping -M do -s 9000 <NetApp box>13:45
*** vbellur has joined #openstack-manila13:46
*** vbellur has quit IRC13:51
openstackgerritAlexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests  https://review.openstack.org/34023613:51
*** chlong has joined #openstack-manila13:52
tbarronbswartz: https://review.openstack.org/#/c/308930/79 can get your +2 back and prob +W now13:59
*** aovchinnikov has quit IRC14:00
bswartztbarron: ty14:02
*** mtanino has quit IRC14:09
openstackgerritAndrew Kerr proposed openstack/manila: [WIP] Add tests for share groups  https://review.openstack.org/35526414:13
openstackgerritIury Gregory Melo Ferreira proposed openstack/puppet-manila: Add authtoken class  https://review.openstack.org/35179214:18
*** lpetrut has joined #openstack-manila14:24
openstackgerritAccela Zhao proposed openstack/manila: Driver private data admin API  https://review.openstack.org/31534614:27
*** cknight has joined #openstack-manila14:28
*** pcaruana has quit IRC14:30
*** rraja has quit IRC14:32
*** JoseMello has quit IRC14:40
*** vbellur has joined #openstack-manila14:40
mkoderer__xyang1: ping14:49
xyang1mkoderer__: hi14: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
xyang1mkoderer__: I think it is useful to have a way to validate those data14:55
*** ociuhandu has joined #openstack-manila14:55
mkoderer__xyang1: IMHO the openstack API should be seen as abstraction layer for different vendors/drivers14:56
xyang1mkoderer__: this is an abstraction, one api to retrieve the data14:56
mkoderer__xyang1: yeah that's the typical vendor trick to lock you in.. just definde the interface broad engough ;)14:57
xyang1mkoderer__: I don't get what you meant.  this is just a generic api to read data14:58
mkoderer__xyang1: 1.) the data is named "private" and is now availible trough api14:59
xyang1mkoderer__: it is admin api14:59
xyang1mkoderer__: admin should be allowed to read private data15:00
*** akerr is now known as akerr_away15:00
*** akerr_away is now known as akerr15:00
mkoderer__xyang1: but it's "private" driver data nodody should really care about15:00
mkoderer__xyang1: 2.) for me changing the api just because of 3rd CI testing isn't a good enough reason15:01
xyang1mkoderer__: I don't agree nobody should care about it.  if it is saved, it is useful15: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-wise15:03
xyang1mkoderer__: I don't know hiw neutron makes decisions, but this api is useful for manila15:05
mkoderer__xyang1: ok I just wanted to raise my doubts - I am fine if others do see it differently15:05
xyang1mkoderer__: ok, thanks for reviewing and raises your concerns15:05
*** senk_ has quit IRC15:06
*** lpetrut has quit IRC15:06
openstackgerritThomas Bechtold proposed openstack/manila: Fix sample config generation for cinder,nova and neutron opts  https://review.openstack.org/35551715:06
mkoderer__yeah docker driver is merging :)15:09
*** Yogi1 has joined #openstack-manila15:10
openstackgerritAlex O'Rourke proposed openstack/manila: 3PAR: Add update_access support  https://review.openstack.org/33979515:32
*** dsariel has joined #openstack-manila15:54
*** mtanino has joined #openstack-manila15:57
*** Suyi has joined #openstack-manila16:22
openstackgerritMerged openstack/manila: NetApp cDOT: Apply network MTU to VLAN ports  https://review.openstack.org/35525416:26
*** yangyapeng has quit IRC16:31
openstackgerritAlexey Ovchinnikov proposed openstack/manila: [DNM] Tests for scenario tests  https://review.openstack.org/34023616:43
openstackgerritTatiana Ovchinnikova proposed openstack/manila-ui: Additional fields for Create/Update Share Type forms  https://review.openstack.org/33850416:46
*** aovchinnikov has joined #openstack-manila16:50
*** harlowja has joined #openstack-manila17:08
*** senk_ has joined #openstack-manila17:45
*** JoseMello has joined #openstack-manila17:47
openstackgerritDai Dang Van proposed openstack/manila: Implement extend and shrink features for glusterfs  https://review.openstack.org/33138918:08
*** dsariel has quit IRC18:09
*** xyang1 has quit IRC18:22
*** xyang1 has joined #openstack-manila18:26
*** senk_ has quit IRC18:27
*** aovchinnikov has quit IRC18:30
*** dsariel has joined #openstack-manila18:30
bswartztbarron: do you know what phusion is?19:30
*** vbellur has quit IRC19:31
tbarronbswartz: no, guessing is it this? https://github.com/phusion/baseimage-docker19:33
bswartztbarron: yeah it19:33
bswartztbarron: yeah it's an off-the-shelf container image which has an init system with systemd built in, which we need for the container driver19:34
bswartzit happens to be ubuntu based19:34
tbarronyeah19:34
bswartzI'd love to know if a fedora-based container exists which has an init system like phusion19:34
bswartzor centos even19:35
tbarronyeah, I'll see what I can find out19:36
tbarronso phusion does both upstart and systemd type init?19:37
tbarronI ask b/c in the docker driver review I asked if it works with xenial and that hasn't been checked yet.19:37
bswartzwell I don't know that detail, and honestly it's not relevant19:37
bswartzwhat matters is that there's an init system19:37
tbarronNot a blocker, but when infra wants to convert to Xenial it will be relevant, no?19:37
bswartzit's quite common for containers to be single-app and to have no init system19:37
tbarronright19:38
bswartzbut we require an init system for the container behind our manila container driver19:38
bswartzI seriously doubt xenial will cause any issues19:38
tbarroni guess if the init system is inside the contaiiner it's insulaated from what infra does w.r.t. 14.04 vs xenial anyways19:38
bswartzyes19:38
tbarronok19:39
bswartzaside from the kernel version, very little matters outside the container19:39
tbarronright19:39
bswartzHonestly I'd be more concerned about changes in docker itself between trusty->xenial19:39
bswartzdocker is rapidly evolving and could have broken something we depend on19:39
tbarronthat makes sense19:45
tbarronso 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
tbarronIt 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
tbarronand we don't have any hard dependency issues introduced.19:48
*** senk_ has joined #openstack-manila19:49
*** senk_ has quit IRC19:50
*** porrua has quit IRC20:16
*** Yogi1 has quit IRC20:19
*** cargonza has joined #openstack-manila20:20
*** akerr has quit IRC20:23
*** xyang1 has quit IRC20:29
*** xyang1 has joined #openstack-manila20:29
*** ociuhandu has quit IRC20:31
*** yangyapeng has joined #openstack-manila20:33
*** gouthamr has quit IRC20:51
*** JoseMello has quit IRC21:00
*** lpetrut has joined #openstack-manila21:03
openstackgerritMerged openstack/manila: Container driver  https://review.openstack.org/30893021:04
openstackgerritMerged openstack/puppet-manila: Add authtoken class  https://review.openstack.org/35179221:05
*** dsariel has quit IRC21:06
*** lpetrut has quit IRC21:12
*** dustins has quit IRC21:15
*** lpetrut has joined #openstack-manila21:23
*** harlowja has quit IRC21:55
*** mtanino has quit IRC22:03
*** lpetrut has quit IRC22:11
*** cknight has quit IRC22:14
openstackgerritMerged openstack/manila: Tox Upper Constraints - strip out reinstalls for remaining jobs  https://review.openstack.org/35525522:14
*** bswartz has quit IRC22:15
*** xyang1 has quit IRC23:08
*** sticker has joined #openstack-manila23:08
*** harlowja has joined #openstack-manila23:36

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