Faseela K [Mon, 16 Jan 2017 14:37:37 +0000 (14:37 +0000)]
Merge "Bug 7466 - NPE thrown for interface without lport tag"
Faseela K [Mon, 16 Jan 2017 11:14:34 +0000 (11:14 +0000)]
Merge "Minor cleanup and formatting"
Stephen Kitt [Mon, 2 Jan 2017 14:30:03 +0000 (15:30 +0100)]
InstructionInfo redesign
This follows the same principles as the ActionInfo redesign; given the
small number of classes involves, the redesign is done in a two
commits (this backwards-compatible change, introducing the new
InstructionInfo descendents, and another clean-up which will be
applicable once the downstream users of InstructionInfo have been
fixed).
Change-Id: Ibfa7a8dc65f36c5f599dbdeffc24508fd136e198
Signed-off-by: Stephen Kitt <skitt@redhat.com>
David Suarez [Sun, 15 Jan 2017 18:29:13 +0000 (19:29 +0100)]
Minor cleanup and formatting
- Minor clean and formatting.
- Speedup lastIndexOf changing to char.
Change-Id: I56f47b90ffb36a057b1b7b3e7faa3b4b154bd35c
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
David Suarez [Sat, 14 Jan 2017 11:39:50 +0000 (11:39 +0000)]
Merge "BUG-5222: do not pull in odl-mdsal-xsql"
Vishal Thapar [Sat, 14 Jan 2017 11:36:03 +0000 (11:36 +0000)]
Merge "Fix Idmanager JUnit test case"
Robert Varga [Fri, 13 Jan 2017 10:23:31 +0000 (11:23 +0100)]
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: I3ea0428b7e74c103d87df8cb108e29285647c335
Signed-off-by: Robert Varga <nite@hq.sk>
Miguel Perez [Fri, 13 Jan 2017 14:51:54 +0000 (15:51 +0100)]
Cleanup unnecessary 'toStringIpAddress'
+ 'toStringIpAddress' is centralized in NWUtil
+ projects should use that one instead of their own copy
Change-Id: I44db9c910d8de57a6f1e6070e9cc37f61c1bc9f2
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>
Kiran N Upadhyaya [Sat, 14 Jan 2017 09:31:35 +0000 (15:01 +0530)]
Fix Idmanager JUnit test case
The JUnit used to test concurrent Id requests from Released Ids has to
wait before assering that values in the IdPool are consistent, as the
ReleaseId allocator uses DJC(spawing yet other independent threads) to
update the ReleasedId container with the correct values on every
allocation.
Change-Id: Ib6e69dc58c7436d9eca6b7c58098f5c307bf5ea7
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>
Faseela K [Sat, 14 Jan 2017 02:06:45 +0000 (02:06 +0000)]
Merge "Enforce checkstyle validations"
Michael Vorburger [Tue, 10 Jan 2017 13:27:16 +0000 (14:27 +0100)]
interfacemanager Organize Imports for Checkstyle compliance
Change-Id: I0b44d9f431065629b46987fc2f28caf8e89883bd
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
David Suarez [Fri, 13 Jan 2017 22:50:48 +0000 (22:50 +0000)]
Merge "TestableDataTreeChangeListenerModule with logging"
David Suarez [Fri, 13 Jan 2017 22:28:05 +0000 (23:28 +0100)]
Enforce checkstyle validations
Checkstyle enforcement.
Change-Id: I061bb99afeabeefa887029aa67a72cf0bb1dcc64
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Michael Vorburger [Fri, 13 Jan 2017 21:46:52 +0000 (22:46 +0100)]
TestableDataTreeChangeListenerModule with logging
Change-Id: I700186eb135dfd641a6804838e9611547c679345
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
David Suarez [Fri, 13 Jan 2017 20:46:56 +0000 (21:46 +0100)]
Enforce checkstyle validations
Checkstyle enforcement.
Change-Id: I8ceb5e261a0e925c75237900c6cbf2c583cf173c
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Faseela K [Fri, 13 Jan 2017 16:21:33 +0000 (16:21 +0000)]
Merge "BUG 7494 : Idmanager returns the same Id from the same pool for different threads with different id keys"
Kiran N Upadhyaya [Mon, 9 Jan 2017 11:32:45 +0000 (17:02 +0530)]
BUG 7494 : Idmanager returns the same Id from the same pool for different
threads with different id keys
Problem with concurrency in IdManager, such that two threads requesting Id
at the same time with different keys might end up getting the same key.
This is because of unsafe read of an in-memory cached local Id Pool.
Also, added IdManager test cases for multiple threads requesting
allocated/ released Ids concurrently.
Change-Id: I3d8c0e921461c4ecc5d3ca4ed41b9d18fee9a725
Signed-off-by: Kiran N Upadhyaya <kiran.n.upadhyaya@ericsson.com>
David Suarez [Fri, 13 Jan 2017 11:48:15 +0000 (11:48 +0000)]
Merge "interfacemanager Checkstyle compliance: variable names & line length"
Tomer Pearl [Thu, 12 Jan 2017 14:37:17 +0000 (16:37 +0200)]
Bug 7466 - NPE thrown for interface without lport tag
Code uses Integer that throws NPE when unboxing.
Now it checks for null beforehand
Change-Id: I9e6e513327c3231c0a98e73ac1738b06058e73c8
Signed-off-by: Tomer Pearl <tomer.pearl@hpe.com>
Stephen Kitt [Thu, 22 Dec 2016 14:04:43 +0000 (15:04 +0100)]
Action redesign: implement equals() and hashCode()
Change-Id: Ie6e86ebac788492220419ed6e6beae6b04c59b1e
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Fri, 16 Dec 2016 13:39:15 +0000 (14:39 +0100)]
Action redesign: migrate set_{destination,source}_ip
Change-Id: I557ad1d1b79a224c5d4adc9b71c2b0aa00652165
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Fri, 16 Dec 2016 15:06:41 +0000 (16:06 +0100)]
Action redesign: migrate NICIRA ARP extensions
Change-Id: I64583a3f57028af371ff1f0c7a8cb2359c384501
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Fri, 16 Dec 2016 10:18:18 +0000 (11:18 +0100)]
Action redesign: migrate move_src_dst_{eth,ip}
Change-Id: Idda8f0ee6b0ebb91514bb2562e483e47362fb973
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Fri, 16 Dec 2016 14:27:53 +0000 (15:27 +0100)]
Action redesign: migrate nx_load_in_port
Change-Id: Ia3975bf642439cc33166613a5c2bfa8e2c58cf52
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Wed, 14 Dec 2016 16:04:07 +0000 (17:04 +0100)]
Action redesign: migrate set_{tcp,udp}_{dest,src}_port
Change-Id: I45da9e1d0c847b0f9b40cf06636ae6a8db6596a6
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Fri, 16 Dec 2016 14:04:49 +0000 (15:04 +0100)]
Action redesign: migrate set_icmp_type
Change-Id: Id9c36b7967c33013b4e43c5f161f654d2ddd1a8a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Wed, 14 Dec 2016 14:54:15 +0000 (15:54 +0100)]
Action redesign: migrate set_udp_protocol
Change-Id: Id5d73de28e83f48840e591cea7a3240ddffb9147
Signed-off-by: Stephen Kitt <skitt@redhat.com>
David Suarez [Thu, 12 Jan 2017 11:23:52 +0000 (11:23 +0000)]
Merge "Action redesign: migrate punt_to_controller"
David Suarez [Thu, 12 Jan 2017 11:19:39 +0000 (11:19 +0000)]
Merge "Action redesign: migrate set_field_eth_{dest,src}"
David Suarez [Thu, 12 Jan 2017 11:18:12 +0000 (11:18 +0000)]
Merge "Action redesign: convert drop_action and nx_resubmit"
David Suarez [Thu, 12 Jan 2017 11:07:11 +0000 (11:07 +0000)]
Merge "Action redesign: convert set_tunnel_{src,dest}_ip"
David Suarez [Thu, 12 Jan 2017 11:06:48 +0000 (11:06 +0000)]
Merge "Action redesign: add typed getters in NxmOfFieldType"
David Suarez [Thu, 12 Jan 2017 11:05:57 +0000 (11:05 +0000)]
Merge "Action redesign: always provide actionKey constructors"
Faseela K [Thu, 12 Jan 2017 09:47:56 +0000 (09:47 +0000)]
Merge "Fix some minor problems."
Stephen Kitt [Wed, 14 Dec 2016 14:43:14 +0000 (15:43 +0100)]
Action redesign: migrate punt_to_controller
Change-Id: I62a64db77432c3ed7174181f12404a4d5f8db214
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Wed, 14 Dec 2016 14:00:50 +0000 (15:00 +0100)]
Action redesign: migrate set_field_eth_{dest,src}
Change-Id: I77b1ed6a9e76d7b66b7cd43941ead50ee01482fb
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Thu, 8 Dec 2016 20:13:31 +0000 (21:13 +0100)]
Action redesign: convert drop_action and nx_resubmit
Change-Id: Ie70eae9c06f36b57dcfc92933b7d816d033d15d3
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Thu, 8 Dec 2016 19:36:32 +0000 (20:36 +0100)]
Action redesign: convert set_tunnel_{src,dest}_ip
Change-Id: I286d333ef38fb3b59b2a41af9e52c755c96776b8
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Thu, 8 Dec 2016 02:19:49 +0000 (03:19 +0100)]
Action redesign: add typed getters in NxmOfFieldType
This is slightly inelegant since we can't replace getHexType() and
getFlowModHeaderLen() wholesale; we need to deprecate them in Carbon,
then replace them in Argon and deprecate the new functions, and
finally replace the now-useless new functions in Oxygen.
Change-Id: If63f9f732832e946ca43390cc6157840601d6da2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Wed, 7 Dec 2016 16:09:13 +0000 (17:09 +0100)]
Action redesign: always provide actionKey constructors
netvirt needs these...
Change-Id: I03da2ddfe18c4a2bdec3fba9fef33957743f7780
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Wed, 7 Dec 2016 10:45:03 +0000 (11:45 +0100)]
Action redesign: ActionType.set_field_...
This patch switches MPLS labels, PBB ISID, VLAN and tunnel id handling
to the new model.
Change-Id: I18b621016f490bf84b4f68d1bfb78e14addc6030
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Stephen Kitt [Wed, 7 Dec 2016 07:42:34 +0000 (08:42 +0100)]
Action redesign: refactor ActionType.learn
Change-Id: I5ffe1d46dd1a88d25a466a976ab4cd03c532ea69
Signed-off-by: Stephen Kitt <skitt@redhat.com>
David Suarez [Tue, 10 Jan 2017 10:41:08 +0000 (11:41 +0100)]
Fix some minor problems.
* Logger name
* Fix exception log
* Name of attributes
Change-Id: I11e3d829fa735c8a804ce9df71fa91da312eb072
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Faseela K [Tue, 10 Jan 2017 17:01:20 +0000 (17:01 +0000)]
Merge "Blueprint for LockManager"
Michael Vorburger [Tue, 10 Jan 2017 14:19:58 +0000 (15:19 +0100)]
interfacemanager Checkstyle compliance: variable names & line length
and also white space issues, and also remove some redunant "public" on
constructor of private classes.
this is incomplete, but it's a start. Let's get this merged, and I'll
continue on more like this another day
Change-Id: Ie4b73fc2f6ef8c6cf14dbd933a70619222972093
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Michael Vorburger [Tue, 10 Jan 2017 14:34:21 +0000 (15:34 +0100)]
Fix ActionRegLoadTest & ActionRegMoveTest failures
this happened following the upgrade of the xtendbeans test utility in
https://git.opendaylight.org/gerrit/#/c/50137/, because of
https://github.com/vorburger/xtendbeans/commit/
b554dcf088646122bbcc6dbed83025023f32f7dc
Change-Id: I44fbda1ff45fa5f3d390f7b8af49dafd43398f8c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Edwin Anthony [Wed, 30 Nov 2016 16:30:02 +0000 (22:00 +0530)]
Blueprint for LockManager
Change-Id: I3e1a1a396520cfd01d725407024b19fff09cb26e
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>
Vishal Thapar [Tue, 10 Jan 2017 04:45:50 +0000 (04:45 +0000)]
Merge "Spec for ITM Tunnel Auto-config feature."
Michael Vorburger [Mon, 9 Jan 2017 16:26:28 +0000 (16:26 +0000)]
Merge "ITM POM minor clean up"
Michael Vorburger [Mon, 9 Jan 2017 13:46:57 +0000 (14:46 +0100)]
ITM POM minor clean up
moved all Testing Dependencies together
removed not needed <version> of org.apache.karaf.shell
Change-Id: I2f363becf04bc2903fb249af7070c94018349ce8
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Miguel Perez [Tue, 27 Dec 2016 18:21:30 +0000 (19:21 +0100)]
Refactor NWUtil.toStringIpAddress
+ Replaces the current StringBuilder approach with the
InetAddress.getByAddress() method that is 1) faster and
2) IpV6 compatible.
+ This method is not used anywhere (yet), but it will.
Change-Id: Ife5fa781542a8465626d8a6bc2feb6d66cfa62b6
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>
Faseela K [Sun, 8 Jan 2017 03:16:05 +0000 (03:16 +0000)]
Merge "Replace dlux with dluxapps features"
Faseela K [Sat, 7 Jan 2017 05:58:48 +0000 (05:58 +0000)]
Merge "Use batching in interface Delete path"
Tarun Thakur [Wed, 30 Nov 2016 14:27:18 +0000 (19:57 +0530)]
Spec for ITM Tunnel Auto-config feature.
1. Added specification document for ITM Tunnel Auto-config feature.
2. Linked spec to specs/index
Change-Id: I1f7afb9ea91cb00ad118bd11aa3d4f7c61a21798
Signed-off-by: Tarun Thakur <tarun.t@altencalsoftlabs.com>
Daniel Malachovsky [Wed, 14 Dec 2016 11:44:17 +0000 (12:44 +0100)]
Replace dlux with dluxapps features
Renaming is due to DLUX project split.
Also, replacing yangui feature with yangman, new, redesigned and restyled
application.
https://wiki.opendaylight.org/view/Weather#Rename_some_odl-dlux-.2A_features_to_odl-dluxapps-.2A
Change-Id: I4cfeef30c7e83820ef628831c9a2ff6fb2622421
Signed-off-by: Daniel Malachovsky <dmalacho@cisco.com>
Stephen Kitt [Tue, 6 Dec 2016 17:39:51 +0000 (18:39 +0100)]
Action redesign: add regression test for ActionType.learn
Change-Id: I1403cad670c2f0f8c60efbf69da38e2006d211d0
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Vishal Thapar [Wed, 4 Jan 2017 13:32:06 +0000 (19:02 +0530)]
Use batching in interface Delete path
Interface delete path is still not using batching which has performance
issues in a scale setup.
Change-Id: Ifb8f54786405c93bdee3357d625881cf1671255e
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Faseela K [Wed, 4 Jan 2017 15:54:27 +0000 (15:54 +0000)]
Merge "Allow Nicira Extension Actions in BoundServices"
Faseela K [Wed, 4 Jan 2017 15:50:51 +0000 (15:50 +0000)]
Merge "handling idmanager csit intermittent failures"
deepthivv88 [Tue, 3 Jan 2017 11:21:48 +0000 (16:51 +0530)]
Allow Nicira Extension Actions in BoundServices
Change-Id: I91531d00dad320efba3737c3dff661dc57601604
Signed-off-by: deepthivv88 <deepthi.v.v@ericsson.com>
Periyasamy Palanisamy [Mon, 2 Jan 2017 05:04:46 +0000 (10:34 +0530)]
Bug 7450 : suppressing unnecessary warning logs
* check to remove unncessary alivenessmonitor warn logs while packet
in handling
Change-Id: Ie592618dbf2b1db6d1891b111b14db04c01272eb
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Faseela K [Wed, 28 Dec 2016 09:05:14 +0000 (09:05 +0000)]
Merge "Moving interface-manager CLI utils to use cached entries"
Faseela K [Mon, 26 Dec 2016 18:00:15 +0000 (18:00 +0000)]
Merge "Bug 7419 : Ids from id pool exhausted"
Periyasamy Palanisamy [Fri, 23 Dec 2016 04:59:05 +0000 (10:29 +0530)]
handling idmanager csit intermittent failures
* Due to faster allocation/deallocation of ids, release id is not able to
find the idKey in parent pool ds which is being fixed by a latch.
* Fix NPE by throwing IdManagerExcepion if requested id range is not
available
Change-Id: I37312000019b6eb7826f10f19b9fa7b2695101a3
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
Faseela K [Tue, 20 Dec 2016 11:27:25 +0000 (16:57 +0530)]
Tunnel States should not be added for non-northbound configured tunnels
Change-Id: Iec4a8f4af7ffb360618c074445607d5762ff93d6
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Kency Kurian [Wed, 21 Dec 2016 12:27:30 +0000 (17:57 +0530)]
Bug 7419 : Ids from id pool exhausted
Observed that there were many threads waiting to allocate block of ids
from parent pool. Once a thread acquired lock it just used to update its
availableIdHolder with that block without checking if some other thread
has already updated the localPoolCache.
After acquiring lock, a check has been added to know if some other thread
has already updated the availableIdHolder/releaseIdHolder.
Change-Id: Id5550bb2a483d14572803f0e346f2209b9cb9bcc
Signed-off-by: Kency Kurian <kency.kurian@ericsson.com>
Faseela K [Fri, 16 Dec 2016 10:52:48 +0000 (16:22 +0530)]
Moving interface-manager CLI utils to use cached entries
Description : vlan:show and vxlan:show commands in interface-manager were
doing a read from datastore, when the entries are available in cache
actually. Changing the utils to make use of the same, so that the reads
can be faster in scaled scenarios
Change-Id: Ia5f7cd262b892fafd3b74551e8e82a0b3a1b5720
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Vishal Thapar [Thu, 22 Dec 2016 01:58:18 +0000 (01:58 +0000)]
Merge "Handle duplicate tunnel on multiple dpns"
Vishal Thapar [Wed, 21 Dec 2016 07:58:06 +0000 (07:58 +0000)]
Merge "Update spec template"
Vishal Thapar [Tue, 20 Dec 2016 11:10:12 +0000 (16:40 +0530)]
Handle duplicate tunnel on multiple dpns
If same tunnel port name shows up on multiple dpns, it is wrongly assumed
to be a case of VM Migration, which only applies to VM ports. In case we
get any such ports, first check on which DPN user configured it. Retain
interface-state entry for the port that is create on the DPN as configured
by user.
Change-Id: I95dab0a36fd1d033747fd887a4fea4a648da04aa
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Michael Vorburger [Fri, 16 Dec 2016 17:01:18 +0000 (18:01 +0100)]
DataStoreJobCoordinator super minor LOG related performance fix
LOG statements, especially trace(), should not do object creation, and
in this particular case, it seems not necessary to transform the Set to
an Array to a String... Set has a suitable toString, so not sure why
this was thought to be necessary; it should be just fine like this.
PS: Found while staring at DataStoreJobCoordinator for async test
infra.. I still can't complete make sense of what this class really
does! (Thus nor how to sanely make it testable.)
Change-Id: I27593b5ca0a5afe1784be23bc49b3fc231aaee51
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Vishal Thapar [Mon, 19 Dec 2016 15:06:06 +0000 (20:36 +0530)]
Update spec template
Refer: https://git.opendaylight.org/gerrit/#/c/49459/
This also adds ToC to existing specs.
Change-Id: I26322d3b288fd8e422b9ee504d72ccdc6ced24c4
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Michael Vorburger [Tue, 13 Dec 2016 14:32:04 +0000 (15:32 +0100)]
InterfaceManagerConfigurationTest reformatting etc. so no warnings
Change-Id: Ifa17867b095f600f0e14b354d17d5f3d3f5ac36f
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
David Suarez [Sun, 18 Dec 2016 23:10:48 +0000 (23:10 +0000)]
Merge "ResourceBatchingManager code clean up and fixed bad LOG"
Michael Vorburger [Fri, 16 Dec 2016 17:48:14 +0000 (18:48 +0100)]
interfacemanager POM fixed - no *-impl bundles
Not sure what historically happened here, but obviously the
implementation bundles of other modules should never be direct runtime
dependencies.. only their *-api. And for component tests, some impl may
be <scope>test dependencies.
Change-Id: I44afb68fa4db3d795cdb045b05f8c16d4a369d0d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Michael Vorburger [Fri, 16 Dec 2016 17:11:29 +0000 (18:11 +0100)]
ResourceBatchingManager code clean up and fixed bad LOG
PS: Found while staring at ResourceBatchingManager for async test
infra.. I still can't complete make sense of what this class really is
supposed to be good for, or what problem it really solves! (Thus nor how
to sanely make it testable.)
Change-Id: Ica12a5695647d7d228cbd4e885ee38943f91e642
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Faseela K [Fri, 16 Dec 2016 13:53:33 +0000 (13:53 +0000)]
Merge "Cleanup for constants and other stuff."
Michael Vorburger [Thu, 15 Dec 2016 14:12:28 +0000 (15:12 +0100)]
DataStoreJobCoordinator fixed broken LOG.warn of root cause exception
unless there is a very particular reason why this is done in an unusual
weird style, which does not properly show the Exception root cause in
logs (noticed while investigating a failure of the
InterfaceManagerConfigurationTest), it's wrong.
Change-Id: Ic42b4c79b9e7afe69eb4e4e6c633bb899ed3b74c
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
David Suarez [Wed, 30 Nov 2016 10:28:07 +0000 (11:28 +0100)]
Cleanup for constants and other stuff.
- Declare constants properly.
- Log exceptions.
- Local variables.
- Basic formatting.
- Replace Guava's Preconditions by Objects'
- Remove catching of NPE
- Adding more constants to improve readability
There is still a NPE catching that will be removed in a later patch in
which I'll simplify the way in which we call the RPCs (see line
204-206).
Change-Id: I5a2173a39245e7d65469d84cb2a71ef9e21fe1be
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Faseela K [Fri, 16 Dec 2016 08:31:50 +0000 (08:31 +0000)]
Merge "Genius: Neutron Qos DSCP Rule support"
Michael Vorburger [Thu, 15 Dec 2016 16:03:12 +0000 (16:03 +0000)]
Merge "Add CacheBridgeRefEntryListener to provider"
Vishal Thapar [Wed, 14 Dec 2016 07:22:39 +0000 (12:52 +0530)]
Add CacheBridgeRefEntryListener to provider
Change-Id: I9aae46b46ed7da3a820ca97521d95af56bf25385
Signed-off-by: Vishal Thapar <vishal.thapar@ericsson.com>
Faseela K [Thu, 15 Dec 2016 06:08:44 +0000 (11:38 +0530)]
flow entries for multiple subports not getting created
Description :Update of vlanmember interfaces is causing unnecessary churn
of deleting and adding back all the interfaces unnecessarily, even if
there is no attribute which is modified.
Change-Id: Ib2d3be95ce48e2d971b27a8ff6e11d113be532ef
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Faseela K [Wed, 14 Dec 2016 12:08:24 +0000 (12:08 +0000)]
Merge "InterfaceManagerConfigurationTest remove BridgeInterfaceEntryBuilder"
P.Poovizhi [Wed, 7 Dec 2016 12:09:27 +0000 (17:39 +0530)]
Genius: Neutron Qos DSCP Rule support
Adds QoS as a new service and a new table is created for QOS DSCP marking.
DSCP set action to tos field of the IP header is defined newly.
Testing procedure:
1. When a port is associated with QOS policy having DSCP rule. OF is
polpoluated for table 90.
Spec Review Link:
https://git.opendaylight.org/gerrit/#/c/48949/
Result:
- traffic flows with the DSCP value configured in the port.
Change-Id: I52057973bae295ec2e5ea4aec392b4ddf906f1bb
Signed-off-by: P.Poovizhi <poovizhi.p@ericsson.com>
Faseela K [Wed, 14 Dec 2016 07:29:04 +0000 (07:29 +0000)]
Merge "InterfaceManagerTestModule LOG InterfacemgrProvider close() Exceptions"
Michael Vorburger [Tue, 13 Dec 2016 22:22:51 +0000 (23:22 +0100)]
InterfaceManagerTestModule LOG InterfacemgrProvider close() Exceptions
Change-Id: I1099c834024d8deb5e230e7a656bce5ed7c5a6e3
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Stephen Kitt [Tue, 13 Dec 2016 16:26:12 +0000 (17:26 +0100)]
Use platform-specific line separators
This should allow the XtendBeanGenerator-based tests to run on Windows
too.
Change-Id: I5c9e5d79405f8fb9a1f5035ad5a259eb9342614e
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Michael Vorburger [Tue, 13 Dec 2016 14:33:54 +0000 (15:33 +0100)]
InterfaceManagerConfigurationTest remove BridgeInterfaceEntryBuilder
apparently it's un-used - or is this a mistake, and it should be used?
Change-Id: Id241bec009aadc5f7f20c65396c06b1540ea3054
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Miguel Perez [Mon, 12 Dec 2016 17:01:26 +0000 (18:01 +0100)]
Add isIpInSubnet utility API to NwUtil
+ That method is extracted from VpnUtil (netvirt). NwUtil.java is
a more suitable place
Change-Id: Ie999d09aec188fb861254fc985a53a9bd925ac68
Signed-off-by: Miguel Perez <francisco.miguel.perez@ericsson.com>
Michael Vorburger [Mon, 12 Dec 2016 15:08:51 +0000 (15:08 +0000)]
Merge "Fix the JMXAlarmAgent"
Faseela K [Mon, 12 Dec 2016 08:19:03 +0000 (08:19 +0000)]
Merge "Bug 7270 Duplicate remote Mcast mac entry in TOR ."
David Suarez [Wed, 7 Dec 2016 22:17:30 +0000 (23:17 +0100)]
Fix the JMXAlarmAgent
- Declare logger properly.
- More generic interface.
- Fix lifecycle: registration and de-registration.
- Fix logging of exceptions.
- The ItmTunnelEventListener can still work even if the system couldn't
initialize the JMX agent properly.
- Cleanup and formatting.
Change-Id: Iecea96b426c1e7b30868eebe124c34427bc06f9b
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
Tomer Pearl [Sun, 11 Dec 2016 11:31:17 +0000 (13:31 +0200)]
Enhancing DataStoreJobCoordinaor logs
In scale testing I had encountered jobs that took a long time to compelete,
between 1 and 10+ seconds.
Adding a warm log line for every job that takes more then 1sec to complete,
in order to help us in the future to detect possible bottlenecks
or just plain bugs.
Change-Id: Id65e00f925267bf1a30112804322d9b359e308eb
Signed-off-by: Tomer Pearl <tomer.pearl@hpe.com>
Sam Hague [Sat, 10 Dec 2016 02:24:16 +0000 (02:24 +0000)]
Merge "Bug 7315: switch ActionOutput to Uri"
Edwin Anthony [Mon, 5 Dec 2016 09:30:05 +0000 (15:00 +0530)]
Blueprint migration for ITM
change-Id: If4a0a852893e2b6d88ec59e0ac88d0af7c2bb3ce
Signed-off-by: Edwin Anthony <n.edwin.anthony@ericsson.com>
Sam Hague [Thu, 8 Dec 2016 17:59:45 +0000 (17:59 +0000)]
Merge "Introduce DataStoreJobCoordinator counters"
Stephen Kitt [Thu, 8 Dec 2016 09:21:16 +0000 (10:21 +0100)]
Bug 7315: switch ActionOutput to Uri
It turns out the output action port really is a string, interpreted by
Uri; so the appropriate type is probably Uri instead of long (or
perhaps String if we want to avoid Uri in the external API).
Change-Id: Ia55734bb4c7656b18c76de123717ea148e8d882f
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Olga Schukin [Mon, 5 Dec 2016 14:32:03 +0000 (16:32 +0200)]
Introduce DataStoreJobCoordinator counters
The changes related to Bug 7236: System unable to handle high
rate of src mac learning packet-ins
Change-Id: Ie051e7d3bc5aca15a06e3e2aa42a75dc786f252c
Signed-off-by: Olga Schukin <olga.schukin@hpe.com>
Faseela K [Tue, 6 Dec 2016 12:21:57 +0000 (17:51 +0530)]
Cleanup unwanted exceptions in interfacemanager
Description : Fixing NumberFormatExceptions thrown by interfacemanager
when nodeConnectorId comes with reserved port keywords, for eg :
openflow:1:LOCAL instead of Integer port numbers
Change-Id: Ifa5b6d4924fe400345c8ac6ed71c1a7460191e29
Signed-off-by: Faseela K <faseela.k@ericsson.com>