Bug 7757 -Removing qos deprecations in carbon
remove leaf qos,leaf queue-uuid from the model and then the relevant code in the ovsdb plugin along with postman collection

Change-Id: I2003b0e9f7e7d17d1b28b19da0a2bf31cadd637a
Signed-off-by: Praveen Mala <praveen.mala@intel.com>

BUG 8280: TerminationPoint reconciliation fails

TerminationPointConfigReconciliationTask is incorrectly setting up
BridgeOperationalState to be empty. This ends up triggering reconciliation
even for ports that are already present in switch.

When we try to add ports that are already present we get constraint
violation in transaction and as per OVSDB RFC it causes all operations in
that transaction to abort. This means that the ports that actually needede
to be reconciled never get added back.

Change-Id: I48819f46b6419c15a8ba85541b06551b950a3055
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

BUG 6692 - Fix checkstyle problems not detected by the current version

This change is required for overall move to new Checkstyle version, see
https://git.opendaylight.org/gerrit/#/q/topic:bumpCheckstyle

Change-Id: If08990eaf8b849e6d826322eb1b7301996bc86c1
Signed-off-by: David <david.suarez.fuentes@ericsson.com>

bug 7781 update vlan bindings and tunnel ips

When physical switch update arrives write it in op datastore.

Change-Id: I46ca4f70b59961eeebbc17c0073fe40f852278b5
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Don't ignore .gitignore

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

Bug 7619: correct the southbound artifact

The feature repository should be southbound-features4, not features4.

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

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

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

Bug-8053 OVS Bridge other_config:hwaddr not preserved for nodes not in config DS

SouthboundUtils.addBridge - add version that takes other configs

Change-Id: Id451a850db7e99aef013decc4466e67feb1370b7
Signed-off-by: Josh <jhershbe@redhat.com>

Bug 7779: Adding try-catch in hwvtep transactions.

Adding Try-Catch in TransactCommandAggregator
and HwvtepOperationalCommandAggregator , so no single failure
will cause us to stop processing other commands.

Catching specific exceptions instead of a generic Exception.

Change-Id: I8e552119e402de93a1c37cf87397451583f02294
Signed-off-by: Tomer Pearl <tomer.pearl@hpe.com>

Add Karaf 4 artifacts

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

Bug 8130: downgrade Karaf 3 dep to provided

org.apache.karaf.features.core is provided by the Karaf container, we
only need the dependency at compile-time.

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

Bug 8055: use Java collection constructors

... instead of Guava's, where appropriate.

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

Bug 8055: various performance issues

* Use Collections.singletonList() instead of Arrays.asList() with a
  single argument.
* Avoid calling toString() on String objects.
* Avoid constructing new Strings with String constants.
* Avoid String concatenation in StringBuffer.append() calls.

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

Bug 8055: remove redundant array creation

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

Bug 8055: remove unnecessary type casts

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

Bug 8055: remove unnecessary type arguments

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

Bug 8055: simplify a couple of streams

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

Bug 8055: use method references instead of lambdas

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

Bug 8055: use Java 8 predicates instead of Guava

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

Bug 8055: use lambdas instead of anonymous classes

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

Bug 8055: remove unnecessary type parameters

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

Bug 8055: remove all unused imports

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

bug 7781 update vlan bindings and tunnel ips

1) when the vlan bindings is deleted in the device , the same thing is not
deleted in operational datastore because the port to be updated is not
searched properly.

2) When the tunnel ip is updated, as the physical switch is merged the
tunnel ips child is appened. Excluded the tunnel ips from physical switch
merge operation and adding and removing them seperately.

Change-Id: I64004638a6ce5031fd0f15fad3586f8c84f869db
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 7599 rename opData method as operData

Change-Id: I468ce2289fe2e11bdec0445f960cf38bb0e9798b
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 7599 avoid mdsal read

while processing the config update avoid mdsal read
maintain the instance identifier vs connectionInstance map.
Update the map upon GlobalUpdate.
Clear the map upon node disconnect event.
Use the map to identifiy the connectionInstance for the changed node.

Change-Id: I8cbb505f0e56bdc0a3f5b46104a26aa02381ad6f
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 7599 improve perf of ucast mcas

