Add method to register listener for unknown msg

Add method to ConnectionAdapter that will allow to register listener for
unknown (alien) messages received from switch.

See also: bug 8772

Change-Id: I3c4e48d0ddfd0a1220850bec5f75aa84e0e662c6
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Bump to odlparent 2.0.0

Change-Id: I9ec83c0f0e28f22185ffa0c50a055a6daa951fa6
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Copy initial configuration file in Karaf 4

Fix copying of initial configuration file in Karaf 4. Problem was
that karaf .xml file that was supposed to handle this was incorrectly
located at `src/main/features/features.xml` but was supposed to be at
`src/main/feature/feature.xml` to work properly in Karaf 4.

Resolves: bug 8692

Change-Id: I9923a2b2598e05e95bb68a01cde5a8894679e534
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Migrate to odlparent 1.9.0

Change-Id: I95874664e56e30c6c4da2cd575dc7015182a2e2f
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Check for transport protocol confi

For some reason the TransportProtocol config
has a null value at the following jenkins job

https://jenkins.opendaylight.org/releng/job/aaa-distribution-check-nitrogen/139/console

    - flipped equals, now null is on slow path

Change-Id: I38c2bb0676d4553452f9ae630554b6c99d445ef4
Signed-off-by: melserngawy <m.elserngawy@gmail.com>
Signed-off-by: Michal Rehak <michal.rehak@pantheon.tech>

Migrate to odlparent 1.8.0-Carbon

Per request of odlparent project we are downgrading all Nitrogen
projects to use the released odlparent 1.8.0-Carbon to allow for the
odlparent project to start performing semver style releases.

Change-Id: I052f5a03008c82346548dd43983f539d5de4c274
Jira: RELENG-159
RT: 41406
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Add missing configuration to build-helper-maven-plugin "

Add missing configuration to build-helper-maven-plugin

