Bug 1058: Eliminate query parameters from the Location header.

Some REST APIs of the VTN Manager copy the request URI into the Location
response-header. But query parameters in the request URI should not be
copied.

Change-Id: I0ed50c9e4089f896295443b1a73dde87ffdf0321
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 9612a4d0066acf9f67f6d99805ae2e58996785a7)

gitreview: update default branch to stable/hydrogen

Change-Id: I94d1146789fdc871fda5319f643e8ed5983c92c8
Signed-off-by: Chris Wright <chrisw@sous-sol.org>

Bug 1043: Create "integrationtests" build profile.

Tests which take long time will be skipped unless "integrationtests"
profile is active.

Change-Id: I353c8a695fcc8145ad9cbaaf6a5c4c9422759ce0
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit a16aa8cb484fd79ca5e504ae4e26c39c53569684)

Use stable/hydrogen snapshot version of controller artifacts.

Change-Id: I84bbbc77b8e4d9a6c3ed0223f3d38830c108ef05
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Bug 1000: Fixed unstable tests in manager.integrationtest.

  * testIListenDataPacket()
    * Fixed invalid initial value of count down latch used to wait for
      completion of flow installation.
    * Need more time to wait for flow entries.
    * Ensure that required edge is established before test.
  * testIListenRoutingUpdates()
    * Check vBridge state using VTN manager listener.
  * FlowProgrammerService.mapNF needs to be synchronized.

Change-Id: I89fc5763924ed4216ec81a7b1676984c433c60de
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 641300d5059d8a6068f924737c99b93d5c912152)

Bug 996: Fixed unstable unit test.

This patch fixes failure of manager.implementation UT caused by
too short event timeout.

Change-Id: I857ba986b41aa682bdfbd21505a0d0898ec59ba2
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 3dec8f7c0ec0bc09a80a568f28815b51f2076721)

Bug 975: Eliminate the use of obsolete JSON-C function.

json_object_object_get() in JSON-C library is obsolete.
json_object_object_get_ex() should be used instead.

Change-Id: I902b80df6a68f3e92f4b2511e29dfc24e2e49f3f
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit d6bfd4ed424e3d8c03705cc2486647689def626d)

Merge "Fix for bug 712." into stable/hydrogen

Fix for bug 712.

Change-Id: I61f12076cc8c92d658bff749f2fc7b69ef5cb00f
Signed-off-by: sameera <sameera.k@hcl.com>

Bug 857: Add workaround for a bug of old Open vSwitch.

Old version of OVS has a bug that it may clear OFPFW_DL_VLAN_PCP bit in
FLOW_REMOVED message incorrectly. This workaround tries to remove the SAL
Flow from the flow database after eliminating DL_VLAN_PR match field if
FLOW_REMOVED handler receives a SAL Flow that has DL_VLAN_PR match field.
This should not cause any problem because current VTN Manager never uses
DL_VLAN_PR match field.

Change-Id: I700c61a51459aa86609dde9d5b192a07bb0965d9
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit c99405272263faa9b44ddef815c4b4a1040ac48e)

Merge "Fix for bug 438." into stable/hydrogen

Fix for bug 438.

Change-Id: Id54de5758afa88a9a93167d04c4676175ce5f200
Signed-off-by: kamatchi <kamatchi-l@hcl.com>

Fix for bug 673.

Change-Id: I5221f23dae6e1c111dc388616340e225640d8356
Signed-off-by: kamatchi <kamatchi-l@hcl.com>

Fix bug 385 to solve logical port computation in multiple controller scenarios

Change-Id: Ic23b8e41d6972e46bf5f0d97d45f3788eb3d944c
Signed-off-by: kamatchi <kamatchi-l@hcl.com>
(cherry picked from commit 25ef73d2aa79ca1f9866db7ee942f740416a5b36)

Bug 804: Fixed unstable test.

This patch fixes bug for unit test that the result was affected by
the order of test cases.

Change-Id: I2fa948882e7204e3cfd186c33490c13b8b49d9fc
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 09fcfab6cbcc67add0e6c943033ad599e5f83b08)

Fix for bug 774.

Fixed incorrect javadoc comment.