for ucast macs update do not read operational state again
get the relevant data from local device info.
This avoids reading the whole node possibly from the other cluster node
for processing a single ucast mac update.

Change-Id: I14eb4419a208a7d38cb665d1ac94b0c41c18bcfe
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 7599 performance improvement for ucast macs

Using the getModifiedChildren api on the modification to identify the
modified children data under node.

updated data and deleted data is stored in operational state for access
by all the commands.

Added the following accessor methods in operational state.
List<? extends Identifiable> getUpdatedData(InstanceIdentifier<Node> key,
Class<? extends Identifiable> cls)
List<? extends Identifiable> getDeletedData(InstanceIdentifier<Node> key,
Class<? extends Identifiable> cls)

Change-Id: I76584d7c90323313da35e28a600019c05081f9d0
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Bug 7160 - operational store still has node after ovs disconnects

Change-Id: I4b8c333bc24fd5d406a8df55f375bb9c51622eb0
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Fix NPE in OvsdbConnectionService updateConfigParameter

seen in SingleFeatureTest (SFT) running under Karaf 4

maybe under Karaf 3 this was never called with null, but under 4 it is

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

Fix Checkstyle "Redundant Modifier: Redundant 'final' modifier."

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

Add ovsdb southbound Karaf 4 features

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

Add ovsdb hwvtep Karaf 4 features

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

Add ovsdb library Karaf 4 features

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

Bug 7836 - Make OVSDB southbound plugin listener port configurable

Change-Id: Ic59cb1af40de46e150bb09ec584f362755ed7e62
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug 2487 - Too large configuration file from OVS

Change-Id: Id8d7c9864503cb996112207c6018957ca302a3ad
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Checkstyle: fix ParenPad violations

Enforcing no space(s) after a left parenthesis and before a right
parenthesis is a proposed change [0] which can only be merged when all
violations are fixed in projects enforcing checkstyle. This patch fixes
those violations.

[0] https://git.opendaylight.org/gerrit/#/c/51316/

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

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

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

Bug 7023 - NPE at org.opendaylight.ovsdb.southbound.OvsdbConnectionManager.getClient

Change-Id: If157e70dbbbafa39d2334c1704f0ae9d7460bbda
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug 5306: Enable the SSL connection for ovs manager

Use the aaa-cert feature to manage the TLS connection
certificates and add configuration to the southbound-impl
to allow SSL communication.

Change-Id: I1354644b7a4cb30493ce2bc5b33a2d1d504aaae0
Signed-off-by: melserngawy <melserngawy@inocybe.com>

Replace dlux with dluxapps features

Bug 7484 - Dlux project split - patch for OVSDB project

Renaming is due to DLUX project split.
Also, replacing yangui feature with yangman, new, redesigned and restyled
application.

Change-Id: Ife40e4279a0875adf328fb214f9f6a5003dba12d
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>

Bug 7563: Add config for inactivity_probe and max_backoff

Fix v2.: Fixes wrong schema definition for MaxBackoff column in Controller
table. Adds two more exceptions to catch in
OvsdbOperationalCommandAggregator.

This adds support for inactivity_probe and max_backoff fields in
Controller entry. Changes to Util method for addBridge and setControllers
is done in a backward compatible way. Once Netvirt changes to new API,
older one will be removed.

Change-Id: Ia41afa0e6de9be28a50f0a408270ef854b3e1bbe
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Replace FindBugs :jsr305 by full :annotation  (Bug 7663)

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

Revert "Bug 7563: Add an ability to configure inactivity_probe for OVS"

This reverts commit 5791295e2c408ff8b975916bf673a6ffe2573ae0.

Change-Id: I44533ea4aa7bc2697be9888e15edad20de914c9b
Signed-off-by: Alon Kochba <alonko@hpe.com>

Bug 7563: Add an ability to configure inactivity_probe for OVS

This adds support for inactivity_probe and max_backoff fields in
controller entry. Changes to Util method for addBridge and setControllers
is done in a backward compatible way. Once Netvirt changes to new API,
older one will be removed.

Change-Id: Ib7fbb1f6ac70b745264ca6fa191b3f382f3580d0
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Bug 7559: Add utilities to expose TP by ext-id, get dpnId from TP