This makes openflowjava work e.g. in Eclipse IDE out-of-the-box (using
https://github.com/vorburger/opendaylight-eclipse-setup configuration),
without any red.  Without this, the paths with the generated code are
not automatically source folder in IDE, so not on classpath, so there's
red errors.

This is how all other projects do it; I just copy/pasted from elsewhere.

PS: In an ideal world, this should be inherited from some parent POM
e.g. in controller, but let's at least already do an ad-hoc fix here.

Change-Id: Ib0b8258de329345422d044f5fea4e9981b67da93
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Add missing <name> fields for pom.xml files

This is used by autorelease scripts to automatically parse which project
is failing a build and report to the mailing list automatically. We need
names in the format:

    ODL :: <groupId> :: <artifactId>

This patch formats in the same format as found in the startup archetypes
patch found here: https://git.opendaylight.org/gerrit/52522

Change-Id: If1f9c01f9c3a0cf3a9672e1d795a2c1d55b38a25
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Merge "Add target-ide/ to .gitignore"

Merge "Bug 7182 related: Remove M2E lifecycle mapping"

Add target-ide/ to .gitignore

Change-Id: Ic58b9795c1388cb0bf7e783e3e2c69340d1a70be
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Increase startup and shutdown timeouts for udpHandler test

    Startup timeout: 2s -> 10s
    Shutdown timeout: inf. -> 10s
    + minor cosmetic changes (IDE warnings cleanup)

Change-Id: I95b604641d7dbec73ab9770640f50a71a310127f
Signed-off-by: Michal Rehak <michal.rehak@pantheon.tech>

Bump versions by x.(y+1).z for next dev cycle

Change-Id: I39929fd1d32cf53c045db31393eb8519465f6aeb
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

BUG-6859 - Binding generator v1 refactoring

Change-Id: If8f5ace6ba7734ae866d7ed5c096ea24bd280d1a
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bug 7182 related: Remove M2E lifecycle mapping

These should never be in individual projects anymore now, we handle this
centrally, either in odlparent, or
https://github.com/vorburger/opendaylight-eclipse-setup, or by
appropriate lifecycle-mapping-metadata.xml in a Maven plugin (that's
what Bug 7182 does for the yang-maven-plugin).

Change-Id: I9923116894c0fa2c2f45ab8dfbe25ff3ef7a0296
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Update OF header lenght

- Set OF header length base on the writer index

Change-Id: I48358e22a92ce400df8917a7e23a0a085b2b0a95
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>

Migrate to karaf4.

Change-Id: Idf1bccf8f4802871a24207c26d6624edeb55ce79
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Add isComplete callback to commitEntry

This callback will determine, if processed message
is completed or not.

Change-Id: I29a0faeaa1b3965a88e4d4516e19aee34bc5bf71
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Replace mockito-all by mockito-core (see Bug 7662)

Change-Id: Ic6cfde3f29b167040ae1588df7622f2505663b5e
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Remove duplicate controller.mdsal.version field

Change-Id: I5d85e1e2bb2871b517603fea947a82951fd7aa73
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge changes I34a6851c,I046e6272,I169f6b40,Iec24bf37,I89933b0c

* changes:
  Change Java version to 1.8
  Fix bug on ByteBufUtils method serializeList
  Scenarios in XML files
  Update utility to test device connections
  Add test device connections utility

Add methods for modifying deserializer mapping

Add new methods to SwitchConnectionProviderImpl that will allow
us to modify type to class mapping of deserializers for top level
messages (like FlowMod, PortMod) to be able to register deserializers
for custom top level types without needing changes in OpenflowJava.

Change-Id: I0f0d0f25a971660e6601069fc6f9f59e8206f2e7
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Remove bundle extension (de)serializers

 - moved to OFP extensions

https://git.opendaylight.org/gerrit/#/c/48896/
https://git.opendaylight.org/gerrit/#/c/48897

Change-Id: Ib2cba9a989daea9f85d7a53280bfcc5f4ad6a753
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Add methods that allows registering any serializer

Add new methods to SwitchConnectionProviderImpl that will allow us
to register any type of OFSerializer and OFDeserializer, so we will
be able to for example register serializers for OpenflowPlugin models
directly in OpenflowPlugin, without requiring changes in
OpenflowJava.
Change HeaderSerializer and HeaderDeserializer to require DataContainer
instead of DataObject.

See also: bug 7136

Change-Id: I002cb787a6db61d864e205d99fafa54692e220a6
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>

Change Java version to 1.8

Change-Id: I34a6851c162fe0f6bf9c900773b4dbb2452d97a7
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Fix bug on ByteBufUtils method serializeList

Also
- optimize imports
- add test to serializeList method

Change-Id: I046e62729676a476818463a5034217778b4295f7
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Scenarios in XML files

-prepared XSD file
-updated scenarioFactory
-added types for unmarshaling XML

-added new parameter to testing tool --xmlScenarioFile

03/15/2016
-renamed ScenarioType to Scenario
-renamed StepType to Step

Change-Id: I169f6b40e65e80533e6cc3a1978bb34f84164abc
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Update utility to test device connections

 * Helper class to defining command line parameters
 * Parameters:
 * --device-count           : number of devices connection to the controller
 * --controller-ip          : controller IP address
 * --ssl                    :
 * --threads                : number of thread shall be used for executor
 * --port
 * --timeout                : timeout in seconds for whole test
 * --scenarioTries          : number of tries of each step of scenario
 * --timeBetweenScenario    : time in milliseconds between tries of steps of scenario
 * --configurationName      : required parameter if using configuration load or configuration save
 * --configurationLoad
 * --configurationSave

3/9/16 - Checkstyle corrections

Change-Id: Iec24bf37bb6cce534ce87ca80585d0f53d0dd638
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Add test device connections utility

Change-Id: I89933b0ce057679e959cf2ef576fe10c8a1cc122
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>

Fix default/legacy openflow ports

Change-Id: I9718d09c389689ec5fc0a45a4a116aabab15410e
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Add bundle control and ONF experimenter error deserializers

 - created and registered deserializers for ONF_ET_BUNDLE_CONTROL
   message and ONF experimenter errors
 - added tests

Reference: ONF approved extension #230
Resolves: Bug 6806

Change-Id: Iaa584aae6ee2e9962f962803d363f3213d99fa2e
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Add bundle messages serializers

 - created and registered serializers for ONF_ET_BUNDLE_CONTROL
   and ONF_ET_BUNDLE_ADD_MESSAGE messages
 - updated Flow/Group/PortMod facories to use more general objects
 - registered additional serializers for inner messages
 - added tests

Reference: ONF approved extension #230
Resolves: Bug 6806

Change-Id: I1e98332eed24e18f17157e9a16ac8d246c72af2c
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Add yang models for bundle messages

 - augmented model of experimenter message for ONF_ET_BUNDLE_CONTROL
   and ONF_ET_BUNDLE_ADD_MESSAGE messages
 - added related defined types (error codes, flags, properties, ...)

Reference: ONF approved extension #230
Resolves: Bug 6806

Change-Id: I03761dc2630928623dbb55c55fd148c762b2489a
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Add methods for experimenter (de)serializer registration

 + update comments
 + remove unused imports

Change-Id: I87f432417a62cb5a58fe4e0103f9ee491e99017d
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Change BarrierReq/Res factories to version assignable

  - made BarrierRequest/BarrierReply factories version assignable
  - removed OF10 factories duplicates
  - registered new deserializers
  - update tests
  - fix typo

Resolves: Bug 4255

Change-Id: I248941b94ea6974679a27313cd6634baaa0fc034
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Change GetConfigReq/Res and SetConfig factories to version assignable

 - made SetConfig, GetConfigRequest and GetConfigReply factories
   version assignable
 - removed OF10 factories duplicates
 - registered new deserializers
 - update tests

Resolves: Bug 4255

Change-Id: Ib4809c58d62aa4608d88894d01c6d107d2fff90c
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Change EchoReq/Res factories to version assignable

 - made EchoRequest/EchoReply factories version assignable
 - removed OF10 factories duplicates
 - registered new deserializers
 - update tests

Resolves: Bug 4255

Change-Id: Ic3e67c4e4db7f1fedcdfc125784219d42a59054c
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Allow any hello mesage and extend hello support for v1.4, v1.5

 - accept any (also unsupported version) hello message for handshake and negotiation
 - added version assignable factory
 - made HelloFactory version assignable
 - register deserializers for hello message of v1.4, v 1.5
 - update logging
 - update tests

Resolves: Bug 6805, Bug 4255

Change-Id: I228108908ecfb2c3a7f8afa866790b7c193f046c
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Revert "Revert "Move the Openflow connections blueprint to OpenflowJava project""

This reverts commit 159164138dd0a64af5a3e2b96e1d7de5cad4b139.

Re-introducing the change

Change-Id: I8b9e4bd2220890d173dec7471eaf3a6e8b837721
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Revert "Move the Openflow connections blueprint to OpenflowJava project"

This reverts commit 4de7239667ec58eef9e2759e1501b9d1cf862cdd.

Change-Id: I45a74aa32abb1ac16ab403fb4cac8a39a307b00c
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Move the Openflow connections blueprint to OpenflowJava project

Moving the Openflow connections (defaultSwitchConnProvider and legacySwitchConnProvider)
blueprint configuration to OpenflowJava project as its data model config "openflow-switch-connection-provider-impl"
exist in OpenflowJava project

Change-Id: I21529b5f8e312d4da0d4a76ef0e6bf02a1e551c5
Signed-off-by: Mohamed El-Serngawy <melserngawy@inocybe.com>

Bug 6744 - the parameters of the function of registerMeterBandSerializer need to be more refined

Change-Id: I076906df05b26b407fe7209487b9181dec029d3a
Signed-off-by: yunyunhan <han.yunyun@zte.com.cn>
(cherry picked from commit 5e8b6a2c5218b2ea22343ec889ad7e7fb4112530)

Bug 6674 - the key of the serialization function registered by the vendor is not refinement enough

Change-Id: Ibe9c63f03850730c7d7346820d37786d6711f861
Signed-off-by: yunyunhan <han.yunyun@zte.com.cn>
(cherry picked from commit f93ddeaf5e4f25a0a47438786777d13c9b95e46d)

Bug 6646 Fix infinite reschedule of flush

 - sometimes (on disconnect) there can be still some unflushed segments
   but they are not able to be flushed if channel is not writable anymore
   and we get to infinite loop of flushing (but not writing)

Change-Id: I74cac21b4635e22f5b8d63f3a602f40796108059
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bug 6638 Failed entries marked as completed also counted as completed

Change-Id: Ia4a0f334d831313eb419b327cf9b8f5fa9a464b4
Signed-off-by: Andrej Leitner <andrej.leitner@pantheon.tech>

Bump versions by 0.1.0 for next dev cycle

Change-Id: I247e00fe7f3ff830760006711c6d61b8174cb4ba
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 5895 - Support of Ext109 openflow tcp flag matching

 - added openflow-approved-extensions.yang module
 - added (de)serializer for TCP flags match entry (ext-109)
 - also a reference material for adding other approved match entry extensions

Change-Id: I042d71e5a8b56f6b6460ef235442568d411533da
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>
Also-By: Anil Vishnoi <vishnoianil@gmail.com>

Add missing license headers

Change-Id: Ia8907b48549f18b627d74c45df38c1bbf2e4d720
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Prepare upgrade to Netty 4.1"

Upgrade ietf-{inet,yang}-types to 2013-07-15

Change-Id: If7f7f70aa0ca0cc5937d4969ef143d21e959879f
Signed-off-by: Lorand Jakab <lojakab@cisco.com>

Prepare upgrade to Netty 4.1

We need to widen the range of allowed versions for odl-netty.

Change-Id: I032697c714327a6ac0928da9f80a694e34d6fc76
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 5928 - Future error after OutboundQueueEntry.java failed

In case that OutboundQueueEntry has failed and after that is called
commit again on the same "entry" (completed entry) with callback, on this
callback was never called onFailure.

- try to store last OutboundQueueErorr
- if commit called on completed entry and it has a callback send onFailure to the callback
- added test for entire class

Change-Id: Ie347811ac6dc2c95d58ad49bbf2aa4d69033b33c
Signed-off-by: Jozef Bacigal <jbacigal@cisco.com>

Remove unused property

Change-Id: I219adf08931871495251d9bd0fb9100a40310bf0
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fixed netty & checkstyle failures

Change-Id: I5287bdcdcbc48513d45a9cd4593bf1df6c7be941
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Fix for the Bug 5637 : When closing OutboundQueue close() should go prior to finishShutdown()
When device gets disconnected with outstanding entries in queue segment,
it could occur that entry.commit() gets invoked between segment.failAll()
and OutboundQueueProvider.close(). This fix to prevent this scenario.

Change-Id: Id297e7d4f10c7e31a550f94ac7d39f43e1320de1
Signed-off-by: Abbas Pareedkunju <abbas.pareedkunju@tcs.com>

Deprecate SwitchConnectionProviderModule

Deprecate the SwitchConnectionProviderModule and corresponding yang.
The createInstance method was modified to obtain the corresponding
service instance created via blueprint from the OSGi registry using
a ServiceTracker.

Change-Id: Ie156466e08d4a5758ceae63f6c877f8210632cfb
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Add SwitchConnectionProviderFactory

Added a SwitchConnectionProviderFactory interface to the protocol SPI
bundle and a SwitchConnectionProviderFactoryImpl to the impl bundle to
create SwitchConnectionProvider instances given a SwitchConnectionConfig.
The SwitchConnectionProviderFactoryImpl is instantiated and advertised as an
OSGi service via blueprint. This allows clients to create new
SwitchConnectionProvider instances while hiding implementation details.
This is equivalent to creating instances via the config yang module.

The SwitchConnectionConfig and related classes are generated via a new
yang file that is equivalent to the config elements defined in the
openflow-switch-connection-provider-impl config yang. This new yang
model will be stored and retrieved via the data store.

Change-Id: I511c12644d5d54cd99c6b1afbf4c078a0cecee8e
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Bug 5173 - Prevent log flooding

Change-Id: I9b26534b8a9330ea0f7e76b68d866b7afa431733
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>
(cherry picked from commit e139552727bffe4da60cec61b5f18fcdb7cecff7)

Bug 4473 - Concentrate multipart-reply (table features) exception logs

 - prevents log from flooding by unsupported structures in table features
 - OVS 2.4 related issue

Change-Id: I07ca50074b0ac47047e6067ee22b1b3ec5414bc5
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>
(cherry picked from commit 0ab3b3d2d5ef029a9fc3ba9ee7e6215b6630df25)

BUG-4862: catch ClassCastException

When the looked up serializer does not implement the expected
HeaderDesersializer interface we end up with a ClassCastException. Catch
it, issue a warning and continue.

Change-Id: I70f23733078710507bdc65931e41c1bd02c0684f
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit b80a043a43abc0948378f2d8cbb5a3b819a1f3bc)