Change-Id: I89605ddaec328f26d7aeabb411931961b9ade471
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 5a8a2c0e0d84c81fe1e7ea612b38423430406093)

Fix for bug 769.

Fixed incorrect log message which indicates failure of flow uninstallation.

Change-Id: I728ac6ec77acd13f1becdf12f03e39b8af641e77
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 07bbee320f3c15468e4b0e3adfec2e68628edacb)

Fix for bug 768.

Fixed unstable tests.

  * VTNManagerImplDisableNodesTest.testDisabledNodeAfterTimeout():
    Need more time to start PACKET_IN service on a new node.
  * VTNManagerImplClusterTest:
    Test should not fail even if cluster event is removed by timer thread.

Additional changes:

  * VTNManagerImpl: Use putEvent() to catch unexpected exception.

Change-Id: I2343417d994920ddce911d7e460fb436d7914b2e
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit b051d5f53497f62223c6ed9945f7358abe6d5504)

Fix for bug 764.

Improved logging.

  * Don't record FLOW_MOD failure if the target node does not exist.
  * Record inventory change as informational log.

Change-Id: Ic7dab1f2e1b9dcd800ab70c62c15880e5c55f7a2
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 83ec37e40293a71443e45665ee2d35a40b97a2bd)

Fix for bug 763.

Try to detect IP address for a host as possible.

VTN Manager will send at most 10 ARP requests to a detected host.

Change-Id: I4356b8d42f91a5901315e75cc5dc69e6fffa322a
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit b4e36cfd45765e446c8d97e881d5df50ca002921)

Fix for bug 743.

Fixed unstable unit test.

A MacAddressEntry instance keeps a set of InetAddress instances using
an unorderd set. So string representation of a MacAddressEntry instance
is unstable because it is affected by the order of InetAddress instances.

Change-Id: Ic91af42fcfa851c67d0197d7ad34c2ae1e974413
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 17fbaae84822573f2fbae54607c669439c27474d)

Merge "Fix for bug 742." into stable/hydrogen

Fix for bug 742.

Bug fix for equals(Object) in virtual node path classes.

Virtual node paths of different types should be treated as different
objects even if they have the same path components.

Change-Id: If2e6ab3cfc1436b8a14210e63284190c284957f8
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit 4b20f15c9d54b6cf7a8259b70b3e480ad5d95ee8)

Fix for bug 738.

Bug fix for port mapping.

This patch fixes a bug that an unexpected port may be mapped if the switch
port to be mapped is specified by its name.

Change-Id: If1643560292e8d57df11b42e24adb1616f00407c
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>
(cherry picked from commit accd1b97ca6aad5773cfc561e720d1e2651b12cc)

Update POM versions for stable/hydrogen branch.

Change-Id: I304c8771057a423a549223a6bdbf68de74b6101a
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

[maven-release-plugin] prepare release master-hydrogen-vtn-bulk-release-1

Switch dependencies to release artifact.

Change-Id: Ifad21b07193ac7865497fdec8f427cc7cc6ecf85
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Deploy README for VTN Coordinator tarball.

Change-Id: Ic42101ff5f12e569f3c87c6543da61f48c32099b
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Modify the Supported Platforms for Virtualization Edition."

Modify the Supported Platforms for Virtualization Edition.

Change-Id: I84e4480be11088b8c58dd514a08e2f189d70f897
Signed-off-by: gvrangan <venkatrangang@hcl.com>

Merge "Define pluginRepositories for manager.checkstyle."

Merge "Clean up maven plugins."

Define pluginRepositories for manager.checkstyle.

Change-Id: I1f71df0c2ca4f5f7bfec2fbca9ac37bf57c843a2
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Clean up maven plugins.

  * Define plugin versions shared between Manager and Coodinator in
    common/pom.xml.
  * Remove unused buildnumber plugin configuration.

Change-Id: I2d56da41ebec98bd24a94c56842a273aed790272
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Fixed build error on Fedora 19 (i686).

Both true and false expression in conditional expression must be evaluated
as the same data type.

Change-Id: I9eeb3846d1656ba9c6ed095d1698c6add8a10536
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Add unit test cases for physical read feature"

Merge "Add functional test for physical read"

Add functional test for physical read