- Add utility to return nodes that contain a termination point
referencing a given interface ID via external interface ID.
- Add utility to return the external ID value of a given termination point.
- Add utility to return the datapath ID of a given node identifier

Change-Id: I630773d6291b33d2fa85a3bcb5c344538d83813b
Signed-off-by: Alon Kochba <alonko@hpe.com>

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: Ic3577c6433490b91f5a0cdba0b4211721d95f6f0
Signed-off-by: Robert Varga <rovarga@cisco.com>

BUG-7446: remove use of deprecated MoreExecutors method

sameThreadExecutor() was split into two methods, use the
correct one.

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

bug 6579 removed boilerplate code

removed the boilerplate code from commands moved it to super class
pulled the methods extractUpdated and extractRemoved to super class
child class implements getData and areEqual methods

extractUpdated method compares old data and new data and returns the
change set.
Comparison logic is taken care by the sub class.

extractDeleted gets the deleted keys .
for local ucast mac remove command ,If logical switch is deleted ,
it will return the local ucast macs to be deleted aswell.

logical switch removal is fired in seperate transaction to avoid
failure of transaction because of some local references which odl is not
aware of sometimes due to racing conditions.

if mcast mac locator set gets emptied , fire removal of mcast mac
if the json rpc transaction fails, clear the intransit data from device
info

Change-Id: Ia172b5887887f202ba1149ba748862d86f9c9869
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

BUG 7310: Add configurable option to skip columns

Adds a new cfg file to configure table columns to be skipped when monitoring updates from OVS
switch. Currently following tables and columns are configurable:

1. Manager: status

Change-Id: I6d869ca559cb69374dcdf75b60282c105af85806
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

BUG 7414: Tunnel pushed to wrong OVS

If we get DTCN with modiications on multiple nodes, we're processing the
changes as if they were recieved on all nodes. updateData() gets client
for each node in change, but instead of passing on the changes that are
made only on that node, it passes it all the changes including ones on
other nodes/clients.

Fix is to have a map of changes per client and only transact this subset
to the client.

Change-Id: Ie03176b470b120eba76478c8c12e44d831329953
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Bug 7373 - ConflictingModificationAppliedException: Node was
created by other transaction

Ovsdb/Hwvtep plugin creates respective topology under network-topology/topology
and also they make a check if the parent node (network-topology) exist or not.
If not they create the root node. In a scenario where user loads both the
plugin together, their is a possibility that both the plugin can try to
create the network-topology as the same time and one of them will end up
with ConflictingModificationAppliedException. To resolve this scenario
this patch avoid the explcit create of the parent node, and uses the md-sal
PUT api that creates the missing parents. Although it's slow compared to
the other version of PUT api, but given that it's one time operation
at warm up time, it of not much concern.

Change-Id: I9171f17ceef43040d91e75e368ab4e6f59970c59
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Signed-off-by: Sam Hague <shague@redhat.com>

bug 6579 checking the depency before transaction

integrated the mcast , ucast mac and logical switch with dependency queue

Change-Id: I60643affc1d275b5a1b3586147ac83a67a01d22d
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 6579 added dependency queue

Added Dependency Queue which gets processed upon config and op events.
Any object ( mac / port ) whose dependencies are not met gets pushed the
queue with an expiry timeout.

Added the following apis in UnMetDependencyGetter
getInTransitDependencies    returns list of keys in json rpc transaction
which are needed by this object to be processed.
getUnMetConfigDependencies returns list of missing config keys

Added the following apis in HwvtepDeviceInfo
onConfigDataAvailable processes config dependency queue
onOpDataAvailable     processes in transit dependency queue , gets called
after every json rpc transaction is done

Change-Id: I012958a40e0ab0aa0e0db14dc670f9c4dacbdd09
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 6579 handling back to back tx updates

Added the following utility methods in operational state and device info
isKeyPartOfCurrentTx indicates this key is also part of the same tx
isKeyInTransit indicates this key is part of prev tx to device

Before inserting any key into any table
check if this key or its dependent keys are part of prev transaction

Change-Id: I87699c244b7feee27d2c95178dfe5727f4148cdb
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 6579 added basic unit tests