Bug 3230 - Attempt to use Epoll native transport if available

Attempts to use Epoll native transport if available.
Uses Epoll.isAvailable() to check availability.
Refer : http://netty.io/wiki/native-transports.html

Change-Id: I0019a084b0f2410f4cea7d5541fe0e5b49699bec
Depends-on: I073093f9a7b28de9890a6842f8bef72d4fdf6872
Signed-off-by: Dileep Ranganathan <dileep.ranganathan@intel.com>

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: I58019ccb199ba1aa5ace515d6f2601c93f9b21b5
Signed-off-by: Mike Kolesnik <mkolesni@redhat.com>

Bug 5118 - Unsent messages reported as failed after disconnect

 - if there were messages delivered to OutboundQueue and disconnect occured,
   unsent messages were not marked as failed, resulting in incorrect / no report
 - channelInactive in AbstractOutboundQueueManager was never triggered as the event
   was consumed in DelegatingInboundHandler

Change-Id: Ifd64c8f9346534a934d49a88ddd5c8f71cbb01e7
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>
(cherry picked from commit 25677520a3fca1a925a0970efa20d586e1445e6f)

Bug 5377: Support configuring cipher suites to use for SSLEngine

Change-Id: I8d573de8e5fd64f48776e274b75ff418a62bee1d
Signed-off-by: Rashmi Pujar <rpujar@inocybe.com>