Change-Id: If518f00995bed3ab5982929599eeed277be55d54
Signed-off-by: balakrishnan <balakrishnan.ka@hcl.com>

Merge "Modify FT script for multicontroller portmap test using mininet"

Modify FT script for multicontroller portmap test using mininet

Change-Id: Ifcab1dff577c616d4115c1d986abcf7abe122711
Signed-off-by: balakrishnan <balakrishnan.ka@hcl.com>

Refine artifact dependencies.

  * Migrate artifact versions to common/pom.xml.
  * Remove unused dependency from northbound IT.

Change-Id: Ia62697846727d2a1433b55b5e97b187928980751
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Fix Bug in TC Module"

Merge "Fixed illegal usage of SQLBindCol() in UPPL module of VTN Coordinator"

Merge "Fix Compilation Error in Unit Tests for Ubuntu 12.04"

Fix Compilation Error in Unit Tests for Ubuntu 12.04

Change-Id: I52afd83777cb25cc25d0cad37b816676f9f4ff0d
Signed-off-by: Karthik.S <siva-karthik@hcl.com>

Merge "Detect PostgreSQL in runtime environment."

Detect PostgreSQL in runtime environment.

This patch makes db_setup script detect PostgreSQL and ODBC driver.
If you want to specify the location of PostgreSQL explicitly, please
define below shell variables in /usr/local/vtn/etc/dbm/unc_db_env.conf.

  PGINST
    Set an absolute path to the PostgreSQL installation directory.

  PGSQL_ODBC_DRIVER
    Set an absolute path to PostgreSQL ODBC driver library.

Change-Id: I51bd0477b4607f46b58c7898308a001c61dec093
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Sync artifact version to the controller.

Change-Id: I519bec762c70df144cf3a7d6f1d3ed9f5834a576
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Fixed bug in VTN Manager unit test.

Change-Id: I3cef6d6462bf4a32d636e688a51b6f06fe0b9693
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Add support for MD-SAL protocol plugin.

  * Accept MD-SAL node and node connector if object identifier is
    a string which starts with "openflow:".
  * Ignore incoming LLDP packet.

Change-Id: I4b8f52e958a135ec9a2314000d9d303dba1ace52
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Support for reading links between switches from controller"

Support for reading links between switches from controller

Change-Id: Ic72a4e5118ce528365f8cc0cab643e9c35f7ee3f
Signed-off-by: surya.s <surya.s@hcl.com>

Fix Bug in TC Module

  * Corrected the default values in database table

Change-Id: I0c2907bd4b980efaa6dd6f09a7a5137633f49d24
Signed-off-by: mausumi.p <mausumi.p@hcl.com>

Merge "Fix Cases in Functional test scripts for multi controller scenario"

Fix Cases in Functional test scripts for multi controller scenario

Change-Id: Ieafc38766d74444f8d420f045ad42ff2d55be667
Signed-off-by: senthil <senthil-b@hcl.com>

Fixed illegal usage of SQLBindCol() in UPPL module of VTN Coordinator

Change-Id: I9c82dbc0ca4cfd6a09449d422826e3e8e06fee31
Signed-off-by: mausumi.p <mausumi.p@hcl.com>

Fixed UT test failure caused by lack of storage for file path.

Change-Id: If91e36d65a76e4232e1eb2b6651da618595cc8d5
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Disable Google Test from outputting colored text on Jenkins build.

Change-Id: I6ea84f3d176503c748b7ecb1ba0a7a21bae1f787
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Verify json-c library supports 64-bit integer.

VTN Coordinator build will fail unless json-c 0.10 or newer is installed.

Change-Id: I5191cce9a2f5d39ca5e40582d34399f2206125a2
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Add unit test cases for physical read feature

Change-Id: I5a57031f214213e93ec9db8ebd13c7b89ce16ca3
Signed-off-by: kamatchi <kamatchi-l@hcl.com>

Refine log levels.

This patch also refines log messages in manager.neutron bundle.

Change-Id: I774f0223c49cdee5617c07998d170408a7429823
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Fix for deadlock while read from multiple controllers

Change-Id: I07303e6b0c6ee0d185f7e835e2aeb6a0b80f501e
Signed-off-by: balakrishnan <balakrishnan.ka@hcl.com>