Added unit tests for the following commands
McastMacsRemoteUpdateCommand
UcastMacsRemoteUpdateCommand
LogicalSwitchUpdateCommand

When logical switch is configured ensure that op.insert is fired
same with mcast mac and ucast mac also.

Change-Id: Ie08664faa233776f82fb0a4ea8d6f0522a4b4810
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Add docs for OVSDB

This patch just initializes sphinx for OVSDB.

Refer: https://git.opendaylight.org/gerrit/#/c/39142/

Change-Id: I1e2b0c83941fdf2d1ab080863d6d6b3fc04e94b0
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Specify karaf dependency types

The -karaf projects only produce .zip and .tar.gz artifacts, so the
-it projects need to specify a type. (Our Nexus provides fake
dependencies to handle this type of situation, but local builds
without snapshot repositories fail.)

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

Checkstyle clean-up: Remove useless "final" in interfaces

The "final" in interfaces only shows up for me in Eclipse, but doesn't
fail the CLI build; I suspect that is becaues of a slightly different CS
version in-IDE and on CLI; on the next CS upgrade this probably would
have been seen in the build as well, and so it's good to clean it up.

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

bug 7202 upon node reboot hwvtep op ds is missing

When the cluster is rebooted hwvtep operational datastore is not populated
for the existing clients.
Existing clients keep retrying , the moment 6640 port is up they establish
connection.
But by that time hwtep bundle is not up.
Eventually when the bundle is up it is only notified of future clients
only and not the already existing clients.

Change-Id: Ic816aea61c43000d7c8fbf14e51e0bcc5661664e
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

bug 7201 skip monitoring stats tables

skip monitoring stats table and version columns
which are generating huge traffic to controller

Change-Id: Icb84f92262d7e9464291a6f233a741a767d79003
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Prep for provider network IT

1. Support for configuring OvsdbPing timeout (probably not necessary but
left it in anyway
2. Support for docker-compose version 2 yaml
3. Support for not specifying ovsdb.controller.address. If this env var
is not passed in, DockerOvs will create a docker network and use that
instead. See the javadoc in DockerOvs.java for more details.

Change-Id: Ife26f5f241ed6a45b00b15750a8ebafa471f8434
Signed-off-by: Josh <jhershbe@redhat.com>

Corrected data type for "src-mac" in hwvtep.yang

Change-Id: I6c866000dc527cb24890a05890793a2deed5a664
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>

Bug 6851 fix: handling OtherConfig column clear

This fix would take care of updating datastore for the OVSDB Node
when 'other_config' column is cleared from openvswitch table.

Update to handle case OtherConfig column cleared

setOtherConfig() is updated to handle all cases to set OtherConfig
in Ovsdb node if present in OpenVSwitch new entry and to remove
OtherConfig from Ovsdb node if present in OpenVSwitch old entry.

Current method was missing the case when OtherConfig is cleared for
OVS table from switch. This case was generating the condition when
OtherConfig is NULL for OpenVSwitch new entry and OtherConfig is
non-NULL for OpenVSwitch old entry.

Change-Id: I366ce0f160079e211e5fe02b0e16c84df40d34cf
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>

bug 6643 hwvtep configuration reconcilation

Added operational node listener to trigger reconcillation
listens for physical switch node added event and schedules the reconcillation.
trigger transactions for changes in macs and logical switches and vlan bindings
trigger transactions for removal of local ucast and mcast macs aswell
so that to be deleted logical switches will not have any references
cancel the queued reconcilation task if any upon receiving node disconnect event

fixed some npes in mac remove commands
run the logical switch remove command post mac entries remove command as
those logical switches removed logical swithes have to be visible in the
other remove commands

validated the following test cases
1) disconnect the device and push the config (logical switch and remote
macs and vlan bindings ) and connect the device again
2) disconnect the device and delete some confing and connect the device
again

Change-Id: I12d0692e4c8b040f261a9c2cc5cffc05bef96f0d
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Change cipher suites for SSL connection