Bug 4614 - Reintroduce xid check for correct RPC handling

Change-Id: I259147af4cbcab5115083df690edfa3cd3c8b5b9
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>
(cherry picked from commit 867607566914d5b8a79e161eb0a40c4be9e01746)

BUG-2825: use provided Ipv4/MacAddress factories

IetfInetUtil and IetfYangUtil provide performance-optimized methods for
converting Ipv{4,6}{Address,Prefix}/MacAddress to and from binary
format.

Note that this changes the strings being produced: ipv6 addresses are
properly shortened and max addresses have the canonical lower-case
format.

Change-Id: Ib912a8db3d4d89e85ee97bedc23a0160b7c36fba
Signed-off-by: Robert Varga <rovarga@cisco.com>

Bump yangtools to 1.0.0-SNAPSHOT

Change-Id: Ie9097f01a9bfe3e68e5dced1f17efdcb736d21bc
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

AllocatedXid assigned incorrectly

 - allocatedXid should be set as endXid of the last unflushed segment
 - incorrect allocatedXid resulted in unnecessary segment creation

Change-Id: I040e4bd3f58ace6a911349b04eea2c2793372aa0
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

AlreadyReading flag not used correctly

 - alreadyReading field is set correctly in channelRead() to signal that
   reading occurs and that any tries to flush are not needed, as a flush
   is always scheduled after the reading is done
 - alreadyReading flag was not set back to false, which
   might cause unnecessary checks for flush being scheduled