Suppress messages to the standard error output.

Change-Id: I0a6f01f6796d7ddf9f5b0d3c76308cc76622f56f
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Fix vtncacheutil Compilation Failure"

Merge "Avoid UT test failure that may happen on low-spec machine."

Fix vtncacheutil Compilation Failure

 * Unused variables were reported.

Change-Id: I320a5479953e475faf28036eff118e58ba047ebd
Signed-off-by: gvrangan <venkatrangang@hcl.com>

Avoid UT test failure that may happen on low-spec machine.

This patch eases allowable error of time for which file lock test blocks.

Change-Id: I90d78c2f9ccddaaeba0ab215b18373b74f6ae48f
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Fixed typo in NB API method name.

Change-Id: I0429661169f650661499816f75380661d1a12723
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Add copyright notice.

Change-Id: Ic14ffad0eac0f4dcd19bf43ea072d98bfa842cce
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

vtn-javaapi: Refine IPC error code definition for upll and uppl.

This patch enables both upll_errors.properties and uppl_errors.properties
to have comments.

Additional changes:

  * UncJavaAPIErrorCode.java: Fixed typo.

Change-Id: I29bdf399796b29d18a45105111fc2fd9f7d00493
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Use LSB information to determine Linux distribution.

Change-Id: I302c560f6282535f2ade1985874500577615359e
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Eliminate direct output to the standard output stream from tests."

Merge "Update copyright information"

Update copyright information

Change-Id: Iaddd793dd61ec1aca1814e8e2891b640006eee68
Signed-off-by: susant <susantchandra.s@hcl.com>

Merge "Add maven-release-plugin."

Read support for switches and ports from controller

Change-Id: I60e82073f7a97eeb7570151bc2d0aa958a28ef71
Signed-off-by: susant <susantchandra.s@hcl.com>

Eliminate direct output to the standard output stream from tests.

Change-Id: I590f4e38a8769d9db0ba4db1035ef06101ed1b2e
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Add maven-release-plugin.

Change-Id: Ibb22cdf8f1b9e6a42577f8166c5978bb722cd21f
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Fixed illegal usage of SQLBindCol() in VTN Coordinator."

Correct invalid javadoc link in IT test.

Change-Id: I48833300176709bd5746fe3bd3ff68d71541ee38
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Fix for ArpHandler emulator.

  * Don't forward IPv4 packet destinated to default gateway.
  * probe(HostNodeConnector): Check VLAN ID in the given HostNodeConnector.

Change-Id: Iaec5fae2a42985c3592d2f2b4fafefeea3dde691
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Refine Java and REST API documents.

Change-Id: Ieb58b756c25c166e1bdeca3614089b0c0a27060b
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Fixed illegal usage of SQLBindCol() in VTN Coordinator.

  * modules/tc/tc_db_handler.cc
    * PostgreSQL ODBC driver converts value into string if a boolean
      column is bound as SQL_C_CHAR type. In this case SQLFetch() will
      store a string value, including terminator, into the buffer passed
      to SQLBindCol(). If a SQLCHAR (1 byte) variable is bound to a
      boolean column as SQL_C_CHAR type, SQLFetch() will cause buffer
      overrun when it stores a string terminator.

    * SQLBindCol() takes a SQLLEN pointer as 6th argument, not
      SQLINTEGER. On 64-bit platform, actual data type of SQLLEN and
      SQLINTEGER are int (4 bytes) and long (8 bytes) respectively.
      If a pointer to SQLINTEGER variable is passed to 6th argument of
      SQLBindCol(), SQLFetch() will cause buffer overrun because it
      will store a SQLLEN value into a SQLINTEGER variable.

  * modules/uppl/odbcm_db_varbind.cc
    * SQLFetch() will store value into buffer specified by SQLBindCol()
      call. That is, all buffer specified by SQLBindCol() must be valid
      until SQLFetch() is called.
      DBVarbind::bind_is_row_exists_output() specifies the address of
      local variable, cbNumericType, to 6th argument of SQLBindCol()
      without calling SQLFetch(). This code will corrupt unexpected
      buffer when SQLFetch() is called because the storage for
      cbNumericType variable is released when bind_is_row_exists_output()
      returns.

