Merge changes If0630105,I9d2d5e61,I1cea2a32,Icc05b6a7,Ic57eb4f8, ...
authorThomas Kee <xsited@yahoo.com>
Fri, 8 May 2015 23:22:45 +0000 (23:22 +0000)
committerGerrit Code Review <gerrit@opendaylight.org>
Fri, 8 May 2015 23:22:45 +0000 (23:22 +0000)
commit3ac03cb7480a07429932577cb0ad91c0106d06ad
tree448d6a23475d48700ad708df323f0b07519a4004
parent3203d813c37b25c0cad0c564e4dac4e8a8860da3
parenta5f96cdd393c8544209f3be1b665d7a332803d38
Merge changes If0630105,I9d2d5e61,I1cea2a32,Icc05b6a7,Ic57eb4f8, ...

* changes:
  Moved up some of the methods from the PEP and PDP OSMsgSender to their respective supers.
  Created abstract super class for all COPS Message Senders as each contained duplicate members. This refactoring should help with the further refactoring slated for the state managers and data process class hierarchies.
  Ensure all *DataProcess classes have been changed from abstract classes to interfaces as each does not contain any methods. Also moving common methods up the class hierarchy to allow for easier refactoring of the COPS connection and state managers.
  Created abstract super class for all COPS state managers called COPSStateMan as each contained a lot of identical code and began making the state managers more immutable.
  Generalize the COPSPdpException class to a COPSException to make upstream exception handling easier.
  Generalize the COPSPepException class to a COPSException to make upstream exception handling easier.
  Expanded CMTS emulator to accept at least one type of gate request. Future work will allow for the ability to configure gates for validation. With an emulator that can accept gate requests, I have expanded the PCMMService tests to include setting and removal of a gate request to a single mock CMTS. Additionally, as I had encountered issues unmarshalling COPSDecisionMsg objects containing a gate request, another test has been included here (which may be better suited to be located in COPSDecisionMsgTest but the gate set mock object being used can only be used in this module) that creates and parses a COPSDecisionMsg object used for setting gates.
  Added comment to the reasoning behind the hashCode() logic implemented in the COPSHandle class per David M. Karr's review on change #19455.
  Removed commented line and replaced with a TODO per David M. Karr's review to change #19457.
  Generalize the COPSPepException class to a COPSException to make upstream exception handling easier.
  Fixed block responsible for parsing the PCMM gate request's traffic profile from a byte[]. Only commented the BestEffortService line as we will most likely need to support the parsing of other types of ITrafficProfile objects.
  Fixed the routine responsible for parsing a gate request decision message.
  Fixed hashCode() so it returns the same value after the data in the _data + _padding is taken into account.
  Removed the requirement that the COPSData member (aka. _data) cannot be null. Found this issue while building tests for the packetcable-policy-server PCMMService class for connecting to and setting gates for a CCAP.
  Improved exception handling, increased immutability, and general houskeeping.
  Generalizing COPSPdpException to a COPSException.
  Fix to NPE during instantiation after moving the instantiation of the gateIidMap member after the call to the private method getGates().
  With previous change to AbstractPCMMServer, instantiation of the CMTS class now requires one to specify the port number that will be opened else the ServerSocket class will assign one automatically.
  Now Karaf module capable of starting the new bundles contained in the new feature odl-packetcable-policy-server-all.
  New feature bundle for the new feature odl-packetcable-policy-server-all.
  New configuration bundle for the new feature odl-packetcable-policy-server-all.
  The provider module for the Arris designed APIs.
  New model module for use in the new feature to be called odl-packetcable-policy-server-all.
  Miscellaneous driver fixes found while migrating and testing the new Arris API submission that will live in a new feature called odl-packetcable-policy-server-all.
  Added an optional COPSClientSI object as a member to the COPSDecisionMsg as per specs.
  Added new constructor. May need to tighten down interfaces in the future.