Change-Id: Icafdf282390340b176cbe4d9702c84558af8b412
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Bump versions by 0.1.0 for next dev cycle

Change-Id: I8e4a58ccdbfca15d14fb9fb9741bf3ae8f05b612
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 4942 - Barrier send condition updated

 - fixes uncovered case - if a barrier was scheduled and another
   barrier was sent due to exceeded "maxNonBarrierMessages", then we
   would cancel / not enqueue the scheduled barrier because of "sinceLast"
   time - this time is lower than "maxBarrierNanos"

Change-Id: I86f1be2ac0dc76241523ee3b6e6bf12bec90889e
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>
(cherry picked from commit 62c0db21eb5d45415011647427ba9f7a0dc69302)

Cleanup VersionMessageWrapper

Fields should really be final and the message buffer is not optional.

Change-Id: I2eda0fece1ac8ef1cb7337ff5f17f528455e2b4a
Signed-off-by: Robert Varga <rovarga@cisco.com>
(cherry picked from commit 3d7142f71d50155111db46032776fa22260afafc)

Pull in slf4j-log4j12 from odlparent

Rely on odlparent's dependency management instead of ${slf4j.version}
(which comes from odlparent anyway).

Change-Id: Ic2e1971fc047330d2386f20ff97d936edf2cd28e
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Extend openflow-protocol-impl serialization

     - Add new serializers/deserializers for OF1.0/1.3
     - Register new MessageFactory's
     - Register new TypeToClass's
     - Add relevant unit tests for each factory

