Wednesday, 2016-03-30

sripriyasridhar_ram: bobh: s3wong please review https://review.openstack.org/29901600:00
s3wongsripriya: in addition to 289614? You are asking for a lot :-)00:01
s3wongsripriya: but ... sure :-)00:01
sripriyas3wong: i have one more critical bug fix coming, so add that up as well ;)00:02
s3wongsripriya: my night is gone :-)00:02
sripriya#release #zombies00:02
*** uck has quit IRC00:11
*** bobh has quit IRC00:27
openstackgerritSripriya Seetharam proposed openstack/tacker-horizon: Fix config and param data handling for Deploy VNF  https://review.openstack.org/29904200:30
sridhar_rams3wong: sripriya: is coding too much.. she is bombarding me as well w/ review requests... need to be handled in Newton ;-)00:38
sridhar_rams3wong: however here is a important one .. https://review.openstack.org/#/c/289614/00:39
sridhar_rams3wong: we have a +1 from lhcheng (horizon core)00:40
sridhar_rams3wong: this is ready to merge00:40
*** prashantD has quit IRC00:40
s3wongsridhar_ram: yes, that is the one I am looking at now00:40
sridhar_rams3wong: cool.. will wait for your +200:40
* s3wong at one point thought that even sridhar_ram is adding more review load :-) 00:41
* sridhar_ram is the now the designated middleman00:41
sripriyas3wong: sridhar_ram: we are still talking about some basic CRUD ops with nfvo/vim/vnfd/vnf :)00:42
s3wongsridhar_ram: part of PTL job description, I am sure :-)00:42
sripriyalet us not even dwell into mgmt/monitor/config/param etc etc00:42
* sripriya ducks00:42
sridhar_rams3wong: yep... the mouse clicker lead (MCL)00:42
sridhar_ramsripriya: so, looks you are going to taking it up a notch in newton with distributed, schedule-able tacker-agents connected thru gRPC00:44
sripriyasridhar_ram: i assume you want to have "more" devs and let the current devs survive :-)00:46
sridhar_ramsripriya: yeah, need to start the tacker roadtrips again to take tacker to various events00:47
sridhar_ramsripriya: do you mind adding a dependency for your multisite horizon for https://review.openstack.org/#/c/299042/ ?00:47
sripriyasridhar_ram: i will resolve the conflict on the other patch once multisite merges00:55
sridhar_rams3wong: I just +2'd multisite UI patchset - https://review.openstack.org/#/c/28961401:13
s3wongsridhar_ram: cool. I am on forms.py now01:13
openstackgerritMerged openstack/tacker-horizon: Implement Horizon support for multisite VIM  https://review.openstack.org/28961401:28
*** sripriya has quit IRC01:37
*** sridhar_ram has quit IRC01:40
*** tbh has joined #tacker01:54
*** sripriya has joined #tacker02:08
*** sripriya_ has joined #tacker02:10
*** sripriya has quit IRC02:13
*** sripriya__ has joined #tacker02:14
*** sripriya has joined #tacker02:15
*** ksantoshk has quit IRC02:16
*** santoshk has joined #tacker02:17
*** sripriya__ has quit IRC02:18
*** sripriya_ has quit IRC02:18
*** santoshk has quit IRC02:26
*** mannidi__ has joined #tacker02:28
openstackgerritSripriya Seetharam proposed openstack/tacker-horizon: Fix config and param data handling for Deploy VNF  https://review.openstack.org/29904202:29
*** santoshk has joined #tacker02:32
*** ksantoshk has joined #tacker02:36
*** santoshk has quit IRC02:39
*** prashantD has joined #tacker02:54
*** lhcheng has quit IRC03:04
*** s3wong has quit IRC03:05
vishwanathjtbh hi03:06
tbhvishwanathj, Hi03:06
vishwanathjtbh I put some minor comments to your image creation patchset, also your patchset is in merge conflict, I am sure you are planning to upload a new patchset, my comments are minor, you can ignore for now03:07
tbhvishwanathj, yes,fixing the conflict, if you are available for 30 more minutes, can you please review the new patchset?03:08
vishwanathjtbh sure, I will be available for review03:10
*** lhcheng has joined #tacker03:13
*** lhcheng_ has joined #tacker03:14
*** lhcheng has quit IRC03:14
tbhvishwanathj, thanks!03:14
*** lhcheng_ has quit IRC03:48
openstackgerritMerged openstack/tacker-horizon: Fix config and param data handling for Deploy VNF  https://review.openstack.org/29904203:52
*** lhcheng has joined #tacker03:55
openstackgerritbharaththiruveedula proposed openstack/tacker: Implements auto image creation  https://review.openstack.org/29203404:03
dkushwahasripriya, Hi, I have re-installed openstack, And now its working fine. Thanks :-)04:05
sripriya dkushwaha: cool, that error was really weird, glad you got it working now04:06
sripriyadkushwaha: regarding https://review.openstack.org/#/c/269424/10 did you happen to try the changes with an openwrt image?04:06
dkushwahasripriya, I have not tested it with openwrt image.04:13
*** vishnoianil has quit IRC04:13
sripriyadkushwaha: we upload openwrt image in glance with tacker devstack install04:14
sripriyadkushwaha: it will be good to test it out04:14
sripriyadkushwaha: i understadn there are some issues with current openwrt image coming up with an IP with dhcp04:15
dkushwahasripriya, Sorry, I am not much clear about your point. Did you mean to test cmd_executer in https://review.openstack.org/#/c/269424/10 to upload openwrt image in glance with tacker devstack install?04:25
sripriyadkuswaha: i implied openwrt as mgmt driver04:28
dkushwahasripriya, ok. I will test it.04:30
sripriyadkushwaha: thank you04:31
sripriyavishwanathj: ping04:31
openstackgerritdharmendra kushwaha proposed openstack/tacker: Remove premature vnf delete from db  https://review.openstack.org/29103204:34
vishwanathjsripriya hi05:00
*** vishwanathj is now known as vishwanathj_zzz05:03
*** ksantoshk has quit IRC05:12
*** santoshk has joined #tacker05:14
*** vishnoianil has joined #tacker05:15
openstackgerritSripriya Seetharam proposed openstack/tacker: Fix keystone v2 support for multisite  https://review.openstack.org/29912405:17
*** prashantD has quit IRC05:18
*** mannidi__ has quit IRC05:20
openstackgerritMerged openstack/tacker: Add default VIM steps for manual install  https://review.openstack.org/29901605:31
*** janki has joined #tacker05:40
*** sridhar_ram has joined #tacker05:45
*** sridhar_ram1 has joined #tacker05:55
*** sridhar_ram has quit IRC05:57
*** Ravikiran_K has joined #tacker05:59
*** lhcheng has quit IRC06:12
*** sridhar_ram1 is now known as sridhar_ram06:14
*** janki has quit IRC06:23
*** janki has joined #tacker06:33
*** santoshk has quit IRC06:57
*** zeih has joined #tacker06:58
sridhar_ramtbh: provided a comment to fix the sample for auto-image-creation06:58
sridhar_ramtbh: I'd suggest to use something like http://paste.openstack.org/show/492393/ with cirros image instead of openwrt image06:58
openstackgerritJanki Chhatbar proposed openstack/tacker: openwrt image download failure handled  https://review.openstack.org/29457307:00
*** sridhar_ram has quit IRC07:09
openstackgerritbharaththiruveedula proposed openstack/tacker: Implements auto image creation  https://review.openstack.org/29203407:41
*** mbound has joined #tacker07:59
*** Ravikiran_K has quit IRC08:28
*** janki has quit IRC08:38
*** janki has joined #tacker09:07
*** janki has quit IRC09:20
*** janki has joined #tacker10:19
*** zeih has quit IRC10:21
*** dhellmann has joined #tacker10:36
*** zeih has joined #tacker10:51
*** janki has quit IRC10:51
*** janki has joined #tacker10:52
*** janki has quit IRC11:00
*** tbh has quit IRC11:31
openstackgerritdharmendra kushwaha proposed openstack/tacker: Remove premature vnf delete from db  https://review.openstack.org/29103211:31
*** openstackgerrit has quit IRC11:47
*** openstackgerrit has joined #tacker11:48
*** janki has joined #tacker11:54
*** zeih has quit IRC12:12
*** janki has quit IRC12:20
*** openstackgerrit has quit IRC12:33
*** openstackgerrit has joined #tacker12:33
*** mbound has quit IRC12:34
*** mbound has joined #tacker12:36
*** zeih has joined #tacker12:39
*** mbound has quit IRC13:16
*** mbound has joined #tacker13:17
*** Shahid_ has joined #tacker13:18
*** openstackgerrit has quit IRC13:18
*** openstackgerrit has joined #tacker13:19
*** tbh has joined #tacker13:23
*** trozet has joined #tacker13:25
*** gongysh has quit IRC13:42
*** Shahid_ has quit IRC13:45
*** bobh_ has joined #tacker13:45
*** zeih has quit IRC13:48
*** gongysh has joined #tacker13:54
*** dkushwaha_ has joined #tacker14:57
*** Shahid_ has joined #tacker15:13
*** vishwanathj_zzz is now known as vishwanathj15:16
*** dkushwaha_ has quit IRC15:16
*** zeih has joined #tacker15:26
zeihsripriya: ping15:27
sripriyazeih: pong15:28
sripriyatbh: ping15:28
zeihsripriya: do you have 10min for a question regarding https://bugs.launchpad.net/heat/+bug/1515768/?15:28
openstackLaunchpad bug 1515768 in nova (Ubuntu) "Instance creation fails with libvirtError: Unable to create tap device: Device or resource busy " [Undecided,New]15:28
sripriyazeih: yes15:29
zeihsripriya: We hit this bug in our lab. (liberty deployed) Do you know in which released paket the backport is included?15:30
sripriyazeih: are you using ubuntu pkg installer for liberty?15:31
zeihye15:31
zeihs15:31
* sripriya digs up15:31
zeihhow can I find out this kind of information?15:31
sripriyazeih: you need to check the releases ubuntu has made for nova15:32
sripriyazeih: they have frequent releases based on liberty stable versions15:32
sripriyazeih: i believe this fix went into stable release 315:33
zeihsripriya: where can I find the ubuntu releases? launchpad?15:36
sripriyazeih: yes15:36
sripriyahttps://launchpad.net/ubuntu/+source/nova15:36
*** sridhar_ram has joined #tacker15:37
*** dkushwaha has quit IRC15:39
sripriyazeih: looks like they did not make a release for nova 12.0.2 , last was on 12.0.115:39
zeihsripriya: Is there somewhere a correlation between openstack release version and ubuntu version?15:39
zeihsripriya: will have a look15:39
zeihsripriya: yes15:40
sripriyazeih: the 12.0.x should help map between openstack and ubuntu release15:41
zeihsripriya: I can see on github, that the release was 23days ago15:44
zeih12.0.215:44
sripriyazeih: yes, dhellman has commented on the bug as well that it went into 12.0.215:44
zeihsripriya: :(  I am afraid I can not find the comment in https://bugs.launchpad.net/heat/+bug/1515768/15:47
openstackLaunchpad bug 1515768 in nova (Ubuntu) "Instance creation fails with libvirtError: Unable to create tap device: Device or resource busy " [Undecided,New]15:47
*** prashantD has joined #tacker15:48
vishwanathjsripriya sridhar_ram when a VNFD template is onboarded, I see that there is no name associated with the template, is this a known bug and is somebody working on it? Tahnks15:50
sripriyazeih: it is in the nova bug: https://bugs.launchpad.net/nova/+bug/152211215:50
openstackLaunchpad bug 1522112 in OpenStack Compute (nova) liberty "ports duplication in the VM XML when using heat and multiple networks" [Medium,Fix committed] - Assigned to David Edery (david-edery)15:50
sripriyavishwanathj: is this through Horizon?15:51
vishwanathjyes15:51
sripriyavishwanathj: yes it is a known bug, low priority though, i had a fix for that15:52
sripriyavishwanathj: https://review.openstack.org/#/c/219048/2/tacker_horizon/openstack_dashboard/dashboards/nfv/vnfcatalog/forms.py15:52
sripriyavishwanathj: will revisit that sometime15:53
*** uck has joined #tacker15:53
zeihsripriya: oh.. I see. thank you15:53
tbhsripriya, pong15:56
tbhvishwanathj, sripriya I face the same problem15:57
tbhvishwanathj, sripriya not sure why we are not taking "name" as VNFD name through horizon15:58
tbhvishwanathj, sripriya but it works for non-tosca templates15:58
vishwanathjtbh that's the enhancement sripriya has an initial patchset for, initial behavior had the name being grabbed from the template file, looks like that behavior is now going to change with the new patchset16:00
sripriyavishwanathj: tbh: if you do not provide a name through Horizon for both kind of templates. it should read it from the template file, does that work?16:01
tbhvishwanathj, sripriya yes but, the name is not "required parameter " https://github.com/openstack/tacker-horizon/blob/master/tacker_horizon/openstack_dashboard/dashboards/nfv/vnfcatalog/forms.py#L2616:02
tbhsripriya, no, it doesn't work for tosca templates16:02
vishwanathjsripriya, when I upload a tosca template, the template name is not being picked up16:02
*** zeih has quit IRC16:03
tbhsripriya, I think it doesn't look for name in meta["template_name"]16:04
sripriyatbh: it does https://github.com/openstack/tacker/blob/master/tacker/vm/infra_drivers/heat/heat.py#L10516:04
tbhsripriya, yes, but that it doesn't pass the "if" condition16:06
tbhsripriya, because it couldn't "name" in device_template_dict16:06
*** zeih has joined #tacker16:06
tbhsripriya, now with your patch, it can find the name16:07
vishwanathjtbh sripriya is the expectation that the name from the template would be picked up if a name is not explicitly provided16:09
sripriyatbh: shouldnt the condition succeed since we do not add add 'name' from Horizon on L104?16:09
sripriyavishwanathj: thats correct16:09
tbhsripriya, ah correct, but the name is ''16:11
vishwanathjsripriya tbh the "'name' not in device_template_dict" always turns out to be False as name is present with a value of ''16:12
sripriyatbh: default is empty string here from API layer16:12
sripriyatbh: we will have to handle that16:13
sripriyavishwanathj: tbh: we will have to add that condition for tosca templates16:14
sripriyaan also fix Horizon16:14
tbhsripriya, so will this work "if  device_template_dict.get('name', '') and 'metadata' in vnfd_dict:"16:14
tbhon L104?16:15
sridhar_rambobh_: sripriya: tacker-cores: please review / merge https://review.openstack.org/292034 ASAP16:15
sridhar_rambobh_: sripriya: we need to close out mitaka release16:15
sripriyasridhar_ram: i looked into that code, it looks good to me16:15
vishwanathjtbh your suggestion would work16:16
vishwanathjsince the name key would always be present16:16
sridhar_ramsripriya: thanks, can you go ahead and merge it please16:17
sripriyasridhar_ram: the only clarification i had for a user passing the file and not the type would result empty for image_dict, is that ok?16:17
tbhsripriya, but, we cannot predict the artifact, with out "type"16:18
vishwanathjtbh let me try your suggestion on my dev system and let you know if that works16:18
tbhsripriya, so I want to check for type first16:18
tbhvishwanathj, "if not", I guess, sure, let me know16:18
sridhar_ramsripriya: we can fine tune the type checkings in newton... if you & tbh agree please open bugs for those things16:20
*** openstack has joined #tacker17:07
*** vishnoianil has joined #tacker17:11
*** prashantD_ has joined #tacker17:17
*** prashantD_ has quit IRC17:19
*** prashantD has quit IRC17:20
*** prashantD_ has joined #tacker17:20
sridhar_ramsripriya: yes17:29
sridhar_ramsripriya: dkushwaha: please use the openwrt image from https://drive.google.com/open?id=0B-ruQ8Tx46wSMktKV3JLRWhnLTA where dhcp is enabled17:29
*** Ravikiran_K has joined #tacker17:45
*** s3wong has joined #tacker18:00
*** sridhar_ram has quit IRC18:25
*** zeih has joined #tacker18:28
*** sripriya has quit IRC18:43
*** dkushwaha has quit IRC18:51
*** uck has quit IRC19:02
*** uck has joined #tacker19:03
*** sripriya has joined #tacker19:07
*** Ravikiran_K has quit IRC19:12
*** mbound has joined #tacker19:48
*** zeih_ has joined #tacker19:54
*** zeih__ has joined #tacker19:59
*** zeih_ has quit IRC20:00
*** sridhar_ram has joined #tacker20:23
*** zeih__ has quit IRC20:27
sridhar_ramTackers - if you are online please try out the latest tacker code base for any blockers20:44
*** openstackgerrit has quit IRC20:48
*** zeih_ has joined #tacker20:48
*** openstackgerrit has joined #tacker20:48
sridhar_rambobh_: I tried to create a vnfd using samples/sample-tosca-vnfd-monitor.yaml and it fails w/ "tosca-parser failed: - 'NoneType' object has no attribute '__getitem__'"21:03
sridhar_rambobh_: have you seen this ?21:03
sridhar_rambobh_: here is the paste - http://paste.openstack.org/show/492500/21:12
*** zeih_ has quit IRC21:16
*** zeih has quit IRC21:59
bobh_sridhar_ram: I have seen that - useful error, isn't it?21:59
bobh_sridhar_ram: I'll take a look and see if I can see the issue21:59
bobh_sridhar_ram: I might have modified the test samples and forgot to update the devstack samples21:59
sridhar_rambobh_: I fixed up template and vnfd-create proceeds fine now...22:00
bobh_sridhar_ram: cool22:00
sridhar_rambobh_: http://paste.openstack.org/show/492502/22:00
sridhar_rambobh_: but monitoring code throws an exception22:01
bobh_sridhar_ram: like I said - forgot to update the devstack samples22:01
sridhar_rambobh_: no worries..22:01
sridhar_rambobh_: can you point check whether ping monitoring kicks in w/ new tosca-parser based sample template ?22:03
sridhar_rambobh_: with old parser ping monitor is hanging :(22:05
sridhar_rambobh_: http://paste.openstack.org/show/492507/22:05
bobh_sridhar_ram: heading home now - I can check tonight22:08
sridhar_rambobh_: okay..22:09
sridhar_rambobh_: using old template pings are flowing fine.. the earlier hang due to pycharm22:09
*** brucet has joined #tacker22:12
*** bobh_ has quit IRC22:14
*** brucet has quit IRC22:18
*** vishwanathj has quit IRC22:27
*** mbound has quit IRC22:28
openstackgerritSridhar Ramaswamy proposed openstack/tacker: Fix monitoring policy in new sample tosca templates  https://review.openstack.org/29965723:05
*** sridhar_ram has quit IRC23:08
*** bobh_ has joined #tacker23:11
*** bobh_ has quit IRC23:12
*** sridhar_ram has joined #tacker23:14
*** sridhar_ram has quit IRC23:14
*** bobh_ has joined #tacker23:15
*** bobh_ has quit IRC23:17
*** sridhar_ram has joined #tacker23:27
*** bobh_ has joined #tacker23:33
sridhar_ramsripriya: thanks for the review on https://review.openstack.org/29965723:40
sridhar_rambobh_: s3wong: please review ^^ .. fixes to samples related monitoring23:40
openstackgerritSripriya Seetharam proposed openstack/tacker: Revert multiple mgmtdriver check for old templates  https://review.openstack.org/29967623:42
s3wongsridhar_ram: done23:54

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