Refactor VTN Manager.

  * Associate MAC address with virtual mapping which maps the address
    in the MAC address table.
  * Associate ingress and egress flow entries with virtual mappings
    which map them in the flow database.
  * Enable to get source and destination hosts configured in flow entries
    for internal use.
  * Optimize the way to purge obsolete flow entries and MAC addresses.
  * Scan TopoEdgeUpdate list only one time when an edge update event
    is received.

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

Suppress unwanted log messages while running tests.

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

Fixed incorrect log message which indicates failure of flow uninstallation.

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

Fixed incorrect javadoc comment.

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

Merge "ArpHandler emulator: Abolish IPv4 forwarding."

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>

ArpHandler emulator: Abolish IPv4 forwarding.

The base controller has moved IPv4 forwarding logic from arphandler to
simpleforwarding. So it also should be removed from ArpHandler emulator.

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

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>

Implement Comparable to MacVlan and MapReference classes.

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

Added VTenantPath.contains(VTenantPath).

VTenantPath.contains(VTenantPath) determines whether the virtual node
associated with the VTenantPath instance contains the virtual node
specified by another VTenantPath instance.

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

Allow unprivileged user to run VTN Coordinator.

  * A user who runs VTN Coordinator needs to have write permission
    for /usr/local/vtn/var directory.
  * db_setup command installs ownership of VTN Coordinator.
    Only a user who executed db_setup command can execute VTN commands,
    vtn_start, vtn_stop, and db_setup.
  * "db_setup -c" reset VTN Coordinator environment to initial state.
    * All data and log files are erased.

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

Terminate Tomcat instance when uncd crashes.

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

Provide Tomcat instance only for the VTN Coordinator.

  * vtn_start starts a Tomcat instance only for the VTN Coordinator,
    and vtn_stop stops.
  * VTN web application, including required Tomcat configuration, is
    pre-installed.
  * Note that TCP port number used by Tomcat is changed from 8080 to 8083.

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

Update VTN Coordinator sources to the latest version.

VTN Coordinator version is changed to 5.1.0.0-SNAPSHOT.

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

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>

Map StatusCode.BADREQUEST to HTTP 400 response (Bad Request) in REST API.

Note that this patch affects error response code returned by REST API.

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

Never learn zero MAC address in vBridge.

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

Implement Comparable to VTenantPath class.

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

Eliminate duplicate code in toString().

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

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>

Refactor VTN Manager.

  * Determine virtual node where an incoming packet is mapped using
    global cluster cache instead of virtual node scan.
  * Ease VLAN mapping restriction.
    * The same VLAN ID can be mapped to different vBridges as long as
      different nodes are specified.
    * A VLAN mapping configured with a non-null node (which maps VLAN
      network on a specific switch) can coexist with another VLAN mapping
      configured with the same VLAN ID and a null node (which maps VLAN
      network on all switches).
      * VLAN mapping configured with a non-null node is always prior to
        VLAN mapping configured with a null node.

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

Remove unused dependency, jersey-json.

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

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>

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>

Fixed typo in PortHandler.java.

Change-Id: I8f065278fb9ca52994678c49194ea9616aa99fc8
Signed-off-by: Hideyuki Tai <h-tai@cd.jp.nec.com>

Additional Functional tests for reading physical attributes from controllers

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

Merge "Add Delay to FT after mininet session termination"

Add Delay to FT after mininet session termination

Change-Id: I6304814847fa9e68ef6a855e45eab5c50eaca8da
Signed-off-by: balakrishnan <balakrishnan.ka@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>

[maven-release-plugin] prepare for next development iteration

[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"