Thursday, 2016-03-24

*** chlong has quit IRC00:11
*** mtanino has quit IRC00:24
*** qeelee has joined #openstack-manila00:48
*** akshai_ has quit IRC00:56
*** qeelee has quit IRC01:00
*** qeelee has joined #openstack-manila01:03
openstackgerritdaiki kato proposed openstack/manila: Fix command execution results in quick start guide  https://review.openstack.org/29685501:08
openstackgerritdaiki kato proposed openstack/manila: Fix command execution results in quick start guide  https://review.openstack.org/29685501:16
*** cknight has joined #openstack-manila01:22
*** gouthamr_ has joined #openstack-manila01:30
*** gouthamr has quit IRC01:34
*** mtanino has joined #openstack-manila01:34
*** chlong has joined #openstack-manila01:36
*** akshai has joined #openstack-manila01:41
*** baojg has joined #openstack-manila01:46
*** houming has joined #openstack-manila01:49
*** zhonghua-lee has quit IRC01:55
*** zhonghua-lee has joined #openstack-manila01:56
*** akshai_ has joined #openstack-manila01:57
*** akshai has quit IRC02:00
*** yangyapeng has joined #openstack-manila02:04
*** akshai_ has quit IRC02:04
*** akshai has joined #openstack-manila02:07
*** cknight has quit IRC02:12
*** yangyapeng has quit IRC02:25
*** jasonsb has joined #openstack-manila03:03
*** cknight has joined #openstack-manila03:22
*** gouthamr_ has quit IRC03:35
*** cknight1 has joined #openstack-manila03:36
*** cknight has quit IRC03:38
*** akshai_ has joined #openstack-manila03:40
*** akshai has quit IRC03:42
*** qeelee has quit IRC03:42
*** bswartz has quit IRC03:55
*** bswartz has joined #openstack-manila03:56
*** absubram has joined #openstack-manila04:07
*** absubram_ has joined #openstack-manila04:08
*** akshai has joined #openstack-manila04:11
*** absubram has quit IRC04:12
*** absubram_ is now known as absubram04:12
*** akshai_ has quit IRC04:14
*** swamireddy1 has joined #openstack-manila04:27
*** swamireddy2 has joined #openstack-manila04:28
*** swamireddy1 has quit IRC04:31
*** qeelee has joined #openstack-manila04:34
*** absubram has quit IRC04:36
*** cknight1 has quit IRC04:36
*** cknight has joined #openstack-manila04:36
*** cknight has quit IRC04:41
*** zhongjun has quit IRC04:49
*** zhongjun has joined #openstack-manila04:50
*** qeelee has quit IRC04:51
*** chlong has quit IRC04:56
*** chlong has joined #openstack-manila05:12
*** baojg has quit IRC05:19
*** akshai has quit IRC05:22
*** qeelee has joined #openstack-manila05:24
*** mtanino has quit IRC05:35
*** pcaruana has quit IRC05:48
*** sgotliv has joined #openstack-manila05:57
*** chlong has quit IRC06:08
*** chlong has joined #openstack-manila06:22
*** baojg has joined #openstack-manila06:27
*** sgotliv has quit IRC06:40
*** baojg has quit IRC06:42
*** esker has quit IRC06:46
*** baojg has joined #openstack-manila06:47
*** houming_ has joined #openstack-manila06:48
*** houming has quit IRC06:48
*** houming_ is now known as houming06:48
*** qeelee has quit IRC07:00
*** lpetrut has joined #openstack-manila07:02
*** qeelee has joined #openstack-manila07:04
*** baojg has quit IRC07:06
*** baojg has joined #openstack-manila07:08
*** houming has quit IRC07:10
*** houming has joined #openstack-manila07:11
*** rraja has joined #openstack-manila07:12
*** pcaruana has joined #openstack-manila07:32
qeeleeHi all, I have a question about _get_private_router08:12
qeeleefor p in private_network_ports:08:12
qeelee    fixed_ip = p['fixed_ips'][0]08:12
qeelee    if (fixed_ip['subnet_id'] == private_subnet['id'] and08:12
qeelee            fixed_ip['ip_address'] == private_subnet['gateway_ip']):08:12
qeelee        private_subnet_gateway_port = p08:12
qeelee        break08:12
qeeleeIs it possible? fixed_ip['ip_address'] == private_subnet['gateway_ip’])08:13
*** yangyapeng has joined #openstack-manila08:16
*** lpetrut has quit IRC08:23
*** qeelee has quit IRC08:38
*** chlong has quit IRC08:38
*** lpetrut has joined #openstack-manila08:39
*** qeelee has joined #openstack-manila08:41
*** lpetrut has quit IRC08:49
mkoderer__qeelee: mh from the first look it seems a bit strange09:08
mkoderer__qeelee: have you tried to debug it?09:09
qeeleeI just set up an OS env with devstack master, and failed to create a share with the generic driver. I’m looking into it.09:10
*** dalgaaf has joined #openstack-manila09:12
*** jasonsb has quit IRC09:28
*** houming has quit IRC09:34
*** tbh has joined #openstack-manila09:44
*** rraja has quit IRC09:46
*** qeelee has quit IRC09:53
*** baojg has quit IRC09:55
*** jasonsb has joined #openstack-manila10:06
*** tpsilva has joined #openstack-manila10:07
*** yangyapeng has quit IRC10:30
*** erlon has joined #openstack-manila10:45
*** ociuhandu has quit IRC10:57
*** ociuhandu has joined #openstack-manila11:18
*** tbh has quit IRC11:38
*** yangyapeng has joined #openstack-manila11:45
*** sgotliv has joined #openstack-manila11:56
*** timcl has joined #openstack-manila11:59
*** mtanino has joined #openstack-manila12:25
*** akerr has joined #openstack-manila12:43
*** ganso has joined #openstack-manila12:49
*** akshai has joined #openstack-manila12:55
*** lpetrut has joined #openstack-manila12:56
*** rraja has joined #openstack-manila12:57
*** rraja has quit IRC12:58
*** eharney has quit IRC13:06
*** Yogi1 has joined #openstack-manila13:25
*** porrua has joined #openstack-manila13:25
*** eharney has joined #openstack-manila13:27
*** swamireddy2 has quit IRC13:30
*** timcl has quit IRC13:32
*** absubram has joined #openstack-manila13:33
*** absubram_ has joined #openstack-manila13:34
*** absubram has quit IRC13:38
*** absubram_ is now known as absubram13:38
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Manage API creating DB record for invalid type  https://review.openstack.org/29670613:43
*** martyturner has joined #openstack-manila13:45
*** gouthamr has joined #openstack-manila13:49
*** timcl has joined #openstack-manila13:54
*** cknight has joined #openstack-manila13:55
bswartzaovchinnikov: ping13:58
aovchinnikovbswartz: hi14:00
bswartzaovchinnikov: we need to wrap this up: https://review.openstack.org/#/c/293385/14:00
bswartzaovchinnikov: where does that stand?14:00
aovchinnikovthat one should be fine14:01
aovchinnikovit fixes the problem locally14:01
*** lpetrut has quit IRC14:02
aovchinnikovshould do the trick in the gate as well14:02
*** a_ta has joined #openstack-manila14:02
*** mtanino has quit IRC14:03
bswartzaovchinnikov: you -W it14:07
bswartzif you say that's the correct fix I'll review it14:08
aovchinnikovI've  already +W'd it14:08
aovchinnikovits a trivial fix actually14:08
bswartzoh wait I reviewed this last week14:08
bswartzthis results in a 800MB file instead of a 300MB file IIRC14:09
bswartzlet me retest...14:09
aovchinnikovhmm, ok, let me check this out14:09
bswartzaovchinnikov: did you find the place in teh code where it copies/renames the file to include the timestamp?14:10
bswartzI looked briefly and it wasn't obvious14:10
bswartzyeah 825MB14:13
aovchinnikovthe thing is there was no such place, the last time we've discussed the issue we've decided that it was some other artifact, or am I missing something?14:13
aovchinnikovthere still is a problem with the resulting image14:13
bswartzthats significantly larger than the 299MB of the old image14:13
bswartzokay you're probably right about the old artifact14:14
bswartzso in this case the problem is the image is insanely large14:14
aovchinnikovit looks like that disk image creator does not really care about the format parameter and takes whatever we supply14:14
*** lpetrut has joined #openstack-manila14:15
aovchinnikovand Jenkins expects qcow214:15
bswartzthe format is "qcow"14:15
bswartzI think compression is getting turned off14:15
bswartzwe need to tell it to build a qcow2 file with compression14:16
bswartzI'm trying to compress this file to see how small it gets14:16
aovchinnikovmaybe, I am looking at it now14:16
bswartzokay when I convert from qcow to qcow2 it ends up about 300MB14:17
bswartzso there's nothing wrong with the image -- just the format14:17
bswartzwith your fix, it might just work14:17
bswartzlet me test your fix14:17
bswartzyeah your fix does the right thing14:20
bswartzvponomaryov: +2A pls? https://review.openstack.org/#/c/29338514:21
vponomaryovbswartz: don't see CI results14:21
*** dustins has joined #openstack-manila14:21
* bswartz kicks jenkins14:22
bswartzokay jenkins jobs queued14:23
bswartzvponomaryov: jenkins +114:34
bswartzvponomaryov: experimental job was removed IIRC14:35
vponomaryovbswartz: why?14:35
bswartzonly used for LXD and LXD currently not supported14:35
vponomaryovbswartz: manila-image-elements is able to build it14:35
vponomaryovbswartz: does not matter that we dropped driver14:36
*** esker has joined #openstack-manila14:36
bswartzaovchinnikov took it out when he removed the other LXD related jobs14:36
bswartzwe can put it back in14:36
*** esker has quit IRC14:36
vponomaryovbswartz: it would be correct thing todo14:36
bswartzbut I think we should instead focus on future of container-based-driver14:36
vponomaryovbswartz: else we will just get it broken in manila-image-elements14:37
*** esker has joined #openstack-manila14:37
bswartzin the short term, yes14:37
bswartzhowever if we, for example, switch to docker, then we need to update manila-image-elements to build docker image instead of lxd image14:37
bswartzI would expect that to happen soon after release of mitaka14:38
*** absubram has quit IRC14:45
*** vbellur has quit IRC14:47
*** markstur_ has joined #openstack-manila14:51
openstackgerritMerged openstack/manila-image-elements: Fix for image name  https://review.openstack.org/29338514:59
mkoderermeeting?15:02
*** dencaval has quit IRC15:02
*** swamireddy1 has joined #openstack-manila15:05
*** netapp_ has joined #openstack-manila15:08
*** vbellur has joined #openstack-manila15:11
*** markstur__ has joined #openstack-manila15:11
*** jseiler_ has joined #openstack-manila15:12
*** markstur_ has quit IRC15:13
*** david-lyle has quit IRC15:13
*** jcsp has quit IRC15:13
*** jseiler has quit IRC15:13
*** netapp has quit IRC15:13
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Fix force-delete on snapshot resource  https://review.openstack.org/29720515:13
*** netapp_ is now known as netapp15:13
*** yangyapeng has quit IRC15:16
*** swamireddy1 has quit IRC15:17
*** yangyapeng has joined #openstack-manila15:17
*** david-lyle has joined #openstack-manila15:17
*** eharney has quit IRC15:25
*** yangyape_ has joined #openstack-manila15:28
*** jcsp has joined #openstack-manila15:29
bswartzesker: #openstack-meeting-alt15:31
bswartzesker: ppl are asking which logo won the poll15:31
*** yangyapeng has quit IRC15:31
openstackgerritDustin Schoenbrun proposed openstack/manila: Add common capabilities matrix to devref  https://review.openstack.org/28850415:32
*** dalgaaf has quit IRC15:32
*** yangyapeng has joined #openstack-manila15:32
gansogouthamr: ping15:33
*** dalgaaf has joined #openstack-manila15:33
gouthamrganso: pong15:34
*** yangyape_ has quit IRC15:34
dustinsgouthamr: I'm about to head to lunch, but I'll ping you when I get back about the devref/adminref/etc stuff15:34
gouthamrdustins: sure thing15:34
mkoderer__esker: if we could also create some shirts would be nice. I can try to get some budget if needed..15:34
gansogouthamr: Hi Goutham, about your comments, when I use exception.NoValidHost("fake") it overrides "message = ..." defined in NoValidHost... thus it is not necessary to use reason="fake"15:35
gouthamrganso: yes.. but one of the other tests used it the other way.. just a nit if you'd like to make it :)15:36
gansogouthamr: oh ok15:36
gansogouthamr: also, I did not understand this comment of yours https://review.openstack.org/#/c/296706/4/manila/share/api.py15:36
gansogouthamr: I am capturing NoValidHost and raising InvalidHost, which will be captured in API and raise HTTPConflict15:37
gansogouthamr: (I am going to fix the one in API to include InvalidHost, good catch about that one)15:37
gouthamrganso: I didn't see NoValidHost here: https://review.openstack.org/#/c/296706/4/manila/api/v1/share_manage.py15:37
gouthamrganso: yep15:37
*** markstur__ has quit IRC15:37
gansogouthamr: no no, I am not reraising it15:37
gansogouthamr: I am raising a different exception15:38
gansogouthamr: NoValidHost does not need to be in API15:38
gouthamrganso: sorry I meant InvalidHost15:38
gansogouthamr: yes, that's the one I am fixing15:38
*** absubram has joined #openstack-manila15:39
gouthamrganso: thanks.. made a couple of dependent changes, will rebase on your latest when ur done!15:39
gansogouthamr: ok! thanks for your review!15:39
*** timcl has quit IRC15:39
gouthamrganso: :) yw15:39
*** eharney has joined #openstack-manila15:42
*** lpetrut has quit IRC15:44
openstackgerritDustin Schoenbrun proposed openstack/manila: Add common capabilities matrix to devref  https://review.openstack.org/28850415:46
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Manage API creating DB record for invalid type  https://review.openstack.org/29670615:58
*** vbellur has quit IRC16:02
*** lpetrut has joined #openstack-manila16:08
*** vbellur has joined #openstack-manila16:20
*** jasonsb has quit IRC16:42
*** lpetrut has quit IRC16:43
*** vbellur has quit IRC16:48
*** esker has quit IRC16:56
*** lpetrut has joined #openstack-manila16:56
*** openstackgerrit has quit IRC17:01
*** timcl has joined #openstack-manila17:01
*** openstackgerrit has joined #openstack-manila17:02
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Fix force-delete on snapshot resource  https://review.openstack.org/29720517:13
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Manage API creating DB record for invalid type  https://review.openstack.org/29670617:14
gansogouthamr: sorry ^17:14
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Fix force-delete on snapshot resource  https://review.openstack.org/29720517:14
*** lpetrut has quit IRC17:14
gouthamrganso: np. its just a button click :)17:14
*** lpetrut has joined #openstack-manila17:14
gansogouthamr: =)17:15
vponomaryovganso; ping17:24
gansovponomaryov: pong17:24
vponomaryovganso: why in https://review.openstack.org/#/c/296706/6/manila/share/api.py you do not provide "host" and "share_proto" to scheduler?17:25
gansovponomaryov: I do provide host o_O17:25
vponomaryovganso; line 50617:26
vponomaryovganso: so, share_instance_properties does not include host17:27
*** lpetrut has quit IRC17:27
gansovponomaryov: in my testing it did not need to include host. In scheduler, you send host as parameter as the one you want to run filters against17:28
vponomaryovganso: let's assume, share_proto?17:28
*** lpetrut has joined #openstack-manila17:29
vponomaryovganso: add share_proto as extra spec, request unsupported proto in manage and see results17:29
gansovponomaryov: same reason. I assumed share_proto was validated somewhere else since it was already a mandatory parameter. If it was not, it would be possible to manage a CIFS share in a NFS backend before the fix merged17:29
gansovponomaryov: there was no scheduler validation before this fix. Was it possible to manage a CIFS share in NFS backend before?17:30
vponomaryovganso: current master contains "share_proto" in shares_properties17:30
gansovponomaryov: I don't know if and where it is validated, but I assumed it was17:30
vponomaryovganso: capability filter will use it17:31
vponomaryovganso: so, I would say that need to put "share_data" values to filter properties17:32
gansovponomaryov: I agree, but what is current behavior if not added? is it possible to break it?17:32
vponomaryovganso: right now you just partially test share type17:33
vponomaryovganso: but it was the main point of change17:33
vponomaryovganso: to validate share type17:34
vponomaryovganso: right?17:34
gansovponomaryov: right, I just tested here in my lab, I could create a CIFS share in my backend which does not support CIFS17:34
vponomaryovganso: ))17:34
*** lpetrut has quit IRC17:35
*** zhonghua-lee has quit IRC17:40
*** pcaruana has quit IRC17:40
*** zhonghua-lee has joined #openstack-manila17:41
dustinsgouthamr: ping17:56
gouthamrhi dustins17:57
*** swamireddy1 has joined #openstack-manila17:58
gansovponomaryov: thanks for pointing that out18:04
*** esker has joined #openstack-manila18:04
openstackgerritMerged openstack/manila: Add common capabilities matrix to devref  https://review.openstack.org/28850418:04
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Manage API creating DB record for invalid type  https://review.openstack.org/29670618:05
*** resker has joined #openstack-manila18:08
*** esker has quit IRC18:11
*** vbellur has joined #openstack-manila18:11
*** lpetrut has joined #openstack-manila18:14
*** swamireddy1 has quit IRC18:14
*** lpetrut has quit IRC18:18
*** mtanino has joined #openstack-manila18:48
*** martyturner has quit IRC19:02
*** zhonghua-lee has quit IRC19:34
*** sage has quit IRC19:35
*** sage has joined #openstack-manila19:38
*** dalgaaf has quit IRC19:38
*** ociuhandu has quit IRC19:41
*** zhonghua-lee has joined #openstack-manila19:41
openstackgerritRodrigo Barbieri proposed openstack/manila: Fix Manage API creating DB record for invalid type  https://review.openstack.org/29670619:43
*** lpetrut has joined #openstack-manila19:46
*** ociuhandu has joined #openstack-manila19:50
openstackgerritGoutham Pacha Ravi proposed openstack/manila: Fix force-delete on snapshot resource  https://review.openstack.org/29720519:52
*** a_ta has quit IRC19:55
*** yangyapeng has quit IRC20:11
*** yangyapeng has joined #openstack-manila20:12
*** akerr has quit IRC20:31
*** a_ta has joined #openstack-manila20:32
*** lpetrut has quit IRC20:33
*** porrua has quit IRC20:34
*** porrua has joined #openstack-manila20:35
gansobswartz: ping20:37
*** porrua has quit IRC20:39
bswartzganso: pong20:40
gansobswartz: Could you please take a look? https://review.openstack.org/#/c/293389/20:40
gansobswartz: it is backport patch20:40
bswartzganso: yeah20:41
bswartzganso: it needs another +1 from a core (if we pretend that core +1s are really +2, which is how I treat these backports)20:42
gansobswartz: can it be my +1? xD20:43
bswartzsure20:44
bswartzwe've talked about this before20:44
gansobswartz: +1'ed20:44
bswartzthere's no rule against +2 stuff from your coworkers as long as you're applying the same standards you would to everyone else20:44
gansobswartz: =)20:45
bswartzthe only rule we enforce is that we don't workflow until we get 2 +2s from cores who aren't from the same company20:45
*** cknight has quit IRC20:47
*** openstackgerrit has quit IRC20:48
bswartzganso: 1 nit -- the bug should have been tagged with "liberty-backport-potential" before the backport was done20:48
bswartzbut it doesn't really matter20:48
gansobswartz: hmmmmm I will remind tpsilva to do this next time, thanks for pointing it out20:49
*** openstackgerrit has joined #openstack-manila20:49
tpsilvaoh, didn't know that... will do this from now on bswartz :)20:52
tpsilvathank you20:52
gansobswartz: thanks20:55
*** eharney has quit IRC21:14
*** a_ta has quit IRC21:16
*** a_ta has joined #openstack-manila21:21
*** zhongjun2 has joined #openstack-manila21:36
*** zhongjun has quit IRC21:38
*** openstack has joined #openstack-manila21:50
*** jcsp has joined #openstack-manila21:56
*** dustins has quit IRC21:57
*** DuncanT has quit IRC22:03
*** DuncanT has joined #openstack-manila22:05
*** sgotliv has quit IRC22:16
*** erlon has quit IRC22:27
*** a_ta has quit IRC22:31
*** chlong has joined #openstack-manila22:55
*** akshai has quit IRC23:00
openstackgerritEmilien Macchi proposed openstack/puppet-manila: Import release notes from stable/mitaka  https://review.openstack.org/29743323:03
*** gouthamr has joined #openstack-manila23:14
*** a_ta has joined #openstack-manila23:14
*** tpsilva has quit IRC23:17
*** a_ta has quit IRC23:35
*** a_ta has joined #openstack-manila23:36
*** cknight has joined #openstack-manila23:39
*** ganso has quit IRC23:40
*** a_ta has quit IRC23:40
*** a_ta has joined #openstack-manila23:59

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