Isaku Yamahata [Fri, 4 Mar 2016 06:34:26 +0000 (22:34 -0800)]
yang model revise: neutron-subnets.yang
This patch revises neutron-subnet yang model and related codes.
- updates type. string -> inet:ip-address
- removes deprecated SPI methods
- removes backpointers in subnets. ports in subnet
Change-Id: Ie85dcd526db72b308c24b345fd6c1036e04b2d82
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Isaku Yamahata [Mon, 9 May 2016 21:19:30 +0000 (14:19 -0700)]
yang model revise: neutron-network.yang
This patch revises neutron network model.
This patch drops deprecated methods in neutron spi.
Change-Id: Ib8a6ba9928f504dc4567ec2e08a8aac4387f38dd
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Wed, 18 May 2016 06:28:42 +0000 (23:28 -0700)]
yang model revise: adds util methods and common types
This patch is a preparation for yang model clean up to introduce
to utility methods and common type, ip-prefix-or-address.
Change-Id: Idb566523a2f66899c558fb12c7a8e76d7d330be0
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Abbas Pareedkunju [Fri, 3 Jun 2016 06:31:52 +0000 (12:01 +0530)]
Bug 3902 : Local variables to be declared final.
This patch is to declare the local variables to final in transcriber module.
The bug 3902 reported for 4 local variables which are not declared final.
Running eclipse source Clean Up utility against the rule for Variable Declaration
listed the chunk of variables can be declared final.
Change-Id: Ied71645f2434d9827be370dadf1b4400d773b595
Signed-off-by: Abbas Pareedkunju <abbas.pareedkunju@tcs.com>
Abbas Pareedkunju [Tue, 31 May 2016 08:35:59 +0000 (14:05 +0530)]
Fix for Bug 3902 : Removal of the Unused imports from neutron.
This patch is specific to the removal of the Unused imports.
Change-Id: I336751e75103f0ed28f9f0e36014a43c1ba66db8
Signed-off-by: Abbas Pareedkunju <abbas.pareedkunju@tcs.com>
Anil Vishnoi [Thu, 12 May 2016 02:34:54 +0000 (19:34 -0700)]
Yang models for OpenStack Neutron SFC APIs.
Patch has the yang models for OpenStack Neutron Service Function
Chaining API. These models are based on the OpenStack SFC flow
classifier and port grouping APIs. Following URL contains the list
of OpenStack SFC APIs exposed by networking-sfc project:
References:
http://docs.openstack.org/developer/networking-sfc/api.html
https://github.com/openstack/networking-sfc/blob/master/networking_sfc/extensions/sfc.py
https://github.com/openstack/networking-sfc/blob/master/networking_sfc/extensions/flowclassifier.py
Change-Id: I5c1a20d18b34df5850af7ad13529da4a3f1ba767
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Isaku Yamahata [Thu, 5 May 2016 00:52:24 +0000 (17:52 -0700)]
Bug 5842 - enable neutron northbound Maven site
This patch enables neutron northbound maven site to catch up the trends.
https://nexus.opendaylight.org/content/sites/site/org.opendaylight.odlparent/boron/maven-site.html
https://meetings.opendaylight.org/opendaylight-meeting/2016/tws/opendaylight-meeting-tws.2016-04-18-17.04.html
Change-Id: Ibc15272641e7eec368726e4d5ac57b9ec3315b3b
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Tue, 3 May 2016 22:06:06 +0000 (15:06 -0700)]
Bug 5831 - build: check style error
This patch catches up for changes of odlparent checkstyle.
The checkset of
3b8a0b31dbe283fe37218dae2b997dd293283752 in odlparent
changed headerLocation of checkstyle from LISENCE.txt to
EPL-LICENSE.regexp.txt
https://maven.apache.org/plugins/maven-checkstyle-plugin/checkstyle-mojo.html#headerLocation
It causes the following error.
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:2.16:check (default) on project project-neutron-parent: Failed during checkstyle execution: Failed during checkstyle configuration: unable to read /home/yamahata/openstack/odl/odl/upstream/neutron-maven-upload/parent/target/checkstyle-checker.xml - unable to parse configuration stream - Property ${checkstyle.header.file} has not been set -> [Help 1]
This patch removes the special configuration for neutron northbound so that
it uses odlparent config.
Change-Id: Ib499f2ce0b25a11c2dc73b716a69c78f057f70cf
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Thu, 21 Apr 2016 19:43:09 +0000 (19:43 +0000)]
Merge "Use AAAShiroFilter as TokenAuthFilter was deprecated in Be"
Ryan Goulding [Wed, 20 Apr 2016 19:40:19 +0000 (15:40 -0400)]
Use AAAShiroFilter as TokenAuthFilter was deprecated in Be
Use the correct shiro filter for neutron endpoints. TokenAuthFilter was
deprecated from the AAA in Beryllium with this commit:
https://git.opendaylight.org/gerrit/#/c/30677/
AAAShiroFilter is the javax.servlet.Filter that replaces TokenAuthFilter,
and provides additional Realm plugins for the AAA framework.
Change-Id: I7daae17a784037f90b9e814f1bc42fee9a596e00
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Ritu Sood [Thu, 24 Mar 2016 21:49:02 +0000 (14:49 -0700)]
Adding Yang Model for Openstack Host Configuration
Adding YANG module to define host configurations that is used
in specifying the physical host type and configuration
for host system. Examples of host type are network node,
compute node, etc. Example for configuration is a Json string
like {'supported_vnic_types': 'VNIC_NORMAL', 'vif_type':
'VIF_TYPE_VHOST_USER', ….} etc. This information is read
and populated by networking-odl in agent_db in Neutron and
is used by Openstack Neutron scheduler. Supports features
like Port binding and Router scheduling.
Change-Id: I630e6d88f1b1beb6d5e442ccddf6a5d7137a5a51
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Isaku Yamahata [Tue, 19 Apr 2016 07:42:08 +0000 (07:42 +0000)]
Merge "Bug 4815 - Server error with transaction chain failed"
Isaku Yamahata [Thu, 17 Dec 2015 23:34:41 +0000 (15:34 -0800)]
Bug 4815 - Server error with transaction chain failed
Depending on condition, neutron northbound returns Server Error 500
with the following exception. This is because the retry logic
in AbstractNeutronInterface is not correct.
Transaction chain can't be used after transaction aborted. So this patch
moves up the retry logic and re-create transaction on transaction abort.
> 2015-12-16 15:15:16,429 | ERROR | neutron/networks | ContainerResponse | 47 - com.sun.jersey.jersey-server - 1.17.0 | The RuntimeException could not be mapped to a response, re-throwing to the HTTP container
> java.lang.IllegalStateException: Transaction chain has failed
> at org.opendaylight.controller.sal.core.spi.data.AbstractSnapshotBackedTransactionChain$Shutdown.getSnapshot(AbstractSnapshotBackedTransactionChain.java:93)
> at org.opendaylight.controller.sal.core.spi.data.AbstractSnapshotBackedTransactionChain.getSnapshot(AbstractSnapshotBackedTransactionChain.java:113)
> at org.opendaylight.controller.sal.core.spi.data.AbstractSnapshotBackedTransactionChain.newWriteOnlyTransaction(AbstractSnapshotBackedTransactionChain.java:158)
> at org.opendaylight.controller.cluster.datastore.LocalTransactionChain.newWriteOnlyTransaction(LocalTransactionChain.java:76)
> at org.opendaylight.controller.cluster.datastore.AbstractTransactionContextFactory.createLocalTransactionContext(AbstractTransactionContextFactory.java:209)
> at org.opendaylight.controller.cluster.datastore.AbstractTransactionContextFactory.maybeCreateLocalTransactionContext(AbstractTransactionContextFactory.java:58)
> at org.opendaylight.controller.cluster.datastore.AbstractTransactionContextFactory.onFindPrimaryShardSuccess(AbstractTransactionContextFactory.java:73)
> at org.opendaylight.controller.cluster.datastore.AbstractTransactionContextFactory.newTransactionContextWrapper(AbstractTransactionContextFactory.java:99)
> at org.opendaylight.controller.cluster.datastore.TransactionProxy.getContextWrapper(TransactionProxy.java:317)
> at org.opendaylight.controller.cluster.datastore.TransactionProxy.getContextWrapper(TransactionProxy.java:308)
> at org.opendaylight.controller.cluster.datastore.TransactionProxy.merge(TransactionProxy.java:159)
> at org.opendaylight.controller.cluster.databroker.AbstractDOMBrokerWriteTransaction.merge(AbstractDOMBrokerWriteTransaction.java:95)
> at org.opendaylight.controller.md.sal.binding.impl.BindingDOMWriteTransactionAdapter.ensureParentsByMerge(BindingDOMWriteTransactionAdapter.java:67)
> at org.opendaylight.controller.md.sal.binding.impl.AbstractWriteTransaction.put(AbstractWriteTransaction.java:41)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.updateMd(AbstractNeutronInterface.java:177)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.addMd(AbstractNeutronInterface.java:151)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.add(AbstractNeutronInterface.java:360)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface$8.action(AbstractNeutronInterface.java:370)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface$8.action(AbstractNeutronInterface.java:367)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.chainWrapper1(AbstractNeutronInterface.java:94)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.add(AbstractNeutronInterface.java:366)
> at org.opendaylight.neutron.northbound.api.AbstractNeutronNorthboundIAware.create(AbstractNeutronNorthboundIAware.java:94)[235:org.opendaylight.neutron.northbound-api:0.6.0.SNAPSHOT]
> at org.opendaylight.neutron.northbound.api.NeutronNetworksNorthbound.createNetworks(NeutronNetworksNorthbound.java:239)[235:org.opendaylight.neutron.northbound-api:0.6.0.SNAPSHOT]
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)[:1.8.0_72-internal]
> at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)[:1.8.0_72-internal]
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)[:1.8.0_72-internal]
> at java.lang.reflect.Method.invoke(Method.java:498)[:1.8.0_72-internal]
> at com.sun.jersey.spi.container.JavaMethodInvokerFactory$1.invoke(JavaMethodInvokerFactory.java:60)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.model.method.dispatch.AbstractResourceMethodDispatchProvider$ResponseOutInvoker._dispatch(AbstractResourceMethodDispatchProvider.java:205)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.model.method.dispatch.ResourceJavaMethodDispatcher.dispatch(ResourceJavaMethodDispatcher.java:75)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.HttpMethodRule.accept(HttpMethodRule.java:302)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.ResourceClassRule.accept(ResourceClassRule.java:108)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.RightHandPathRule.accept(RightHandPathRule.java:147)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.RootResourceClassesRule.accept(RootResourceClassesRule.java:84)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl._handleRequest(WebApplicationImpl.java:1511)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl._handleRequest(WebApplicationImpl.java:1442)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl.handleRequest(WebApplicationImpl.java:1391)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl.handleRequest(WebApplicationImpl.java:1381)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.spi.container.servlet.WebComponent.service(WebComponent.java:416)[174:com.sun.jersey.servlet:1.17.0]
> at com.sun.jersey.spi.container.servlet.ServletContainer.service(ServletContainer.java:538)[174:com.sun.jersey.servlet:1.17.0]
> at com.sun.jersey.spi.container.servlet.ServletContainer.service(ServletContainer.java:716)[174:com.sun.jersey.servlet:1.17.0]
> at javax.servlet.http.HttpServlet.service(HttpServlet.java:668)[142:org.apache.geronimo.specs.geronimo-servlet_3.0_spec:1.0.0]
> at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:684)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1496)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlets.CrossOriginFilter.handle(CrossOriginFilter.java:247)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlets.CrossOriginFilter.doFilter(CrossOriginFilter.java:210)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1467)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:501)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceServletHandler.doHandle(HttpServiceServletHandler.java:69)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:137)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:557)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:231)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1086)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceContext.doHandle(HttpServiceContext.java:240)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:429)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:193)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1020)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.JettyServerHandlerCollection.handle(JettyServerHandlerCollection.java:75)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.Server.handle(Server.java:366)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:494)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:982)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1043)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:696)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:53)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at java.lang.Thread.run(Thread.java:745)[:1.8.0_72-internal]
> at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1086)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceContext.doHandle(HttpServiceContext.java:240)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:429)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:193)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1020)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.JettyServerHandlerCollection.handle(JettyServerHandlerCollection.java:75)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.Server.handle(Server.java:366)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:494)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:982)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1043)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:696)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:53)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at java.lang.Thread.run(Thread.java:745)[:1.8.0_72-internal]
Change-Id: I3b6127bc9c395360a5bf408402734279015def0d
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Ryan Goulding [Wed, 13 Apr 2016 15:35:36 +0000 (11:35 -0400)]
Add dependency on javax.validation validation-api
In order to upgrade to org.eclipse.persistence to 2.6.0+ in odlparent,
downstream consumers must include the dependency on javax.validation
validation-api version 1.1.0.Final as described here:
http://stackoverflow.com/questions/
28568154/
how-to-get-eclipselink-2-6-0-m3-working-with-jersey-1-18-3
The work to upgrade odlparent's moxy version is included in this patch:
https://git.opendaylight.org/gerrit/#/c/37541/
Change-Id: I808f90066189852300e38df6b50daf55d5c3aab2
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Isaku Yamahata [Tue, 22 Mar 2016 22:37:17 +0000 (22:37 +0000)]
Merge "Bug 5478 - northbound: improve 404 check on update/delete"
Isaku Yamahata [Tue, 22 Mar 2016 13:59:32 +0000 (13:59 +0000)]
Merge "ITNeutronE2E: refactor test_fetch"
Isaku Yamahata [Mon, 21 Mar 2016 17:32:25 +0000 (10:32 -0700)]
ITNeutronE2E: refactor test_fetch
refactor test_fetch method.
Change-Id: I009968c1342a94cf6bd0ed40fe0c9b006b97c5cd
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Mon, 21 Mar 2016 21:21:43 +0000 (21:21 +0000)]
Merge "NeutronSecurityRuleInterface: remove nonsense code"
Isaku Yamahata [Mon, 21 Mar 2016 21:21:12 +0000 (21:21 +0000)]
Merge "NeutronSubnetInterface: remove non sense code"
Isaku Yamahata [Mon, 21 Mar 2016 21:20:53 +0000 (21:20 +0000)]
Merge "NeutronLoadBalancerPoolInterface: remove unused static final variables"
Isaku Yamahata [Fri, 18 Mar 2016 19:22:48 +0000 (12:22 -0700)]
Bug 5478 - northbound: improve 404 check on update/delete
canDelete should not be called on deletion on non-exist resource. Instead,
it should return 404 immediately without calling I*Aware.
This patch improves 404 check by adds missing 404 check by checking
return value on update and delete with a bit clean up.
Change-Id: Ia4fe068540022f687d942204f3b243ccbcbac578
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Sat, 19 Mar 2016 16:24:56 +0000 (16:24 +0000)]
Merge "Do not deploy the karaf artifact"
Isaku Yamahata [Fri, 18 Mar 2016 18:53:37 +0000 (11:53 -0700)]
NeutronLoadBalancerPoolInterface: remove unused static final variables
Change-Id: I31e4effccd59c0a6f9a08a452ce2411231d590df
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Fri, 18 Mar 2016 18:47:45 +0000 (11:47 -0700)]
NeutronSubnetInterface: remove non sense code
addSubnet and removeSubnet methods carries legacy code
which doesn't have any effect.
This patch Removes them and simplifies the code
Change-Id: I30d7f71336f9b7a99690235a7474e2fda67b8315
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Fri, 18 Mar 2016 18:34:15 +0000 (11:34 -0700)]
NeutronSecurityRuleInterface: remove nonsense code
updateSecGroupRuleInSecurityGroup and removeSecGroupRuleFromSecurityGroup
have no effect any more. They modifies variables no one else refers to.
There is no point to carry those methods.
So this patch removes those methods and simplify related code.
Change-Id: I378b8ce72b2d064940a5f345d1b55e60dd9981ae
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Thanh Ha [Wed, 16 Mar 2016 22:38:11 +0000 (18:38 -0400)]
Do not deploy the karaf artifact
The karaf artifact is only used for testing. It does not need to be
released to the world.
Change-Id: Ic1ff344fa606666c08ec6c75e89984c535454bf3
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Mike Kolesnik [Tue, 15 Mar 2016 06:33:21 +0000 (08:33 +0200)]
Use controller.mdsal.version when appropriate
Use the property controller.mdsal.version when looking for mdsal
artifacts from org.opendaylight.controller.
Otherwise, it's confusing since mdsal.version actually referrs to
artifacts from org.opendaylight.mdsal.
Change-Id: I9571969998a04a0fdae65eb61ffab414b45ee60d
Signed-off-by: Mike Kolesnik <mkolesni@redhat.com>
Isaku Yamahata [Wed, 3 Feb 2016 10:02:33 +0000 (02:02 -0800)]
Bug 4550 - always rebuild security group rule in security group
The change set of
906836c289a7e4c3d33669d90515d77e75cfb6d,
the effort transparent neutron northbound api, caused not to update
security group rule in security group.
So always rebuild rules in NeutronSecurityGroup and keep
security group rule in security group of yang model empty list.
openstack service provider that uses MD-SAL should not use
security group rules in security rule. But it should listen to
data change notification on security group rules.
Change-Id: I7da2e23d9c45549d2f5d138dfea148e36bd8061b
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Thu, 10 Mar 2016 23:37:25 +0000 (23:37 +0000)]
Merge "Bug 5431: Fixing bug & adding new tests for code coverage"
Isaku Yamahata [Wed, 9 Mar 2016 10:10:51 +0000 (10:10 +0000)]
Merge "Added PortSecurity attribute support"
Isaku Yamahata [Wed, 9 Mar 2016 10:02:04 +0000 (10:02 +0000)]
Merge "Bug 4354 -fromMd() doesn't convert tenant_id to neutron object"
Isaku Yamahata [Wed, 9 Mar 2016 10:00:59 +0000 (10:00 +0000)]
Merge "Bug 5431 - Fixing bug & adding new tests for VPN files"
Isaku Yamahata [Wed, 9 Mar 2016 10:00:19 +0000 (10:00 +0000)]
Merge "Bug 5431 - Fixing bug & adding new tests for LB files"
Dileep [Wed, 17 Feb 2016 14:12:39 +0000 (06:12 -0800)]
Added PortSecurity attribute support
Added portsecurity extension as augmentation for backward compatibility
Refer - http://developer.openstack.org/api-ref-networking-v2-ext.html
Change-Id: I75523702f6b1e83f192587396e3d103e507872d3
Signed-off-by: Dileep <dileep.ranganathan@intel.com>
Ritu Sood [Sat, 27 Feb 2016 06:03:27 +0000 (22:03 -0800)]
Bug 4354 -fromMd() doesn't convert tenant_id to neutron object
L2 Gateway and L2 Gateway connection doesn't convert
tenant_id to neutron object properly. Adding a test case
to improve coverage. Fixed a test case to use proper
UUID format.
Change-Id: I92518fbffe5ee01c68c2bea79cb7299330737f4a
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Sat, 27 Feb 2016 04:43:08 +0000 (20:43 -0800)]
Bug 5431: Fixing bug & adding new tests for code coverage
Fixing parameter type in @GET function in
some NN API and adding new tests for NN API
files. Increased code coverage ~ 20% for these files
Change-Id: I63a871b7fc0270e325da0d03ef991c5aaa6f8843
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Sat, 27 Feb 2016 02:05:59 +0000 (18:05 -0800)]
Bug 5431 - Fixing bug & adding new tests for VPN files
Fixing parameter type in @GET function in IPSEC
Site Connection and VPN NN API adding new tests
VPN files.
Increases code coverage ~20% for these files
Change-Id: I7061735fec84bedc1bae210dcfa4bc7b5df464c0
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Fri, 26 Feb 2016 03:20:46 +0000 (19:20 -0800)]
Bug 5431 - Fixing bug & adding new tests for LB files
Fixing wrong parameter type in @GET function in
LB NN API and adding new tests for LB
files. Increased code coverage ~ 20% for these files
Change-Id: I4a5c373fc199c868655f6fef6be581cef761d24a
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Fri, 26 Feb 2016 01:55:46 +0000 (17:55 -0800)]
Bug 5431 - Fixing bug & adding new tests for firewall file
Fixing parameter type in @GET function in
firewall_policy NN API adding new tests for firewall
files. Increased code coverage ~ 20% for these files
Change-Id: Ic73601e99d91d6e67b7db7852c5e1f07974b300b
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Thu, 25 Feb 2016 04:03:26 +0000 (20:03 -0800)]
Adding new test cases for Security Group and rules API
Adding a helper function to iterate over the JSON
string used during create to extract key values
pairs and then invoke GET with query with that
key value pair. Updated security group and security
rule tests to call this new tests. ~15% Code coverage
improvement with these new tests for the two files.
Change-Id: Ib9a8878a49feb0d897f8d67b50ca2f2429dc8341
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Isaku Yamahata [Fri, 4 Mar 2016 23:14:43 +0000 (23:14 +0000)]
Merge "Bug 5210 - GET/DELETE L2GatewayConnection with optional portId"
Isaku Yamahata [Fri, 4 Mar 2016 23:14:33 +0000 (23:14 +0000)]
Merge "maven version requirement is 3.1.1"
Ritu Sood [Mon, 11 Jan 2016 20:39:11 +0000 (12:39 -0800)]
Bug 4944 - Fixing a QueryParam name typo
QueryParam string is wrongly called shared when it
should be admin_state_up. Added test for checking
response of a query in a collection and added
testcase for reproducing the bug. Changeset
445690e94dd990ad6d117b4d7c86e95b8bf9392b
in stable/beryllium fixes the same issue.
Change-Id: I2e5a40ed392d57d9f1d61696910ad3613cdb206e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Dileep [Fri, 5 Feb 2016 15:22:07 +0000 (07:22 -0800)]
Bug 5210 - GET/DELETE L2GatewayConnection with optional portId
Removed getNeutronInterface method, using getNeutronCRUD instead.
Changed PortId null check to avoid NPE.
Added test cases with optional PortId.
Change-Id: I68c118fbcdde8b24cc4210bb54d7cee7c8fc9139
Signed-off-by: Dileep <dileep.ranganathan@intel.com>
(cherry picked from commit
82bc8d9b594e17a904c4a72dd251b886e4698fc7)
Isaku Yamahata [Sat, 13 Feb 2016 00:20:43 +0000 (00:20 +0000)]
Merge "Bug 5037 - Adding tests for L2 Gateway Connection API"
Ritu Sood [Mon, 1 Feb 2016 03:44:19 +0000 (19:44 -0800)]
Bug 5037 - Adding tests for L2 Gateway Connection API
Adding tests for L2 Gateway Connection Northbound API.
Patch also contains fix for a typo
in the NeutronL2gatewayConnectionNorthbound file.
Change-Id: I257008bfe0a897eb778cc6916a15261b8bca0612
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>
(cherry picked from commit
3d9691433d3e4f25f1168802d57b96471129e407)
Isaku Yamahata [Fri, 12 Feb 2016 21:51:18 +0000 (13:51 -0800)]
maven version requirement is 3.1.1
As maven version is 3.1.1 or newer, set to it.
Although this doesn't harm much, this patch is for consistency.
Change-Id: Iae7cd197bb6c18e7a308d8acfce9574653dfa6b9
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Fri, 12 Feb 2016 19:48:26 +0000 (19:48 +0000)]
Merge "Adding tests for L2 Gateway Northbound API"
Isaku Yamahata [Fri, 12 Feb 2016 19:33:22 +0000 (19:33 +0000)]
Merge "Bug 5278 [BORON] Remove checks that require neutron providers for L3 and others"
Ritu Sood [Tue, 19 Jan 2016 08:08:08 +0000 (00:08 -0800)]
Adding tests for L2 Gateway Northbound API
Adding tests for post, put, get and Delete API's
Change-Id: Iaa8412a48f03a90cee715b65d49c6e17dd4cb77c
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
(cherry picked from commit
b4f14c65faa90889dfbb4274d7a40e6b7468eb27)
Isaku Yamahata [Sun, 31 Jan 2016 13:51:18 +0000 (13:51 +0000)]
Bug 5278 [BORON] Remove checks that require neutron providers for L3 and others
DO NOT MERGE in Beryllium.
This patch is follow up of
49479f5244ac3c7a9a2fc06f7bbb0fed9ca3b203
It missed L3, lbaaspools, and Metering Label.
This patch includes also related code clean up to use this.getInstances().
Change-Id: I82c3330dd16309fe9d6bf90e96b371ddafa9a12a
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Pramod [Wed, 3 Feb 2016 21:46:44 +0000 (13:46 -0800)]
Adding more test cases to increase the code coverage of Neutron tests
Refactoring the code for collection tests to remove the wait from the test
Fixed the required changes
Change-Id: If6ff2851e26836cde25161ba43aade0d10448a91
Signed-off-by: Pramod <pramod.raghavendra.jayathirth@intel.com>
Vishal Thapar [Mon, 1 Feb 2016 08:51:15 +0000 (14:21 +0530)]
BUG-5137: extraroutes in router not working
Neutron API defines extraroutes as list of destination and nexthop but in
neutron-l3.yang they're defined as list of strings. So, when these routes
are present in a router object sent by openstack we are unable to store
them in MDSAL. Fix is to change the type of routes in l3-attributes to
match with what Neutron API sends us.
Change-Id: I83519edcbdf6f4fcae34ae9650cf8ede40256296
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
(cherry picked from commit
59f3919dd01797fb056fa5dcdebda2076a936954)
Pramod [Mon, 25 Jan 2016 22:11:05 +0000 (14:11 -0800)]
Add more tests to improve Code Coverage for Neutron Load balancer related files
Change-Id: If940948c993930f47840c935f14bcd039d7f5ba8
Signed-off-by: Pramod <pramod.raghavendra.jayathirth@intel.com>
(cherry picked from commit
492923ba34786b212e263cb84f452e26659be15d)
Thanh Ha [Thu, 21 Jan 2016 20:56:52 +0000 (15:56 -0500)]
Bump yangtools to 1.0.0-SNAPSHOT
Change-Id: I74de7043fe75c081f6d5a8b8b686fb3e9516aacd
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Flavio Fernandes [Tue, 22 Sep 2015 14:51:09 +0000 (10:51 -0400)]
[BORON] Remove checks that require neutron providers
DO NOT MERGE in Beryllium.
- Remove exceptions caused when there are no I*Aware providers registered
Patch set 2: Make I*Aware interfaces deprecated in a separate gerrit [1]
Patch set 6: Rebase and use AbstractNeutronNorthboundIAware
With this change, we take one step closer to the desired goal of making
neutron northbound md-sal based. By Boron, ODL applications will be expected
to access all neutron northbound related data via the API provided by
the corresponding yang data models.
[1]: https://git.opendaylight.org/gerrit/#/c/27297/
Change-Id: I15ae07ff85620b3ceaf9e13b6aa386ec0aa669b0
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Thanh Ha [Fri, 15 Jan 2016 04:40:30 +0000 (23:40 -0500)]
Bump versions by 0.1.0 for next dev cycle
Change-Id: Ia84b1a7e2860e4af71b119b7f0f9e98cf40d51e6
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
Isaku Yamahata [Thu, 14 Jan 2016 19:57:56 +0000 (19:57 +0000)]
Merge "Add more tests to improve Code Coverage for VPN related files"
Isaku Yamahata [Thu, 14 Jan 2016 19:57:53 +0000 (19:57 +0000)]
Merge "Add more tests to improve Code Coverage for Load Balancer related files"
Isaku Yamahata [Thu, 14 Jan 2016 19:57:49 +0000 (19:57 +0000)]
Merge "Add more tests to improve code coverage"
Isaku Yamahata [Thu, 14 Jan 2016 19:57:46 +0000 (19:57 +0000)]
Merge "Add more tests to improve Code Coverage for Metering files"
Isaku Yamahata [Thu, 14 Jan 2016 19:57:43 +0000 (19:57 +0000)]
Merge "Add more tests to improve Code Coverage"
Ritu Sood [Thu, 31 Dec 2015 04:15:27 +0000 (20:15 -0800)]
Enhance Neutron Firewall tests to get better code coverage
Adding new test get with query achieved more functional coverage.
Moving the collection_get_test later after the nodes were created
helped getting more condition coverage in list functions.
These changes improve code coverage by ~15% in the files covered.
Also reverted the changes made to existing tests.
Change-Id: I8da4c7b7058915bb1c23a857b5f85d4281c3700e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Mon, 11 Jan 2016 05:38:42 +0000 (21:38 -0800)]
Add more tests to improve Code Coverage
Adding new test get with query achieved more functional coverage.
Moving the collection_get_test later after the nodes were created
helped getting more condition coverage in list functions.
These changes improve code coverage by 12% for these files
Change-Id: Ib80bd954ce2373fc13d36191d27e2bdb06f74a39
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Mon, 11 Jan 2016 01:50:46 +0000 (17:50 -0800)]
Add more tests to improve Code Coverage for Metering files
Adding new test get with query achieved more functional coverage.
Moving the collection_get_test later after the nodes were created
helped getting more condition coverage in list functions.
These changes improve code coverage by ~13% in these files
Change-Id: I2c55ee615e432280c73e3ee643e476a7528fef8e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Mon, 11 Jan 2016 01:03:07 +0000 (17:03 -0800)]
Add more tests to improve Code Coverage for Load Balancer related files
Adding new test get with query achieved more functional coverage.
Moving the collection_get_test later after the nodes were created
helped getting more condition coverage in list functions.
These changes improve code coverage by ~15% in these files
Change-Id: I7fff8a65b3a33bab5f97a9480fee177982add169
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Flavio Fernandes [Fri, 8 Jan 2016 17:37:26 +0000 (17:37 +0000)]
Merge "Bug 4845 - prefix neutron stuff to model bundle name"
Ritu Sood [Fri, 8 Jan 2016 04:19:05 +0000 (20:19 -0800)]
Add more tests to improve Code Coverage for VPN related files
Adding new test get with query achieved more functional coverage.
Moving the collection_get_test later after the nodes were created
helped getting more condition coverage in list functions.
These changes improve code coverage by ~17% in these files
Change-Id: Ia256aa2df4f0be138e6683c3dd0b42c8f42a724b
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Ritu Sood [Thu, 7 Jan 2016 03:43:23 +0000 (19:43 -0800)]
Add more tests to improve code coverage
Adding new tests 1)update and 2)get with query achieved more
functional coverage. Moving the collection_get_test later after the
nodes were created helped getting more condition coverage in
listGroups and listRules functions. Improved code coverage from
66% to 80% for NeutronSecurityGroupsNorthbound and
42% to 70% NeutronSecurityRulesNorthbound.
Change-Id: I56708a0c1c66158dd29784a9a2edd99e5749077e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
gong yong sheng [Wed, 30 Dec 2015 01:52:52 +0000 (09:52 +0800)]
Bug 4848 - Remove provider attributes from top neutron container
Provider attributes should not be under top neutron container.
Change-Id: I0369d651f9791d3077cf65fed2b6509a598dc7aa
Signed-off-by: yong sheng gong <gong.yongsheng@99cloud.net>
gong yong sheng [Mon, 28 Dec 2015 10:34:59 +0000 (18:34 +0800)]
Bug 4845 - prefix neutron stuff to model bundle name
Add maven-bundle-plugin build plugin in POM to generate proper
bundle name.
Change-Id: I95821608ffe3acc63ced3c153bd880d404f4ff3d
Signed-off-by: yong sheng gong <gong.yongsheng@99cloud.net>
Flavio Fernandes [Thu, 24 Dec 2015 02:29:24 +0000 (02:29 +0000)]
Merge "Bug 4818 dnsNameservers not stored in subnet MDSAL"
Isaku Yamahata [Wed, 23 Dec 2015 22:31:48 +0000 (14:31 -0800)]
Bug 4838 - add missing LICENSE file
This patch adds a LICENSE file to show the terms of license explicitly.
Change-Id: Ie1732ea1b523220bb5a582e858a6dec6d279464e
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Vishal Thapar [Fri, 18 Dec 2015 08:37:30 +0000 (14:07 +0530)]
Bug 4818 dnsNameservers not stored in subnet MDSAL
NeutronSubnet.initDefaults() wrongly initializes dnsNameservers even if
user passed a value. There should be a null check and initialization should
be done only if user didn't pass anything.
This adds check for dnsNameservers and improves the test cases in NeutronSubnetTests
Change-Id: Ibade65f18a65b9c8e6c0a26de02bbbbbe5cac0b7
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Isaku Yamahata [Tue, 22 Dec 2015 17:10:19 +0000 (17:10 +0000)]
Merge "Bug 4805 add host_routes info to MDSAL"
Flavio Fernandes [Tue, 22 Dec 2015 15:10:15 +0000 (15:10 +0000)]
Merge "Bug 4829 - ITNeutronE2E.java: connect test cases for bgpvpn"
Vishal Thapar [Thu, 17 Dec 2015 20:50:23 +0000 (02:20 +0530)]
Bug 4805 add host_routes info to MDSAL
Transcriber is missing code to add hostRoute information to
MDSAL when Subnet is created/updated.
This change adds code to toMd() and fromMd() to handle hostRoutes.
Change-Id: I7b7b7dc5092971d86aec38e5365518712d84e8c2
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Flavio Fernandes [Tue, 22 Dec 2015 04:23:05 +0000 (04:23 +0000)]
Merge "Bug 4797 - prefix neutron stuff to model bundle name"
Isaku Yamahata [Mon, 21 Dec 2015 21:31:41 +0000 (13:31 -0800)]
Bug 4829 - ITNeutronE2E.java: connect test cases for bgpvpn
Somehow testsets for bgpvpn isn't connected to ITNeutronE2E
so that it isn't run by CI.
Change-Id: Ib125ffcef2ea2d69f45b3166ff41028292618ce9
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Mon, 21 Dec 2015 20:36:43 +0000 (20:36 +0000)]
Merge "Bug 4775 - handle cases when tenant-id is an empty string"
Flavio Fernandes [Mon, 21 Dec 2015 15:09:01 +0000 (15:09 +0000)]
Merge "Bug 4816 - Change common attributes' descriptions into resource neutral"
Flavio Fernandes [Mon, 21 Dec 2015 15:04:01 +0000 (15:04 +0000)]
Merge "Bug 4807 fix destination type in subnets yang"
Flavio Fernandes [Tue, 15 Dec 2015 04:33:25 +0000 (23:33 -0500)]
Bug 4775 - handle cases when tenant-id is an empty string
Enhanced transcriber to check cases when tenantId is an empty
string. Previous check were only looking for cases when tenantId
was null.
When ODL is used with l3 agent and l3_ha, subnet/network created
for handling keep alive traffic contains tenant id set to "".
Patch set 2: code review: Simplify changes by adding logic for
checking on empty string at the base object (NeutronObject).
Patch set 3: code review: Remove check in toMd of NeutronPortInterface
because it is no longer needed.
Patch set 4: add UT
Change-Id: I05b61d74fff9c1d4fe373446f60cf0bd98a54e14
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>
Flavio Fernandes [Fri, 18 Dec 2015 13:28:05 +0000 (13:28 +0000)]
Merge "Bug 4789 - model allowed address pairs"
gong yong sheng [Fri, 18 Dec 2015 07:56:06 +0000 (15:56 +0800)]
Bug 4816 - Change common attributes' descriptions into resource neutral
status, admin-status-up and tenant-id attributes are used by
many object types. Their descriptions should not be network
type specific.
Change-Id: Iaa671329e48fbc690aaefcc6ee99215730cc9703
Signed-off-by: yong sheng gong <gong.yongsheng@99cloud.net>
Vishal Thapar [Thu, 17 Dec 2015 21:15:05 +0000 (02:45 +0530)]
Bug 4807 fix destination type in subnets yang
host-routes attribute of subnets has destination
field defined as type inet:ip-address. As per Neutron
APIv2 spec it should be of type CIDR
Fix is to change it to ip-prefix
Change-Id: I0c8653b2e24eac06d98c5afd679a0a70d6669e3b
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
gong yong sheng [Thu, 17 Dec 2015 06:15:57 +0000 (14:15 +0800)]
Bug 4797 - prefix neutron stuff to model bundle name
Add maven-bundle-plugin build plugin in POM to generate proper
bundle name.
Change-Id: I05948de9feaace567d099c4147dda884aedfbec4
Signed-off-by: yong sheng gong <gong.yongsheng@99cloud.net>
Isaku Yamahata [Wed, 16 Dec 2015 04:06:24 +0000 (20:06 -0800)]
Bug 4789 - model allowed address pairs
Allowed address pairs doesn't include port id according to neutron spec.
actually ip_address is ip address or network address.
So for simplicity, networking-odl should pass always pass network address.
in case of ip address case, address/32 will be passed.
This patch should be merged after
https://git.opendaylight.org/gerrit/#/c/31402/
is merged
Change-Id: I40b0142cd9a1af8fd30d1fa987ae7b8a5fa49db2
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Flavio Fernandes [Tue, 15 Dec 2015 16:57:30 +0000 (16:57 +0000)]
Merge "Drop properties from features.xml"
Flavio Fernandes [Mon, 14 Dec 2015 15:40:59 +0000 (15:40 +0000)]
Merge "Bug 4745 - extra dhcp opt: neutron API updated to include ip_version"
Stephen Kitt [Fri, 11 Dec 2015 10:20:54 +0000 (11:20 +0100)]
Drop properties from features.xml
features-parent allows properties to be replaced with {{VERSION}} in
many places in features.xml; this helps clean up odlparent.
Change-Id: I38ef9c8668c92f9e1aced7466db110bf7626a6f7
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Tue, 8 Dec 2015 21:04:55 +0000 (13:04 -0800)]
Bug 4745 - extra dhcp opt: neutron API updated to include ip_version
Exception occurs as follows with the context of extra_dhcp_opts.
Neutron API of extra_dhcp_opts added ip_version to distinguish ipv4 and ip6.
It needs to be handled, otherwise exception occurs.
create/update port with extra_dhcp_opts
"extra_dhcp_opts": [
{
"opt_value": "pxelinux.0",
"ip_version": 4,
"opt_name": "bootfile-name"
},
{
"opt_value": "123.123.123.45",
"ip_version": 4,
"opt_name": "server-ip-address"
},
{
"opt_value": "123.123.123.123",
"ip_version": 4,
"opt_name": "tftp-server"
}
> 2015-12-08 10:06:30,307 | ERROR | v2/neutron/ports | ContainerResponse | 47 - com.sun.jersey.jersey-server - 1.17.0 | The RuntimeException could not be mapped to a response, re-throwing to the HTTP container
> java.lang.IllegalArgumentException: All keys must be specified for class org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.port.attributes.ExtraDhcpOptsKey. Missing key is getIpVersion. Supplied key is ExtraDhcpOptsKey [_optName=bootfile-na
> me]
> at com.google.common.base.Preconditions.checkArgument(Preconditions.java:145)
> at org.opendaylight.yangtools.binding.data.codec.impl.ValueContext.getAndSerialize(ValueContext.java:46)
> at org.opendaylight.yangtools.binding.data.codec.impl.IdentifiableItemCodec.serialize(IdentifiableItemCodec.java:124)
> at org.opendaylight.yangtools.binding.data.codec.impl.IdentifiableItemCodec.serialize(IdentifiableItemCodec.java:31)
> at org.opendaylight.yangtools.binding.data.codec.impl.KeyedListNodeCodecContext.serialize(KeyedListNodeCodecContext.java:80)
> at org.opendaylight.yangtools.binding.data.codec.impl.BindingToNormalizedStreamWriter.startMapEntryNode(BindingToNormalizedStreamWriter.java:156)
> at org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.port.attributes.ExtraDhcpOpts$StreamWriter.serialize(DataObjectSerializerPrototype.java)
> at org.opendaylight.yang.gen.v1.urn.opendaylight.neutron.ports.rev150712.ports.attributes.ports.Port$StreamWriter.serialize(DataObjectSerializerPrototype.java)
> at org.opendaylight.yangtools.binding.data.codec.impl.BindingNormalizedNodeCodecRegistry$DataObjectSerializerProxy.serialize(BindingNormalizedNodeCodecRegistry.java:295)
> at org.opendaylight.yangtools.binding.data.codec.impl.BindingNormalizedNodeCodecRegistry.toNormalizedNode(BindingNormalizedNodeCodecRegistry.java:106)
> at org.opendaylight.controller.md.sal.binding.impl.BindingToNormalizedNodeCodec.toNormalizedNode(BindingToNormalizedNodeCodec.java:133)
> at org.opendaylight.controller.md.sal.binding.impl.AbstractWriteTransaction.put(AbstractWriteTransaction.java:39)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.updateMd(AbstractNeutronInterface.java:177)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.addMd(AbstractNeutronInterface.java:151)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.add(AbstractNeutronInterface.java:360)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface$8.action(AbstractNeutronInterface.java:370)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface$8.action(AbstractNeutronInterface.java:367)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.chainWrapper1(AbstractNeutronInterface.java:94)
> at org.opendaylight.neutron.transcriber.AbstractNeutronInterface.add(AbstractNeutronInterface.java:366)
> at org.opendaylight.neutron.northbound.api.AbstractNeutronNorthboundIAware.create(AbstractNeutronNorthboundIAware.java:64)[235:org.opendaylight.neutron.northbound-api:0.6.0.SNAPSHOT]
> at org.opendaylight.neutron.northbound.api.NeutronPortsNorthbound.createPorts(NeutronPortsNorthbound.java:247)[235:org.opendaylight.neutron.northbound-api:0.6.0.SNAPSHOT]
> at sun.reflect.GeneratedMethodAccessor99.invoke(Unknown Source)
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)[:1.8.0_45-internal]
> at java.lang.reflect.Method.invoke(Method.java:497)[:1.8.0_45-internal]
> at com.sun.jersey.spi.container.JavaMethodInvokerFactory$1.invoke(JavaMethodInvokerFactory.java:60)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.model.method.dispatch.AbstractResourceMethodDispatchProvider$ResponseOutInvoker._dispatch(AbstractResourceMethodDispatchProvider.java:205)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.model.method.dispatch.ResourceJavaMethodDispatcher.dispatch(ResourceJavaMethodDispatcher.java:75)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.HttpMethodRule.accept(HttpMethodRule.java:302)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.ResourceClassRule.accept(ResourceClassRule.java:108)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.RightHandPathRule.accept(RightHandPathRule.java:147)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.RootResourceClassesRule.accept(RootResourceClassesRule.java:84)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl._handleRequest(WebApplicationImpl.java:1511)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl._handleRequest(WebApplicationImpl.java:1442)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl.handleRequest(WebApplicationImpl.java:1391)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl.handleRequest(WebApplicationImpl.java:1381)[47:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.spi.container.servlet.WebComponent.service(WebComponent.java:416)[174:com.sun.jersey.servlet:1.17.0]
> at com.sun.jersey.spi.container.servlet.ServletContainer.service(ServletContainer.java:538)[174:com.sun.jersey.servlet:1.17.0]
> at com.sun.jersey.spi.container.servlet.ServletContainer.service(ServletContainer.java:716)[174:com.sun.jersey.servlet:1.17.0]
> at javax.servlet.http.HttpServlet.service(HttpServlet.java:668)[142:org.apache.geronimo.specs.geronimo-servlet_3.0_spec:1.0.0]
> at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:684)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1496)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlets.CrossOriginFilter.handle(CrossOriginFilter.java:247)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlets.CrossOriginFilter.doFilter(CrossOriginFilter.java:210)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1467)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:501)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceServletHandler.doHandle(HttpServiceServletHandler.java:69)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:137)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:557)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:231)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1086)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceContext.doHandle(HttpServiceContext.java:240)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:429)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:193)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1020)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.JettyServerHandlerCollection.handle(JettyServerHandlerCollection.java:75)[157:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.Server.handle(Server.java:366)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:494)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:982)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1043)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:696)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:53)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)[148:org.eclipse.jetty.aggregate.jetty-all-server:8.
Change-Id: I6f2713ad6baedd60be58dbd38ca420906157810a
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Flavio Fernandes [Fri, 4 Dec 2015 16:00:04 +0000 (16:00 +0000)]
Merge "L2gateway: Added yang, APIs,Transcriber for L2gateway."
raveek [Wed, 29 Jul 2015 21:08:25 +0000 (02:53 +0545)]
L2gateway: Added yang, APIs,Transcriber for L2gateway.
Patch Set 4: Removed I*Aware related code, Addressed review comments.
Patch Set 5: Aligned code with merged changes. Addressed review
comments.
Openstack Neutron API:
https://github.com/openstack/networking-l2gw/blob/master/specs/kilo/l2-gateway-api.rst
Patch Set 8: fixed the review comments
Patch Set 9: fixed the review comments
Change-Id: Icfc93bc39f56f6b103b13834359128f29df804b9
Signed-off-by: raveek <raveek@hp.com>
Isaku Yamahata [Thu, 20 Aug 2015 23:50:53 +0000 (16:50 -0700)]
Bug 4188 - null pointer exception in NeutronSubnetInterface.addSubnet()
When subnet is updated, the following exception happends.
This is because the corresponding network isn't always created before
subnet creation depending on situation.
The order of creation/update of network and subnet can be reordered.
In fact subnets list of network in MD-SAL isn't already well-maintained,
and it can be calculated from list of subnets with its parent network uuid.
Actually Openstack Neutron db generates subnet list by foreign key
with ORM mapper. So it makes sense for ODL neutron northbound to
eliminate subnet list instead of maintaining subnet list correct.
> 2015-08-20 14:10:21,025 | ERROR | p492280269-15502 | ContainerResponse | 195 - com.sun.jersey.jersey-server - 1.17.0 | The RuntimeException could not be mapped to a response, re-throwing to the HTTP container
> java.lang.NullPointerException
> at org.opendaylight.neutron.transcriber.NeutronSubnetInterface.addSubnet(NeutronSubnetInterface.java:118)
> at org.opendaylight.neutron.northbound.api.NeutronSubnetsNorthbound.createSubnets(NeutronSubnetsNorthbound.java:218)[373:org.opendaylight.neutron.northbound-api:0.6.0.SNAPSHOT]
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)[:1.7.0_79]
> at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)[:1.7.0_79]
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)[:1.7.0_79]
> at java.lang.reflect.Method.invoke(Method.java:606)[:1.7.0_79]
> at com.sun.jersey.spi.container.JavaMethodInvokerFactory$1.invoke(JavaMethodInvokerFactory.java:60)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.model.method.dispatch.AbstractResourceMethodDispatchProvider$ResponseOutInvoker._dispatch(AbstractResourceMethodDispatchProvider.java:205)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.model.method.dispatch.ResourceJavaMethodDispatcher.dispatch(ResourceJavaMethodDispatcher.java:75)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.HttpMethodRule.accept(HttpMethodRule.java:302)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.ResourceClassRule.accept(ResourceClassRule.java:108)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.RightHandPathRule.accept(RightHandPathRule.java:147)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.uri.rules.RootResourceClassesRule.accept(RootResourceClassesRule.java:84)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl._handleRequest(WebApplicationImpl.java:1511)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl._handleRequest(WebApplicationImpl.java:1442)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl.handleRequest(WebApplicationImpl.java:1391)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.server.impl.application.WebApplicationImpl.handleRequest(WebApplicationImpl.java:1381)[195:com.sun.jersey.jersey-server:1.17.0]
> at com.sun.jersey.spi.container.servlet.WebComponent.service(WebComponent.java:416)[196:com.sun.jersey.servlet:1.17.0]
> at com.sun.jersey.spi.container.servlet.ServletContainer.service(ServletContainer.java:538)[196:com.sun.jersey.servlet:1.17.0]
> at com.sun.jersey.spi.container.servlet.ServletContainer.service(ServletContainer.java:716)[196:com.sun.jersey.servlet:1.17.0]
> at javax.servlet.http.HttpServlet.service(HttpServlet.java:668)[159:org.apache.geronimo.specs.geronimo-servlet_3.0_spec:1.0.0]
> at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:684)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1496)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlets.CrossOriginFilter.handle(CrossOriginFilter.java:247)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlets.CrossOriginFilter.doFilter(CrossOriginFilter.java:210)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1467)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:501)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceServletHandler.doHandle(HttpServiceServletHandler.java:69)[174:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:137)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:557)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:231)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1086)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.HttpServiceContext.doHandle(HttpServiceContext.java:240)[174:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:429)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:193)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1020)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:135)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.ops4j.pax.web.service.jetty.internal.JettyServerHandlerCollection.handle(JettyServerHandlerCollection.java:75)[174:org.ops4j.pax.web.pax-web-jetty:3.1.4]
> at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:116)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.Server.handle(Server.java:370)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.handleRequest(AbstractHttpConnection.java:494)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection.content(AbstractHttpConnection.java:982)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AbstractHttpConnection$RequestHandler.content(AbstractHttpConnection.java:1043)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseNext(HttpParser.java:865)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.http.HttpParser.parseAvailable(HttpParser.java:240)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.server.AsyncHttpConnection.handle(AsyncHttpConnection.java:82)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint.handle(SelectChannelEndPoint.java:696)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.io.nio.SelectChannelEndPoint$1.run(SelectChannelEndPoint.java:53)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:608)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:543)[165:org.eclipse.jetty.aggregate.jetty-all-server:8.1.15.v20140411]
> at java.lang.Thread.run(Thread.java:745)[:1.7.0_79]
Change-Id: Ie7861ae292ee7a1666505f0901cad99fb29a9ff3
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Flavio Fernandes [Fri, 23 Oct 2015 17:29:36 +0000 (17:29 +0000)]
Merge changes from topic 'bgpvpn-fixup'
* changes:
use INeutronObject instead of NeutronObject where possible
bgpvpn: simplify bgpvpn related code
northbound: Split out AD-SAL logic from AbstractNeturonNorthbound
Isaku Yamahata [Fri, 16 Oct 2015 23:42:50 +0000 (16:42 -0700)]
remove unused ConcurrentHashMap
The variable somehow was missed to be deleted.
Change-Id: I9784b64ee7b32154241d0efe183756e60404b251
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Fri, 9 Oct 2015 22:46:33 +0000 (15:46 -0700)]
use INeutronObject instead of NeutronObject where possible
Change-Id: Id5abdaa58af68985d4ec88ecfd4a1e0c9a60b653
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Fri, 9 Oct 2015 20:01:09 +0000 (13:01 -0700)]
bgpvpn: simplify bgpvpn related code
This patch simplifies bgpvpn related code.
Change-Id: I36fdb996d4b20dfb1319ec11fe815a882b1bed85
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
Isaku Yamahata [Fri, 9 Oct 2015 18:43:46 +0000 (11:43 -0700)]
northbound: Split out AD-SAL logic from AbstractNeturonNorthbound
This is a follow up patch for
changeset of
43f88bf737adcb2e5df9e106d8b491de1549c41d
change id of I3de3f9461047363501f0e3a6428580f93402c65b
Neutron*Northbound: consolidate crud logic
and
changeset of
aafe31458a9990f50023566aed7f7b296b95067c,
change id of I2c36c104dbd6c891a36c10419ed28d8060958d29.
BGPVPN: Added yang, api and transcriber for BGPVPN
bgpvpn which was newly introduced after AD-SAL deprection doesn't implement
AD-SAL. So the class of NeutronBgpvpnsNorthbound requires its super class,
AbstractNeutronNorthbound, not to use AD-SAL.
This patch factors out ad-sal logic from AbstractNeutronNorthbound
into new subclass, AbstractNeutronNorthboundIAware with minor compilation
adjustment.
Change-Id: I9ce990558f178f1571cfac9760dd333b5b40174f
Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>