Change-Id: I8edbcc6b736b2a3bd4201ab5e99a3441e0ec2e81
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Enable to build VTN Coordinator on Ubuntu."

Fixed vtncacheutil UT build error on 32-bit platform.

Change-Id: Ia352c891d1640c0e2374b195b71e51217a210143
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Enable to build VTN Coordinator on Ubuntu.

  * Specify object files to linker before libraries to be linked.
  * Add some gcc options to suppress unwanted warnings.
  * Detect gcc multiarch.
  * unixodbc package does not provide odbc_config command.
  * libgtest-dev package does not provide gtest-config command.
    * libgtest-dev provides only sources for Google Test libraries.
      You must build and install Google Test libraries if you want to
      run UT tests which use Google Test.
  * Remove unnecessary checks in configure script.
  * Change PostgreSQL installation directory for Ubuntu.
    * This patch assumes that postgresql (version 9.1) and odbc-postgresql
      package are installed to Ubuntu.
  * Define BoolsAsChar parameter for PostgreSQL ODBC driver explicitly.

Change-Id: Ib2fdb6047b75eca1740dd9917d538f7c54fc4cd0
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

CacheUtil Support for the Physical-Read Feature

Change-Id: Ief9ab181c11fa31cc9b28f2daf83d8b196d277d7
Signed-off-by: surya.s <surya.s@hcl.com>

Merge "Update data files for multi-controller FT test case"

Update data files for multi-controller FT test case

Change-Id: I32709c92e2dc460d9bfa490dc053820a964c1666
Signed-off-by: senthil <senthil-b@hcl.com>

Suppress gcc warning on VTN Coordinator UT build.

Below is a summary of warnings which may be detected by gcc.

  * Comparison between signed and unsigned values.
  * Comparison between different enum types.
  * Call of open() with specifying O_CREAT and without specifying
    permission.
  * Buffer overrun caused by specifying invalid size to memcpy().
  * Converting bool to pointer type, caused by EXPECT_EQ()/EXPECT_NE()
    macro with specifying bool value.

Change-Id: I27155f4ed259ce9f4a1d82b1ab4762df0deb766c
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

upll: Fixed buffer overrun.

Although sizeof(val_flowfilter_entry.redirect_node) is 32, upll module
passes 33 (kMaxLenFlowListName + 1) to strncpy() when it copies a string
into val_flowfilter_entry.redirect_node. It causes buffer overrun because
strncpy() always copies the specified number of bytes to the destination
address.

Change-Id: I4c40e33c05bf361baa97a16ceeb3ea91eef9e360
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Add README for VTN Coordinator"

Merge "Don't expect build scripts for VTN Coordinator are executable."

Add README for VTN Coordinator

Change-Id: Icd7c98d8e308ad934bcc74e6d348a21882337fe8
Signed-off-by: susant <susantchandra.s@hcl.com>

Merge "Modify FT script for portmap test using mininet"

Don't expect build scripts for VTN Coordinator are executable.

Currently all scripts under coordinator/dist/build have executable
permission. But executable permission bit may be turned off when the source
tree is copied. This patch enables the VTN Coordinator build even if
build scripts are not executable.

In addition, this patch eliminates tab characters and trailing whitespaces
in coordinator/dist/pom.xml.

Change-Id: I2b4385cb62dc921e4367696d690e6b22aed1a0d1
Signed-off-by: Shigeru Yasuda <s-yasuda@da.jp.nec.com>

Merge "Upgrading Jackson to 2.3.0 & be in sync with Controller's upgrade."

Merge "Adding support for the git-review command line tool"

Upgrading Jackson to 2.3.0 & be in sync with Controller's upgrade.

Please refer to https://git.opendaylight.org/gerrit/#/c/3905/

Change-Id: I469da2d2ee7475b927fa168e060a9adbdb6d5f85
Signed-off-by: Madhu Venugopal <mavenugo@gmail.com>

Modify RestJsonUtil UT to use the common stub source

Change-Id: I4507cce1157c01684b54a1a53e2b5c031db3cb0b
Signed-off-by: Karthik.S <siva-karthik@hcl.com>