Change the list of enabled protocls and cipher suites to meet FIPs security requirement.
Following protocols and cipher suites will be enabled:
ssl protocols [SSLv2Hello, TLSv1.1, TLSv1.2]
cipher suites [TLS_ECDH_RSA_WITH_AES_128_CBC_SHA256, TLS_ECDH_ECDSA_WITH_AES_128_CBC_SHA256,
TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256, TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256,
TLS_RSA_WITH_AES_127_CBC_SHA256]

Change-Id: Ie257c8c89a789911255e92b54997b915deb6c2a4
Signed-off-by: Hsin-Yi Shen <syshen66@gmail.com>

Bug 6692: don't mock Optional

There's no need to ever mock Optional.

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

Bug 6692: remove InstanceIdentifierCodec instance from SBU

Turn SouthboundUtil into a "clean" utility class: pull out all methods
involving InstanceIdentifierCodec.

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

Bug 6692: use non-deprecated firstKeyOf() variant

This patch also adds a cast in SouthboundUtilTest to fix a
false-positive deprecation detection.

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

Bug 5995: remove org.json

org.json doesn't seem to be used in ovsdb, remove it.

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

Merge "Bug 6692: improve TyperUtils tests (first pass)"

Merge "Bug 6692: clean up MonitorRequestBuilder"

Bug 6692: improve TyperUtils tests (first pass)

This also drops the unused type parameter on getTableSchema().

For now we preserve the IllegalArgumentException-based behaviour in
getTableSchema() (instead of handling the missing schema and returning
null).

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

Bug 6692: clean up MonitorRequestBuilder

MonitorRequest can't be made immutable, so the utility of this builder
is to support fluent-style construction and addColumns() with multiple
columns. This patch removes unneeded methods from MonitorRequest,
drops the static builder() in favour of the constructor, adds support
for addColumns() with multiple strings, and constructs a new instance
for every call to build(). It also performs the skip cleanup in one
step instead of looping over the columns.

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

Merge "Clean up SouthboundConstants"

Merge "Fix clear bug related to "num" in JSON Node"

Bug 6486: get right docker instance

Change-Id: I7319674e4ed54f45582b4ccd38a48ddcdad51be4
Signed-off-by: Sam Hague <shague@redhat.com>

Clean up SouthboundConstants

