David [Fri, 17 Mar 2017 22:09:09 +0000 (23:09 +0100)]
Fix minor issues regarding checkstyle
- Make BulkOMaticUtils a final class since its constructor is private.
- Parenthesis not needed
Change-Id: I2e0a58dfe204aec89ab49cb3fbd870229932c317
Signed-off-by: David <david.suarez.fuentes@ericsson.com>
Shuva Jyoti Kar [Mon, 20 Mar 2017 03:32:43 +0000 (03:32 +0000)]
Merge "Bug 7915 - Zero flows populated in all switches when connected to Leader Node"
Anil Vishnoi [Tue, 13 Dec 2016 03:57:05 +0000 (19:57 -0800)]
Bug 7349 - Flow ID not updated in operational after removing and adding a flow with same match
Change-Id: Ie370cf017bd8642c52f1469cc0b01b10fa38842d
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit
66c19f4bbb00399056dce40db7d3312375558bc1)
Anil Vishnoi [Thu, 16 Mar 2017 07:10:20 +0000 (00:10 -0700)]
Bug 7915 - Zero flows populated in all switches when connected to Leader Node
Change-Id: I8e8df5d6cb51c114b23f8690d3d6f01fef37f81b
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Jozef Bacigal [Thu, 16 Mar 2017 13:38:29 +0000 (13:38 +0000)]
Merge "Add arbitrary mask for nxm-reg"
Jozef Bacigal [Thu, 16 Mar 2017 13:37:06 +0000 (13:37 +0000)]
Merge "Fix statistics race condition on big flows"
Shuva Jyoti Kar [Thu, 16 Mar 2017 10:20:38 +0000 (10:20 +0000)]
Merge "Fix checkstyle warnings"
Shuva Jyoti Kar [Thu, 16 Mar 2017 10:20:31 +0000 (10:20 +0000)]
Merge "Fix checkstyle warnings"
Shuva Jyoti Kar [Thu, 16 Mar 2017 10:20:18 +0000 (10:20 +0000)]
Merge "Fix checkstyle warnings"
Shuva Jyoti Kar [Thu, 16 Mar 2017 10:17:41 +0000 (10:17 +0000)]
Merge "Remove drop-test feature from He design"
Shigeru Yasuda [Fri, 24 Jun 2016 13:10:07 +0000 (22:10 +0900)]
Bug 6110: Fixed bugs in statistics manager due to race condition.
* Stats notification listener needs to wait for the XID to be cached.
* Enqueue DS operation after all notifications are received.
Change-Id: I42ac315a65be1a1f02152fbd9ea9510bee586eb3
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
Abhijit Kumbhare [Wed, 15 Mar 2017 22:14:34 +0000 (22:14 +0000)]
Merge "Bug 7916: MatchConvertorImpl detailed IllegalStateException instead NPE"
Jozef Bacigal [Thu, 2 Mar 2017 13:24:46 +0000 (14:24 +0100)]
Fix checkstyle warnings
- org.opendaylight.openflowplugin.api.openflow.lifecycle
Change-Id: Iaa95b9c49441adc943a4c510e1583ac92146ee63
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Thu, 9 Mar 2017 09:24:14 +0000 (10:24 +0100)]
Remove drop-test feature from He design
Change-Id: I3c9ffc1aa0d105601ceaa427f2bc043324d8e296
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Thu, 2 Mar 2017 13:17:58 +0000 (14:17 +0100)]
Fix checkstyle warnings
- org.opendylight.openflowplugin.api.openflow.device
Change-Id: I775fd56bee4a1d80b2c2ace37f32836c993c5b97
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Tomas Slusny [Tue, 14 Mar 2017 09:27:48 +0000 (10:27 +0100)]
Add arbitrary mask for nxm-reg
According to specification, nxm-reg should be fully maskable
using arbitrary mask.
Resolves: bug 7897
Change-Id: I912017e6bd33813ac65ed5799c8bfa7ebf217889
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Jozef Bacigal [Wed, 1 Mar 2017 11:07:46 +0000 (12:07 +0100)]
Fix checkstyle warnings
- org.opendaylight.openflowplugin.api.openflow.connection
Change-Id: Ib061f1fd2dbb6da1cd24e23e1846d79e41026f3e
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Tomas Slusny [Fri, 24 Feb 2017 08:40:45 +0000 (09:40 +0100)]
Fix statistics race condition on big flows
Fix race condition that happens when we receive statistics faster
than flow is deleted from device, but we already sent request for
deletion and removed this flow from device registy.
Resolves: bug 6917
Change-Id: I95510fbc5d507eaffc34d58b3b7743e60911b541
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Jozef Bacigal [Wed, 1 Mar 2017 10:40:21 +0000 (11:40 +0100)]
Fix checkstyle warnings
- org.opendaylight.openflowplugin.api
Change-Id: I7b24e696af69505163bbd515ffa7db3b32ca497d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Fri, 10 Mar 2017 09:09:37 +0000 (09:09 +0000)]
Merge "Remove old drop-test module and files."
Jozef Bacigal [Fri, 10 Mar 2017 09:09:17 +0000 (09:09 +0000)]
Merge "Implement core API's methods."
Jozef Bacigal [Fri, 10 Mar 2017 09:09:09 +0000 (09:09 +0000)]
Merge "Define new API for context chain and context chain holder."
Jozef Bacigal [Fri, 10 Mar 2017 09:08:40 +0000 (09:08 +0000)]
Merge "Add module and feature for ONF extensions"
Anil Vishnoi [Tue, 7 Mar 2017 02:49:23 +0000 (18:49 -0800)]
Bug 7910 - Flow with ethernet mask (ff:ff:ff:ff:ff:ff),
get stored under alien-id in operational data store
Custom comparator for ethernet mask address is modified
to take care of this issue.
Change-Id: I0c5c1dbd61dd1a3fb83d3b16695553649c4ecf8f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Jozef Bacigal [Thu, 9 Mar 2017 17:04:28 +0000 (17:04 +0000)]
Merge "Add configuration knob for Single Layer"
Jozef Bacigal [Thu, 9 Mar 2017 17:00:08 +0000 (17:00 +0000)]
Merge "Turn Single Layer Serialization on by default"
Jozef Bacigal [Mon, 21 Nov 2016 11:30:37 +0000 (12:30 +0100)]
Implement core API's methods.
Change-Id: I3005c82e2cdea0cf719c5f500041ad3ad9ce4480
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Mon, 14 Nov 2016 09:23:53 +0000 (10:23 +0100)]
Define new API for context chain and context chain holder.
Change-Id: I1ad94bab7b5ff3d98dee939420b2f28d3eb36261
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Wed, 8 Mar 2017 17:15:40 +0000 (18:15 +0100)]
Revert "Use Karaf 4 for the Karaf distribution"
This reverts commit
c14a28f3a6f7f095d597fb7354cbc317b3c3100a.
Change-Id: I6f9c29023380ff7942ebdb3e903d6cfbc2071f6d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Michael Vorburger [Wed, 8 Mar 2017 02:54:43 +0000 (03:54 +0100)]
Bug 7916: MatchConvertorImpl detailed IllegalStateException instead NPE
This won't fix bug 7916, but perhaps it can help understand the root
cause of it, for a future follow-up actual fix.
Change-Id: I3a006bdc38fec1f2b194a6d9d637c920ddd6b742
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Tomas Slusny [Thu, 2 Mar 2017 16:50:50 +0000 (17:50 +0100)]
Add configuration knob for Single Layer
Add configuration knob to openflowplugin.cfg that will enable/disable
use-single-layer-serialization.
Change-Id: I9473a80495060aaa252757ba4d0b1450d252b08e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 28 Feb 2017 12:07:02 +0000 (13:07 +0100)]
Turn Single Layer Serialization on by default
Change leaf user-single-layer-serialization to "true" to turn
Single Layer Serialization on by default.
More informations about this change can be found
here: http://ow.ly/NNJu309qSRv.
Change-Id: I00846d0776a64c160e8fff122b30cfc620eb79b0
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Jakub Toth [Tue, 7 Mar 2017 12:04:46 +0000 (13:04 +0100)]
BUG-6859 - Binding generator v1 refactoring
Fix compilation failure after binding generator move.
Change-Id: I93a34904054aff4f9f0766ea35a25731adb60e3a
Signed-off-by: Jakub Toth <jatoth@cisco.com>
Jozef Bacigal [Mon, 23 Jan 2017 13:22:59 +0000 (14:22 +0100)]
Add module and feature for ONF extensions
Change-Id: I5d983bfa5f01b98efd68d8094e519fa50905dce7
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Mon, 6 Mar 2017 09:08:21 +0000 (10:08 +0100)]
Update .gitingore file
- netbeans local configuration file.
Change-Id: I5b97eb0f926e6a3b65781c934954319daa3877b1
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Stephen Kitt [Tue, 21 Feb 2017 10:47:40 +0000 (11:47 +0100)]
Use Karaf 4 for the Karaf distribution
Change-Id: I6d8ce21866b4ffd5c25590076ad1042dba4e7346
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Jozef Bacigal [Tue, 31 Jan 2017 14:54:56 +0000 (15:54 +0100)]
Add Karaf 4 features.
Change-Id: I31781439d7b5cba76f6e6a52109b13b2546ee924
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Robert Varga <rovarga@cisco.com>
Abhijit Kumbhare [Tue, 28 Feb 2017 20:22:53 +0000 (20:22 +0000)]
Merge "Replace odl-dlux-core with odl-dluxapps-topology"
Jozef Bacigal [Tue, 28 Feb 2017 18:42:22 +0000 (18:42 +0000)]
Merge "Convert Functions to private methods"
Jozef Bacigal [Mon, 27 Feb 2017 16:56:15 +0000 (16:56 +0000)]
Merge "Fix some FRM checkstyle violations"
Jozef Bacigal [Mon, 27 Feb 2017 16:52:45 +0000 (16:52 +0000)]
Merge "Fix comparison of match extensions"
Robert Varga [Mon, 27 Feb 2017 13:27:07 +0000 (14:27 +0100)]
Convert Functions to private methods
Using method handles allows us to eliminate explicit dependency
on Function and simplify declaration.
Change-Id: I8317fd3f37c569612b8d865a39871eb09dd30c81
Signed-off-by: Robert Varga <rovarga@cisco.com>
Robert Varga [Wed, 15 Feb 2017 09:28:26 +0000 (10:28 +0100)]
Fix some FRM checkstyle violations
- whitespace around generic arguments
- unneeded public modifiers
- line length
Change-Id: I3febf13cf3f01e043f2406b20b57f9e37a6103b4
Signed-off-by: Robert Varga <rovarga@cisco.com>
Tomas Slusny [Tue, 21 Feb 2017 15:04:57 +0000 (16:04 +0100)]
Fix connection closing on switch IDLE state
When device is disconnected, and connection is not equal primary
connection context, do not shut down all services and just remove
this connection context.
Resolves: bug 7689
Change-Id: I3c6149a44f266b423fdc4f6396022ad4c650e872
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Jon Castro [Mon, 6 Feb 2017 23:35:13 +0000 (10:35 +1100)]
Bug 7736 - Forwarding Rules application cluster singleton id should not use the same cluster singleton id as the openflow switch singleton connection handler
Change-Id: Ie82b2d7d23444927375ed5b63f5e765f3277ed00
Signed-off-by: Jon Castro <castro.jon@gmail.com>
Tomas Slusny [Tue, 14 Feb 2017 09:01:58 +0000 (10:01 +0100)]
Fix comparison of match extensions
Add comparison between match extension lists to MatchComparatorFactory
to treat matches with different extensions as unique.
Change-Id: Ia43322d5e3ca3ee62e8313fd1286a6a9c68f7423
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Jozef Bacigal [Wed, 22 Feb 2017 16:17:56 +0000 (16:17 +0000)]
Merge changes from topic 'ofj-models-to-ofp-models'
* changes:
Add single layer deserialization support
Fix errors in serializers and deserializers
Add multipart reply deserializers unit tests
Update AbstractService to support isComplete
Add multipart reply deserializers
Add PortMessageDeserializer
Add MeterMessageDeserializer
Add GroupMessageDeserializer
Add FlowMessageDeserializer
Add Instruction deserializers
Add InstructionDeserializerInjector
Add Action deserializers
Add ActionDeserializerInjector
Add Match entry deserializers
Add MatchDeserializerInjector
Add MessageDeserializerInjector
Add PortMessageService
Add GroupMessageService
Add MeterMessageService
Add FlowMessageService
Add AbstractMessageService
Add configuration parameter for single layer
deathbeam [Mon, 30 Jan 2017 18:16:45 +0000 (19:16 +0100)]
Add single layer deserialization support
- Update MultiMsgCollector to support multiple reply types
- Update DeviceContext to support mulyiple reply types
- Update AbstractService and MultipartService* implementations
to support multiple reply types
- Update GroupDirectStatisticsService
- Update FlowDirectStatisticsService
- Update MeterDirectStatisticsService
- Update PortDirectStatisticsService
- Update QueueDirectStatisticsService
- Update SalExperimenterMpMessageService
- Update SalTableService
- Change giant StatisticsGatheringUtils to separate statistics
writers and add support for writing both deserialization paths
- Refactor structure of services because of a lot of new classes
added, it became mess
- Create new initialization chain for device that supports both
deserialization paths
Resolves: bug 7141
Change-Id: I21b9b894d89db220ccf20ebb031fcb9e039b184a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 8 Feb 2017 19:14:02 +0000 (20:14 +0100)]
Fix errors in serializers and deserializers
Fixed issues with deserializing missing multipart table features,
flows and unsigned longs.
Fixed issues when serializing Ipv4 and Ipv6 source and destination
match entries.
Added additional safety checks to match entry deserializers.
See also: bug 7139, bug 7141
Change-Id: Ie93a486aa847fdb39f01d24768f259cc4661fcad
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Aswin Suryanarayanan [Sat, 18 Feb 2017 08:19:10 +0000 (13:49 +0530)]
Bug 7811 - Conntrack Action deserialization - IndexOutOfBoundsException
1)Added check for the presence of NAT Action during deserialization
2)Added UT to check the presence of NAT Action.
Change-Id: I87ec08f81b1f6372690e5e8d0dbb7ae1ab3e7c74
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Jozef Bacigal [Thu, 16 Feb 2017 11:15:03 +0000 (12:15 +0100)]
Remove old drop-test module and files.
- for drop test using drop-test-karaf module
Change-Id: I3fc3f490bff2d451aa3a0f2773be32e0715f24ca
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
miroslav.macko [Tue, 24 Jan 2017 15:00:29 +0000 (16:00 +0100)]
Add multipart reply deserializers unit tests
Add unit tests
See also: bug 7140
Change-Id: Ice9189919efca286c06ea1cced0265dd7e43429d
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
deathbeam [Mon, 30 Jan 2017 12:21:41 +0000 (13:21 +0100)]
Update AbstractService to support isComplete
Overload AbstractService#handleServiceCall to also have
parameter with isComplete function.
See also: bug 7141
Change-Id: I51ee1032ac7b443cf8b442ef66eca42380e938b0
Signed-off-by: deathbeam <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 17 Jan 2017 15:07:05 +0000 (16:07 +0100)]
Add multipart reply deserializers
- Add yang models for multipart reply
- Add MultipartDeserializerInjector that will inject multipart reply
deserializers to OpenflowJava
- Add these mutlipart reply deserializers:
* Flow aggregate stats
* Flow stats
* Flow table stats
* Group desc
* Group features
* Group stats
* Meter config
* Meter features
* Meter stats
* Port stats
* Queue stats
* Table features
* Experimenter
See also: bug 7140
Change-Id: I0844c54435b3b029982e7cd6e6b9c2cb01138c67
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Mon, 12 Dec 2016 16:28:24 +0000 (17:28 +0100)]
Add PortMessageDeserializer
Add PortMessageDeserializer Inject PortMessageDeserializer into OFJ.
This deserializer will deserialize raw bytes into PortMessage.
See also: bug 7140
Change-Id: Ia3fd168d424138e81eb9f24ede232d954503a40c
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Mon, 12 Dec 2016 15:16:49 +0000 (16:16 +0100)]
Add MeterMessageDeserializer
Inject MeterMessageDeserializer into OFJ. This deserializer will
deserialize raw bytes into MeterMessage.
See also: bug 7140
Change-Id: Ia66378d4ad01a6b273308f6d6e25e86f639f90a5
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 4 Jan 2017 12:41:58 +0000 (13:41 +0100)]
Add GroupMessageDeserializer
Inject GroupMessageDeserializer into OFJ.
This deserializer will deserialize raw bytes into GroupMessage.
See also: bug 7140
Change-Id: I07e0ec7601155eb4138b598c9c4f39dfd1e31d03
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Thu, 5 Jan 2017 15:54:46 +0000 (16:54 +0100)]
Add FlowMessageDeserializer
Inject FlowMessageDeserializer into OFJ.
This deserializer will deserialize raw bytes into FlowMessage.
Change-Id: Ic098a80a6d0b97fbf8ab3761d15a09de3d46defd
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Thu, 5 Jan 2017 13:15:43 +0000 (14:15 +0100)]
Add Instruction deserializers
Inject instruction deserializers into DeserializerRegistry.
These deserializers will deserialize these instructions:
- GoToTable
- WriteMetadata
- WriteActions
- ApplyActions
- ClearActions
- Meter
See also: bug 7140
Change-Id: Ie38961458157cac46a4bdeba18c0e6c9020d3671
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Thu, 5 Jan 2017 09:02:30 +0000 (10:02 +0100)]
Add InstructionDeserializerInjector
Add injector where all instruction deserializers will be
registered and call it from general DeserializerInjector.
Create AbstractInstructionDeserializer and
AbstractActionInstructionDeserializer that will define
structure for all instruction deserializers.
See also: bug 7140
Change-Id: Icf76321ed0e51c004d003be585f40cfe4710b1b3
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Mon, 2 Jan 2017 09:01:30 +0000 (10:01 +0100)]
Add Action deserializers
Inject action deserializers into DeserializerRegistry.
These deserializers will deserialize these actions:
- CopyTtlIn
- CopyTtlOut
- DecMplsTtl
- DecNwTtl
- Group
- Output
- PopMpls
- PopPbb
- PopVlan
- PushMpls
- PushPbb
- PushVlan
- SetField
- SetMplsTtl
- SetNwTtl
- SetQueue
See also: bug 7140
Change-Id: I0f36092ca99de0ca51719c3c2099ebe1d904a72f
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Thu, 22 Dec 2016 12:35:26 +0000 (13:35 +0100)]
Add ActionDeserializerInjector
Add injector where all action deserializers will be registered and call it
from general DeserializerInjector.
Add AbstractActionDeserializer that will define structure for all action
deserializers.
Add ActionUtil#readAction that will try to deserialize action or use
extensions.
Add MessageCodeExperimenterKey for registering action deserializers with
experimenters.
See also: bug 7140
Change-Id: I1a1eaec5c9aa128f2c2d5e2a31b37086f2a56734
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 14 Dec 2016 08:23:19 +0000 (09:23 +0100)]
Add Match entry deserializers
Inject match entry deserializers into MatchDeserializer.
These deserializers will deserialize these match entries:
- ArpOp
- ArpSha
- ArpSpa
- ArpTha
- ArpTpa
- InPort
- InPhyPort
- EthSrc
- EthDst
- EthType
- Metadata
- VlanPcp
- VlanVid
- IpDscp
- IpEcn
- IpProto
- TcpSrc
- TcpDst
- UdpSrc
- UdpDst
- SctpSrc
- SctpDst
- Icmpv4Code
- Icmpv4Type
- Icmpv6Code
- Icmpv6Type
- Ipv4Source
- Ipv4Destination
- Ipv6Source
- Ipv6Destination
- Ipv6ExtHeader
- Ipv6Flabel
- Ipv6NdSll
- Ipv6NdTll
- Ipv6NdTarget
- MplsLabel
- MplsBos
- MplsTc
- PbbIsid
- TunnelId
- TcpFlags
Change-Id: Ic584d4610f2b2230542e449d3fae3f4affcc2756
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 13 Dec 2016 10:00:48 +0000 (11:00 +0100)]
Add MatchDeserializerInjector
Add injector where all match and match entry deserializers will be
registered and call it from general DeserializerInjector.
Add MatchEntryDeserializer and MatchDeserializer to be able to actually
create match entry deserializers and deserialize match body.
Add new method to MatchExtensionHelper that can inject extension
augmentations one by one.
See also: bug 7140
Change-Id: Id15419539792cb4a21a5862be2dd423be34ab117
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Mon, 12 Dec 2016 13:48:13 +0000 (14:48 +0100)]
Add MessageDeserializerInjector
Add injector where all message deserializers will be registered and call
it from general DeserializerInjector
Add unit tests for MessageDeserializerInjector
See also: bug 7140
Change-Id: I28e2795148597a1c15cb52dec46728a4324a4d0a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 24 Jan 2017 12:55:04 +0000 (13:55 +0100)]
Add PortMessageService
Add PortMessageService that will convert Port to PortMessage and send
this meesage to serialization.
Update SalPortServiceImpl to use PortMessageService if supported and
enabled.
See also: bug 7139
Change-Id: I73a452770a745c5c7a7debe95cbcf7e04afebb89
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 24 Jan 2017 09:31:56 +0000 (10:31 +0100)]
Add GroupMessageService
Add GroupMessageService that will convert Group to GroupMessage and
send this meesage to serialization.
Update SalGroupServiceImpl to use GroupMessageService if supported
and enabled.
See also: bug 7139
Change-Id: I07c2f8949f4b89d0cb921d82dfaed3159ca7da8e
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 24 Jan 2017 09:52:45 +0000 (10:52 +0100)]
Add MeterMessageService
Add MeterMessageService that will convert Meter to MeterMessage
and send this meesage to serialization.
Update SalMeterServiceImpl to use MeterMessageService if supported and enabled.
See also: bug 7139
Change-Id: I158ddc58deb62394c50bc2e6d195abaa3f283532
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
miroslav.macko [Fri, 20 Jan 2017 13:00:38 +0000 (14:00 +0100)]
Add FlowMessageService
See also: bug 7139
Change-Id: Idb4f5b40ce8f033deee209eec1596df2b8ff7d10
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
miroslav.macko [Fri, 20 Jan 2017 07:51:38 +0000 (08:51 +0100)]
Add AbstractMessageService
Add AbstractMessageService that will define structure for
all other message services.
Modify AbstractService#handleServiceCall not to be final.
See also: bug 7139
Change-Id: I0bcfe8c0a5ba8ad572427240b29d7522ce0c5cf5
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Tomas Slusny [Mon, 23 Jan 2017 12:32:51 +0000 (13:32 +0100)]
Add configuration parameter for single layer
- Add use-single-layer-serialization to openflow-provider-config.yang
- Pass this new configuration parameter to DeviceContext
See also: bug 7139
Change-Id: I00557cd1c0c637fcf83c42e47808a5bf96a57a1b
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Michal Rehak [Mon, 6 Feb 2017 13:12:45 +0000 (14:12 +0100)]
Remove redundand case in nicira-action.yang
- created probably as unattended copy/paste
- equal cases are forbidden by yang spec and next version
of yang parser will check this rule
See: BUG-7424
Change-Id: I83c551dc5dd613163c3f8a47a443081d71f3c328
Signed-off-by: Michal Rehak <mirehak@cisco.com>
Jozef Bacigal [Mon, 13 Feb 2017 08:40:03 +0000 (09:40 +0100)]
Fix failing test.
Yangtools fixed their issue and merged it.
Change-Id: I47c3397145a53c2d2db72093b153a40da5cee571
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Anil Vishnoi [Fri, 10 Feb 2017 18:30:27 +0000 (18:30 +0000)]
Merge "Bug 7764 - Do no throw warning on explicit task cancellation"
Anil Vishnoi [Thu, 9 Feb 2017 08:12:20 +0000 (00:12 -0800)]
Bug 7764 - Do no throw warning on explicit task cancellation
Change-Id: I9477823ffd1b4e9b28484ae8cde166a2d03f5a9f
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
(cherry picked from commit
d813c7f656b4dd6f460b66a2fa303a448f6a4e07)
Aswin Suryanarayanan [Fri, 23 Dec 2016 08:50:25 +0000 (14:20 +0530)]
NAT Action added in Nicira Extension
Nat action is added as a port of conntrack action, it is an optional
action field.
Change-Id: Iec97d9d859e0e2060abc3a31ceb929c1b1e43b4a
Signed-off-by: Aswin Suryanarayanan <asuryana@redhat.com>
Anil Vishnoi [Thu, 9 Feb 2017 23:54:32 +0000 (15:54 -0800)]
Bug 7769 - Master branch is broken because of the test failures.
Change-Id: I0f3d787011d388947b10f4fe94ed20e4a270293c
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Jozef Bacigal [Thu, 9 Feb 2017 09:30:11 +0000 (09:30 +0000)]
Merge "Fix comparison between port numbers in match"
miroslav.macko [Thu, 2 Feb 2017 13:33:59 +0000 (14:33 +0100)]
Fix PacketInV10TranslatorTest
Use object instead hardcoded string
Change-Id: I6408a927e0c6c186e78884a9851f1dbea56456aa
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Abhijit Kumbhare [Fri, 3 Feb 2017 19:26:33 +0000 (19:26 +0000)]
Merge "Replace mockito-all by mockito-core (see Bug 7662)"
Jon Castro [Thu, 2 Feb 2017 01:57:49 +0000 (12:57 +1100)]
Bug 7499 - ensure statistics scheduler does not die and keep trying while the controller keeps the ownership of the device
Change-Id: Ia5923f200b6c7a8888d4e3c96fdf8358603d4ac9
Signed-off-by: Jon Castro <castro.jon@gmail.com>
michal rehak [Tue, 31 Jan 2017 12:36:25 +0000 (12:36 +0000)]
Merge "BUG-7608: use blueprint action-provider/action-service"
eeiillu [Thu, 12 Jan 2017 04:59:18 +0000 (10:29 +0530)]
Bug 7453 - FlowRemoved doesn't have Removed Reason Information
On flow being removed from switch on idle timeout, onFlowRemoved() api was
invoked and while conversion reason was not being converted and added, so
the application using API onFlowRemoved() was not getting the reason for
flow being removed from the switch.
This change will provide the reason invoking onFlowRemoved() API
Change-Id: Id8b957b5b2cc581215fb84b8ca84d1d9a04e5619
Signed-off-by: eeiillu <suja.t@ericsson.com>
Robert Varga [Fri, 27 Jan 2017 12:47:21 +0000 (13:47 +0100)]
BUG-7608: use blueprint action-provider/action-service
The contract for RPC services containing Routed RPCs (RFC7950
actions) with respect to routing is different from normal RPCs.
In order to properly express dependencies, it needs a concept
of a promise to register instantiations (action-provider)
and the dependency on that promise (action-service). Use these
concepts instead of realying on rpc-service which happens to
work due to sal-remoterpc-connector (incorrecty) providing
the equivalent of action-provider for all Routed RPCs defined
in the local SchemaContext.
Change-Id: Iaad5b5b0b4384a2e7d6302bd7de5bfa00c24d89b
Signed-off-by: Robert Varga <rovarga@cisco.com>
Michael Vorburger [Mon, 23 Jan 2017 20:07:10 +0000 (21:07 +0100)]
Replace FindBugs :jsr305 by full :annotation (Bug 7663)
Change-Id: I05b3a0fd993c246617808d33d743d04976e0286c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Tomas Slusny [Thu, 26 Jan 2017 12:36:36 +0000 (13:36 +0100)]
Add isComplete callback to commitEntry
Implement commitEntry with isComplete callback
from https://git.opendaylight.org/gerrit/#/c/51064/ to
OpenflowPlugin OutboundQueueProviderImpl.
See also: bug 7140
Change-Id: I7c4224e029f175a54b46246e1d409356f2447b16
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 24 Jan 2017 16:57:13 +0000 (17:57 +0100)]
Fix comparison between port numbers in match
Properly extract port number from NodeConnectorId when comparing
port numbers, as NodeConnectorId is basically string, and sometimes
when going back from device, it do not contains datapath id
or prefix (for example in new serialization/deserialization path).
See also: bug 7139
Change-Id: I89999b71dba2e834ea1f37f42b5cddf1082bdd12
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Michael Vorburger [Mon, 23 Jan 2017 18:58:43 +0000 (19:58 +0100)]
Replace mockito-all by mockito-core (see Bug 7662)
Change-Id: Iceeb2f1082a6ed3be04e99e17b2e6c3c3c4ebb31
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
miroslav.macko [Mon, 16 Jan 2017 09:00:06 +0000 (10:00 +0100)]
Fix Direct statistics RPC - actions part
Handle actions for direct statistics.
See also: bug 7232
Change-Id: Ib29f9a760bf123bac1745ea2fe8bc7e6bd3de73c
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
D Arunprakash [Fri, 13 Jan 2017 10:50:49 +0000 (16:20 +0530)]
RPC opendaylight-direct-statistics:get-flow-statistics not taking nicira
extension match
This is just a draft review, still testing in progress
Added new MatchPath RPCFLOWSSTATISTICS_FLOWANDSTATISTICSMAPLIST_MATCH.
This will differentiate between the augment model between notification and
rpc.
Change-Id: If80ff86522ba406577b15c4a3d8a30ee364001db
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
Shuva Jyoti Kar [Wed, 18 Jan 2017 08:38:51 +0000 (08:38 +0000)]
Merge "Bug7485 Make statistics poller parameters configurable."
Robert Varga [Fri, 13 Jan 2017 09:03:20 +0000 (10:03 +0100)]
BUG-5222: do not pull in odl-mdsal-xsql
xsql is deprecated, buggy and never was meant for production
use. Do not pull it in.
Change-Id: Id534d17150a940ffad7f7c757146bca46fa1a923
Signed-off-by: Robert Varga <rovarga@cisco.com>
Sai MarapaReddy [Fri, 6 Jan 2017 00:52:06 +0000 (16:52 -0800)]
Bug7485 Make statistics poller parameters configurable.
Currently the the time intervals used for polling the statistics
are harcoded. (basicTimerDelay, maximumTimerDelay).
This patch helps makes parameters more configurable by defining
current default values in yang. Using BluePrint extensions user
can now configure the parameters without changing the source code.
Change-Id: Id06cb1e569b0d3c653c142bf5f5af5a24fd6e3ea
Signed-off-by: Sai Marapareddy <sai.marapareddy@gmail.com>
Shuva Jyoti Kar [Wed, 11 Jan 2017 15:29:09 +0000 (15:29 +0000)]
Merge "BUG 7502: Switch throws an error message for the action type"
Daniel Malachovsky [Tue, 10 Jan 2017 17:16:35 +0000 (18:16 +0100)]
Replace odl-dlux-core with odl-dluxapps-topology
Due DLUX split, we removed odl-dlux-topology feature from odl-dlux-core.
Because your project have dependency on odl-dlux-core, i assume, you used
Topology app.
More info:
https://wiki.opendaylight.org/view/Weather#Rename_some_odl-dlux-.2A_features_to_odl-dluxapps-.2A
Change-Id: I037a9297f1cad0b9ee961c6e12876e9f463e061a
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>
Shuva Jyoti Kar [Tue, 10 Jan 2017 15:33:15 +0000 (15:33 +0000)]
Merge "Remove use of triemap"
Robert Varga [Tue, 10 Jan 2017 12:48:41 +0000 (13:48 +0100)]
Remove use of triemap
Triemap is not used anywhere, do not declare it as
a dependency.
Change-Id: I296374e2be486d6e77a74d709ce2bf021cab9088
Signed-off-by: Robert Varga <rovarga@cisco.com>
Slava Radune [Tue, 20 Dec 2016 14:55:21 +0000 (16:55 +0200)]
Bug 7071: adding support for fin-timeout
Add support for serialization/deseralization and encoding/decoding of fin-timeout action.
This action currently is created using the learn action, so we nead only deseralization/decoding way (when flow statistics is sent to odl),
but this patch suppors also the other way of insalling flows from odl, in case it will be needed in the future.
Change-Id: I061100fab61e8d3398fddde98e76493a5afef468
Signed-off-by: Slava Radune <slava.radune@hpe.com>
epgoraj [Tue, 10 Jan 2017 05:53:13 +0000 (11:23 +0530)]
BUG 7502: Switch throws an error message for the action type
(move:NXM_NX_REG1[0..19]->OXM_OF_MPLS_LABEL[] )
error message: OFPAT_BAD_MATCH due to wrong OXMClassCode.
code-review contains the fix to address correct OXMclasscode for the matchtype.
Change-Id: I113e76111b4ca84e2a7c1f0f443af7330f33328f
Signed-off-by: epgoraj <p.govinda.rajulu@ericsson.com>
Abhijit Kumbhare [Mon, 9 Jan 2017 21:57:26 +0000 (21:57 +0000)]
Merge "Bug 7481 - Flows with nicira actions get corrupted after cluster failure"