Bug 5008 - QoS and Queue fixes for ovsdb southbound 36/32836/3
authorEric Multanen <eric.w.multanen@intel.com>
Mon, 14 Dec 2015 15:10:10 +0000 (07:10 -0800)
committerSam Hague <shague@redhat.com>
Mon, 25 Jan 2016 15:01:08 +0000 (15:01 +0000)
commit6f88e57f42cbe67e1a0cdba8e741ec81bfb2149d
tree18b038f3ac9fb8311063aa5445a4dc2b049534eb
parent0cb28987d51eb62a536566c20ac58fa20688f972
Bug 5008 - QoS and Queue fixes for ovsdb southbound

Update the operational MD-SAL for Queue and Qos updates from the OVS
host.

Support configuration of QoS and Queue via configuration MD-SAL.

- Support for adding and deleting QoS entries.  A qos-id is used
  as a handle for qos entries in the MD-SAL.  This is added to
  the 'external-ids' of the qos entry.  Qos entries originating
  from the OVS host are assigned a qos-id of "qos://<uuid>".
- Qos entries can be assigned to a Port (termination point)
  which is present in the MD-SAL by setting the port qos
  attribute to the UUID of the relevant Qos entry.
- Qos entries are cleared from a port by updating the
  port configuration with the qos attribute removed.
- Support for adding and deleting Queue entries. a queue-id
  is used as a handle for queue entries in the MD-SAL.  This
  is added to the 'external-ids' of the queue entry.  Queue
  entries originating from the OVS host are assign a qos-id
  of "queue://<uuid>".
- Queue entries can be added or deleted to the queue list
  of a Qos entry present in the config by filling out (or
  removing from) the queue-list attribute of the qos entry.

v2 - added setting of exising qos uuid to an existing port
   - added clearing of a qos setting for a port
v3 - ovsdb.yang whitespace fixes
   - fixes for failed tests
v4 - fixed model - queue list in Qos entries needed
     a queuenumber key value
   - fixed up comments on whitespace and log messages
v5 - finished set/clear of queues list in a qos entry
   - added more description about the functionality in
     the patch set.
v6 - fixed up more whitespace, and copyright comments.
v7 - address review comments:
     - white space
     - remove unnecessary code
     - remove commented out code
     - adjust some LOG levels
     - remove unused imports

Change-Id: Idec02aaa82e1ffa66e58862ecfc5edff05050444
Signed-off-by: Eric Multanen <eric.w.multanen@intel.com>
(cherry picked from commit d64c55d65dda23f61737086d1522b0c2347b2391)
17 files changed:
southbound/southbound-api/src/main/yang/ovsdb.yang
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/SouthboundConstants.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/SouthboundMapper.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/QosRemovedCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/QosUpdateCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/QueueRemovedCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/QueueUpdateCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TerminationPointUpdateCommand.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TransactCommandAggregator.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbOperationalCommandAggregator.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbPortUpdateCommand.java
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbQosRemovedCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbQosUpdateCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbQueueRemovedCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/main/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbQueueUpdateCommand.java [new file with mode: 0644]
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/ovsdb/transact/TransactCommandAggregatorTest.java
southbound/southbound-impl/src/test/java/org/opendaylight/ovsdb/southbound/transactions/md/OvsdbOperationalCommandAggregatorTest.java