Remove the values in SKIP_OVSDB_TABLE, they aren't used; if any
complementary information needs to be provided, it should be done as
comments. (As it stands the information doesn't seem to be useful.)

Use the varargs variants of Arrays.asList() in SKIP_COLUMN_FROM_TABLE,
or Collections.singletonList() for single values.

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

Bug 6486: get right docker instance

Change-Id: I35d7e773dd4f9bccf46a790b6e3699166ba0b6a1
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Bug 5555: Exception during operations on ReadWriteTransaction"

Fix clear bug related to "num" in JSON Node

Before this change, getConstraints() would call populateEnum()  which
would ... do nothing, ultimately; this was pointless, as it is currently
implemented.

I'm assuming here that the code was meant to do what I turned it into,
but I don't understand enough of this code to be certain.  I've also NOT
tested this in any way.  It does kind of seem reasonable though, looking
at what the lines before in getConstraints do.

An alternative would be to just remove those populateEnum methods - if
they do nothing, and there is no real bug here, then that was just dead
code.  More likely, I suspect it is a real bug that hasn't been found
yet.

I'll try to add FindBugs error detection for such cases in the future -
UC_USELESS_OBJECT is exactly for finding these kind of programming
errors, AFAIK.

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

Fixed inappropriate WARN message.

The plugin output the following inappropriate WARN message sometimes.

    Found non-topological node Optional.absent() on path {}

This patch fixed the inappropriate message, and changed the log level
from WARN to DEBUG because it is a debug information.

Other change:
  Modified the code which used Optional<> in an ineffective way.

Change-Id: I714a10b509e9a88676e910ec9fba717800c52272
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Bug 6486: Add support for two node DockerOvs

Change-Id: I61996cc2bc5b573309f6c389c670e09540aa2710
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 5555: Exception during operations on ReadWriteTransaction

An IllegalStateException can be happened during operations on
ReadWriteTransaction. This was resulting in system not being able to
process any southbound ovsdb events, since the code didn't handle this
exception.

Change-Id: I24ab3acc01fd5fb4d9380ccc8bcc7d332738d242
Signed-off-by: K.V Suneelu Verma <k.v.suneelu.verma@ericsson.com>

Merge "Bug 6505 - blueprint managed-property is not reading the .cfg file during the reboot."

Merge "Bug 6475: Removed unnecessary read operations."

Bug 6505 - blueprint managed-property is not reading the .cfg file during the reboot.

Change-Id: Ifb70a789e809ff5ecaa1cf138fa915606f530bfd
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Bug 6506: Add bridge stp_enble to ovsdb.yang

We can use command "ovs-vsctl" to enbale spanning tree protocol on a bridge,
e.g., "ovs-vsctl set Bridge br0 stp_enable=true". But there is missing the
stp_enble in ovsdb.yang.

Change-Id: Ied5c093fb739f220fe673c80cd74cf84be7e0c0b
Signed-off-by: Peng Zhang <pzhang@ctbri.com.cn>

Bug 6475: Removed unnecessary read operations.

Some classes always read data from MD-SAL even in the case they don't
use these data. This patch modified these classes to remove unnecessary
read operations.

Other change:

 * Removed unused members
   - updatedOpenVSwitchRows from OvsdbQosRemovedCommand.java
   - updatedOpenVSwitchRows from OvsdbQueueRemovedCommand.java

Change-Id: I53af02892bbfbb8f4f95ab94d988825fbc832f1b
Signed-off-by: Hideyuki Tai <Hideyuki.Tai@necam.com>

Bug 6486 - Add waits for IT Utils

Change-Id: I9e537ecf65cf4c88408213760b8119b7be709505
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 6486 - Add waits for IT Utils

Change-Id: Ieea6276eda62f4b437436bf73336c8dea2aa0c0b
Signed-off-by: Sam Hague <shague@redhat.com>

Merge "Bug 6472 - JsonRpcEndpoint Reaper Thread's daemon property is not set"

BUG 6467: Add cli flag to use virtual environment

Add a trailing "/" to $WORKSPACE being passed in.
Works a little different in jenkins than on
local machine.

Change-Id: I1a12aa0a7f10a1fe621a1bf7c8def203630297be
Signed-off-by: Victor Pickard <vpickard@redhat.com>

BUG 6467: Add cli flag to use virtual environment

This change is a followup to patch:
 https://git.opendaylight.org/gerrit/#/c/44022/

Add cli flag for virtual environment

Docker-compose is installed in a virtual environment on Jenkins.

Add a cli flag to enable switching to a virtual environment, if needed.

Netvirt IT requires docker-compose, so the netvirt IT jenkins
script will set this flag: -Ddocker.vEnvWs=$WORKSPACE

Change-Id: I989d071ef2303f4b4813c3145435b2ddb785a228
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Bug 6472 - JsonRpcEndpoint Reaper Thread's daemon property is not set

Change-Id: I0cfd2a192eed62eab988831398f8f8e89876417e
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Merge "Add cli flag for virtual environment"

Merge "Bug 6454: ProcUtils stdout should be grouped together"

Bug 6454: ProcUtils stdout should be grouped together

Change-Id: I559d66666c7f8af05f3f380ed9a90c77b1437691
Signed-off-by: Sam Hague <shague@redhat.com>

Bug 6463 - Monitoring _version column on the ovsdb
table is generating huge updates from switch

Change-Id: I421d2af0adabb2197af7ec2ccb68a2e95e58d2e1
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>

Filter out monitor updates for some columns

This filters out statistics and _version columns from all tables
when setting up monitoring for table updates.

This also filters out status column for Controller table.

Change-Id: I3ed875a6070c2ce61ab8ecf626a6856f99f8179f
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>

Add cli flag for virtual environment

Docker-compose is installed in a virtual environment on Jenkins.

Add a cli flag to enable switching to a virtual environment, if needed.

Netvirt IT requires docker-compose, so the netvirt IT jenkins
script will set this flag: -DuseVenv=true

Change-Id: Ie66e274dd10f1cefcbe134d837faea7c52a55013
Signed-off-by: Victor Pickard <vpickard@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
Signed-off-by: Victor Pickard <vpickard@redhat.com>