Change-Id: I9a711aef3d7c4a934369ed1d9e9edfdd0d995f98
Signed-off-by: peppepetra <giuseppex.petralia@intel.com>

Add get configuration function to be able to access the openflow connection config from other bundle

Change-Id: I2f0b12f1649771226d032ff2644230895e10191b
Signed-off-by: Mohamed El-Serngawy <melserngawy@inocybe.com>

Barrier turn on/off - no Barrier pipeline

* impl noBarrier OutboundQueueManager and StackedOutboundQueue

Change-Id: I02abcd3618337a9a9373eb59f98565ce61d90ad6
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 64aebad95b27a6f903a7f65e419cd41df1816c90)

Barrier turn on/off - PacketOut bypass

Note: PacketOut doesn't need to wait for any response
so it's marked as completed during the process of flushing

Change-Id: Ib7372e77cefbafe921f28d48bc6c63d9ff12388c
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 381841508961dcb1587680cdd96437b75383a13b)

Barrier turn on/off - move more functionality from StackedOutboundQueue

* move more functionality for reusing to AbstractStackedOutboundQueue
* add general methods for Channel msg wrapper
* fix NPE from OFEncoder

Change-Id: I351300c4af40693ba444d3c10a1121e76d004d1b
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 05ab0ada9e8c590102caf34b462c8d989f09d72f)

Barrier turn on/off - StackedOutboundQueue definition

* add abstract definition for StackedOutboundQueue to allow
mor variability for possible child implementation

Change-Id: I04d8658e0ede049fb0b9265b57a5f7f528998442
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 3545506a79947dec9b4ded9afe51388dadb16e27)

Barrier turn on/off - Split OutboundQueueManager

* Split OutboundQueueManger functionaly to Abstract parent
for split barrier functionality and Channel management

Change-Id: Iff85917145034689fa494a679bf6a65b969c775b
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit deca37b27340c6e9b1576c281982d5d7f4ee1795)

Barrier turn on/off - Split ConnectionAdapter functionality

Split ConnectionAdapter functionaly to Abstract parents
for clean implementation

* add AbstractConnectionAdapter primary implement OpenflowProtocolSevice
* add AbstractConnectionAdapterStatistics StatCouter wrapper

Change-Id: Ibfd809d6bc7e2e95339f7c21e26958306b157d41
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit e35d417dacc1bf002076e432b575931e15769f65)

Barrier turn on/off-add switcher value to Config-Subsystem

* use-barrier support
* base trapnsport impl to ConnectionAdapter
* fix tests

Change-Id: I0ccb92cd9296880954a84dd9935a5273262840a1
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>
(cherry picked from commit 07ac625cae363f78ea5dd48356f617544c73913d)

Use odlparent-lite as artifacts parent

We created odlparent-lite to provide a minimal pom which allows projects
to publish to the correct server. Without this ${project}-artifacts may
fail to deploy to Nexus.

Change-Id: Ieb6f598818d0a7be91954a65a9878118682c12b1
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Use odlparent's netty feature

Instead of pulling n specific version of netty, pull in the feature from
odlparent.

Change-Id: Id954fa4179a93e9c9d3056e5b4252350bb6aab27
Signed-off-by: Robert Varga <rovarga@cisco.com>
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Bug 4473 - Ignore unsupported features on table features

This patch just ignores any unsupported feature, so the controller is
 still able to connect to the switch and use the currently supported
 features.

Without this change, the connection gets dropped if the switch
implements a new feature, e.g. experimenter action, that is not yet
supported by the controller.

Change-Id: Ibb8d3e5d5bde35903277df923aeb226a42ff947b
Signed-off-by: Renato Aguiar <renato.aguiar@hpe.com>

Added maven central required tags

Change-Id: I0ea22ab090ee45d7f906b921cffda056c3e1ddf3
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

BUG-4283: experimenter msg support - deserialization part

 - added support for symmetric and multipart experimenter message
   deserialization

Change-Id: Id0efd13f8f6254d55a30cd10871840d93174117a
Signed-off-by: Michal Rehak <mirehak@cisco.com>

BUG-4283: experimenter msg support - serialization part

 - added serialization support for symmetric and multipart
   experimenter message
 - comments worked in

Change-Id: I8fc4d00e6afc058b335034e8298f9b846d24d8bc
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Bug 4460 -  Util project is too general project name

Project with name "util" is too general. I guess, this project
is not common util project for whole Openflow or ODL projects.
So name has to be change to "openflowjava-util" because it looks
like util project only for openflowjava parent.

Change-Id: I5e5477b10c2542ae84a0cf4ffa3a5bec95e07004
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Fix BUG 4281

Caused by: java.lang.IllegalStateException: Resource '/META-INF/yang/openflow-instruction.yang' is missing

added KEY_WORD into modules ment to live outside of md-sal

Change-Id: I3d6cc4d3d9bf5eb845ab5757447bed462567c335
Signed-off-by: Yi Yang <yi.y.yang@intel.com>
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Bug 4432 - NPE problem in OFEncode

* add check for listener in msg wrapper
* add default listener for logging in OutboundQueueManager

Change-Id: Ib652d60a210feaddf96840334af3b435e7d0a14f
Signed-off-by: Vaclav Demcak <vdemcak@cisco.com>

Fixed missing prefix in augmentation.

Change-Id: Ia210b3c77cd7b593949439aca06e1f7d3bd719bd
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Use odlparent configuration for jacoco

Change-Id: I2825a7637eb95e085cbeeb3286e468362c3545b9
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Migration to use MD-SAL Project

Change-Id: I6cc9068df952934d153b328670508aa8e2314628
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Introduced openflowjava parent pom.xml

Change-Id: I68b321532e494ead49105012401332cc4a5df19c
Signed-off-by: Michal Polkorab <michal.polkorab@pantheon.sk>

Use odlparent checkstyle.version

Change-Id: I2633028a7bb218dadb2faf9c69829702ef5fb790
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

BUG-4072: Echo reply missing

 - propper wrapper added at place where OF-message is written
   into netty channel

Change-Id: I4a65cffff946c4f7a1ad8dbc589ab008958c8cf8
Signed-off-by: Michal Rehak <mirehak@cisco.com>

Explicitly set git-review branch to master

Change-Id: I902966c7f4827b152521e95aeacd774c6fe25f9f
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Bug 2875 - fix initiating OFPT_HELLO over SSL from the controller

Change-Id: I69a19898c2d4ea916f895d588dc9af20f036e1ef
Signed-off-by: Matus Marko <matus.marko@pantheon.sk>