Robert Varga [Tue, 28 Jul 2020 11:22:12 +0000 (13:22 +0200)]
Fix infrautils reference
We want to use a direct reference rather than a property.
Change-Id: I7d17123919c10d8d1e988aa916ae212b5dc0925a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Tue, 28 Jul 2020 11:22:12 +0000 (13:22 +0200)]
Use released version of infrautils
infrautils-1.8.0 has been released, use released version instead
of snapshots.
Change-Id: Ie33387d191e6d65e3e1c6fd6de8f45ff8c6835de
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Mon, 27 Jul 2020 00:00:43 +0000 (02:00 +0200)]
Bump odlparent/yangtools/mdsal/controller
Adopt upstream versions:
- odlparent-7.0.5
- yangtools-5.0.5
- mdsal-6.0.4
- controller-2.0.3
Change-Id: I221bfd55fefc4f412efcf2a605782f29d3e1bde6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Edw7n [Tue, 23 Apr 2019 11:22:59 +0000 (16:52 +0530)]
Shard Status logic in genius
- idmanager mbean waits on shards specified in network constants
- this reference will be available to others only when shard leader
election is done
- any mbean which reads/writes to the data store would have to wait
- ItmProvider and Ifmprovider already wait on idmanager, hence not
included
- other mbeans which are directly or inderectly on such providers/mbeans
do not need idmanager reference explicitly
Change-Id: Ia1c77ebcc978da51a3897e0741ea7570937d16a4
Signed-off-by: Edw7n <n.edwin.anthony@ericsson.com>
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Robert Varga [Tue, 18 Jun 2019 10:04:03 +0000 (12:04 +0200)]
Convert MDSALManager to use NamedLocks
MDSALManager is using synchronized(String.intern()), which makes
the locking domain absolutely opaque. Replace this antipattern
with NamedLocks with appropriate keys.
Change-Id: If9d43362cc23d5e25f979b71f4c024b05d6f8dd5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 29 Jun 2019 11:11:12 +0000 (13:11 +0200)]
Remove CacheUtil and CacheTestUtil
CacheUtil has only a single user in netvirt, which is being removed
and CacheTestUtil has only a single internal user. Remove them both.
Change-Id: I6e7e4311f60f82ec87e13a248657bc89d1bb759d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Thanh Ha [Thu, 26 Mar 2020 19:46:16 +0000 (15:46 -0400)]
Set master branch to Aluminium
Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: Iab2bc8e2dc1fe6166d57045ed517a832dcb75aad
Nishchya Gupta [Fri, 3 Jul 2020 12:26:55 +0000 (17:56 +0530)]
Updating tep:show command.
Removing redundant/unwanted fields.
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>
Change-Id: If0a48884a1be07d784e3a3e225acea49a8814abd
Nishchya Gupta [Fri, 3 Jul 2020 12:09:11 +0000 (17:39 +0530)]
TEP is part of old and new TZ when tz is updated after del-manager
Issue observed:
1. TZA was present and TEPs were part of it.
2. ovsdb connection is removed by running delete-manager command on
switch
3. Change transport-zone of TEPs from TZA to TZC
4. ovsdb connection is set again by running set-manager command on
switch
5. TEPs go under teps-not-hosted since TZC is not yet present. It is
noted here that TEPs are still part of TZA in ITM config DS.
6. TZC is created from northbound
7. TEPs are moved from teps-not-hosted to TZC.
8. In the end, TEPs are part of TZA and TZC both. This is issue.
Fix provided:
When TEPs add notification come again with updated TZC, then
check if same TEP (key: dpid) is already present in any other TZ.
If present, then remove TEP from old TZ and then add TEP into newly
configured TZ.
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>
Change-Id: I98c3c89f70e60676ea9c5f73d90b1a750943e7a5
Chetan Arakere Gowdru [Tue, 5 May 2020 08:02:03 +0000 (13:32 +0530)]
L2GW Junit failure
Description:
Th Junit case as adding DeviceTeps for External tunnels only into
default-transport-zone and was querying in TZA resulting in read
failure.
Since the current RPC only have ip and node-id as inputs, Changes are
done in addL2GwMlagDevice() API to add the given Teps in all available
transportzone.
Signed-off-by: Chetan Arakere Gowdru <chetan.arakere@altencalsoftlabs.com>
Change-Id: I75db7338b696471db653417bd3124bc742e5e85d
Signed-off-by: Chetan Arakere Gowdru <chetan.arakere@altencalsoftlabs.com>
Robert Varga [Fri, 24 Jul 2020 11:40:24 +0000 (13:40 +0200)]
Migrate ListenableFutures.addErrorLogging() callers
ListenableFutures.addErrorLogging() has been migrated to
LoggingFutures, use these methods from their new home.
Change-Id: Ic23e8d67be04e1b1eb864c6307f8ffb6fc9a95a7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Mon, 28 Oct 2019 17:07:38 +0000 (18:07 +0100)]
Turn ActionableResource into an abstract class
This prevents outside instantiation of the interface, so that
we can hide the details of dispatch from outside world, making
it an implementation detail.
JIRA: GENIUS-281
Change-Id: I5fa6208515501bc7f8f915544da69ac47cd3a8cc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Mon, 28 Oct 2019 16:50:49 +0000 (17:50 +0100)]
Hide ActionableResourceImpl
With all users migrated to ActionableResources, we can safely hide
this implementation class and remove the now-unused setters, making
things neatly final and predictable.
JIRA: GENIUS-281
Change-Id: I00caad5574526369b62d50f018d1c95423f370ff
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Nishchya Gupta [Thu, 23 Jul 2020 08:50:36 +0000 (14:20 +0530)]
Unblock the Aluminium Jenkins Verified Build Pass
Issue
=====
Post Openflowplugin https://git.opendaylight.org/gerrit/c/openflowplugin/+/90390
merge, genius build is starting to get, failed with below key related issues.
/w/workspace/genius-distribution-check-aluminium/genius/interfacemanager/
interfacemanager-impl/src/main/java/org/opendaylterfacemanager/pmcounters/NodeConnectorStatsImpl.java:cannot find symbol
13:24:17 symbol: class FlowAndStatisticsMapListKey
13:24:17 location: package org.opendaylight.yang.gen.v1.urn.opendaylight.flow.statistics.rev130819.flow.and.statistics.map.list
Signed-off-by: Nishchya Gupta <nishchyag@altencalsoftlabs.com>
Change-Id: I0b31725a3871f178ae2f1256670442a601ecf8bf
Robert Varga [Tue, 26 May 2020 18:00:26 +0000 (20:00 +0200)]
Use Assert.assertThrows()
org.opendaylight.infrautils.testutils.Asserts.assertThrows() has been
deprecated, as we now have a more modern JUnit, which provides this
facility. Migrate all users and remove dependency on
infrautils-testutils where is it no longer required.
Change-Id: If7bb59bf45ad290fb79747105c690af4a684dfdb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Mon, 22 Jun 2020 20:13:15 +0000 (22:13 +0200)]
Use a range for MDSAL feature
odl-mdsal-model-rfc7224 should be referred to via a range to ease
MRI project upgrades, do that now.
Change-Id: I1bee9318e99a56f7ae7bd7c5630b1037615d6cb3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Thu, 11 Jun 2020 21:54:29 +0000 (23:54 +0200)]
Integrate controller-2.0.2
This bumps versions to the following:
- odlparent-7.0.3
- yangtools-5.0.3
- mdsal-6.0.2
- controller-2.0.2
Change-Id: Ida57ff1454c9298715de0a3dd2653b624cba5622
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
R P Karthika [Tue, 2 Jun 2020 07:58:35 +0000 (13:28 +0530)]
Bug fixes after MRI version bump
Change-Id: I9fc2351cf37877427c698fdb6a2d38a0ce5b02f4
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
Robert Varga [Tue, 26 May 2020 17:27:47 +0000 (19:27 +0200)]
Clean up mis-migration
This is a follow-up to
8a228e4974b9e6909df0a4aa7866eadfee9e438c,
correcting a minor change in semantics.
Change-Id: I4478c1df8b12c7a798c73b9707cf903705cc1587
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sun, 31 May 2020 09:26:17 +0000 (11:26 +0200)]
Integrate controller-2.0.1
This bumps versions to the following:
- odlparent-7.0.2
- yangtools-5.0.2
- mdsal-6.0.1
- controller-2.0.1
Also fix up MD-SAL API migration.
JIRA: GENIUS-279
Change-Id: Iac4963fa4d833481e34e9683450cb61a339df3d7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Tue, 26 May 2020 17:33:01 +0000 (19:33 +0200)]
Remove commented-out dependencies
Dependencies on controller artifacts are commented out, remove
them to prevent them from coming back.
Change-Id: Ibbfecf1ae7f185be650eba78541b36d9786ef827
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Ankit Jain [Fri, 29 May 2020 04:54:03 +0000 (10:24 +0530)]
ignore test testMultithreadedIdAllocationFromReleaseIds of Idmanager
This test is failing and causing build faliure for other patches.
It seems like the issue with test framework and required investigation.
For now ignoring the test so that build can go through for other
patches.
Signed-off-by: Ankit Jain <ankit.j.jain@ericsson.com>
Change-Id: Id613ac771b6d8774f26c4902c8a3a24bb6cfe331
Nidhi Adhvaryu [Wed, 6 May 2020 10:21:48 +0000 (15:51 +0530)]
MRI version bump for Aluminium
This bumps versions to the following:
- odlparent-7.0.1
- yangtools-5.0.1
- mdsal-6.0.0
- controller-2.0.0
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
Signed-off-by: Nidhi Adhvaryu <nidhi.adhvaryu@ericsson.com>
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
Change-Id: I508ba70555f52da12439a3963d642fc60cfe7775
R P Karthika [Wed, 29 Apr 2020 09:56:46 +0000 (15:26 +0530)]
Fix javadoc failures in genius
Fixing javadoc failures post MDSAL-API migration.
Change-Id: I9e6c5408c5f132006bc0c737a0980ac0fe6425c8
Signed-off-by: Nidhi Adhvaryu <nidhi.adhvaryu@ericsson.com>
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
Chetan Arakere Gowdru [Tue, 14 Apr 2020 11:15:09 +0000 (16:45 +0530)]
InfraUtil JobCordinator Changes.
Desc:
https://git.opendaylight.org/gerrit/c/infrautils/+/89057
Change-Id: I76744bf161cc6d1edadf013f2310f6aa6bb1efae
Signed-off-by: Chetan Arakere Gowdru <chetan.arakere@altencalsoftlabs.com>
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
R P Karthika [Thu, 9 Apr 2020 18:30:00 +0000 (00:00 +0530)]
MDSAL-API Migration
1)Change Futures to FluentFuture
- Remove ReadFailedException
- Replace TrnasactionCommitfailedException with Execution
and Interrupted exception
- submit() -> commit()
- checkedGet() -> get()
2)Change org.opendaylight.controller.md.sal package to org.opendaylight.mdsal
- Primarily databroker, all transactions
- ReadOnlyTransaction -> ReadTransaction
3)Change com.google.common.base.Optional to java.util.Optional
4)Change serviceutils.tools.mdsal.listener to serviceutils.tools.listener
Change-Id: I91f6178e1ef363d989cd19e2ab7048001e811ed7
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
Robert Varga [Wed, 25 Mar 2020 21:54:37 +0000 (22:54 +0100)]
Use Magnesium controller
As part of controller transition to being MRI, downgrade the version
used to Magnesium release. This will be bumped to Aluminium proper
during the MRI window.
Change-Id: I84d1c7cdeaf8ae9bd3b00070c434e8a27ba23665
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Hema Gopalakrishnan [Mon, 30 Mar 2020 09:13:07 +0000 (09:13 +0000)]
Revert "doesExists() utility method"
This reverts commit
8b0a3fd63b457589f0f869897c46c696994c618c.
Reason for revert: Remove Deprecated APIs
Signed-off-by: Hema Gopalakrishnan <hema.gopalkrishnan@ericsson.com>
Change-Id: I60b0723c3e927ffe65ed0f55f05e2241bb4c51b1
Shashidhar Raja [Thu, 5 Mar 2020 12:22:04 +0000 (17:52 +0530)]
doesExists() utility method
Common utility method doesExists() added to check whether any DS entry
exists or not in SingleTransactionDataBroker
Change-Id: I92a8f5bda8d4d924be6a820e6a18acd598a40dae
Signed-off-by: Shashidhar Raja <shashidharr@altencalsoftlabs.com>
jenkins-releng [Sat, 22 Feb 2020 04:30:15 +0000 (04:30 +0000)]
Bump versions by x.y.(z+1)
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Change-Id: I897b2682f347c8e1eb9f03e211882a1c008df499
Robert Varga [Mon, 28 Oct 2019 16:20:05 +0000 (17:20 +0100)]
Encapsulate ActionableResourceImpl
This provides ActionableResources for use by external users, which
enforce well-formedness off ActionableResource. This allows us to
eliminate the need for ActionableResource.getKey(), which is not
used anywhere aside from ResourceBatchingManager, where we can
simply default to ActionableResource.toString().
This eliminates the need to eagerly invoke InstanceIdentifier.toString()
just to enqueue a task and makes sure this is not done unless the
resource is constructed using deprecated interfaces.
JIRA: GENIUS-281
Change-Id: I48691f9d3dbcbf2715f7eddaf775b0e9a37fd7b4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Nidhi Adhvaryu [Mon, 4 Nov 2019 09:15:49 +0000 (14:45 +0530)]
GENIUS-282 Resolving genius yang model errors
Some errors that appeared while testing the Yang models,
https://logs.opendaylight.org/releng/vex-yul-odl-jenkins-1/distribution-merge-managed-magnesium/657/opendaylight-models/genius/
which is being addressed in this patch
Signed-off-by: Nidhi Adhvaryu <nidhi.adhvaryu@ericsson.com>
Change-Id: Id0b3f212983876e7908f6b4ee142d91c538d6d59
Hema Gopalakrishnan [Tue, 11 Feb 2020 11:36:14 +0000 (11:36 +0000)]
Revert "Packet duplication due to forwarding from HWvTEP tunnel to ITM"
This reverts commit
a1b8a5918a7f9319214d597ffe85e212004ae71c.
Change-Id: I377f5e67a311a18493baa606ff926a05ef3fb6cf
Signed-off-by: Hema Gopalakrishnan <hema.gopalkrishnan@ericsson.com>
Amitesh Soni [Tue, 14 Jan 2020 07:29:31 +0000 (12:59 +0530)]
Packet duplication due to forwarding from HWvTEP tunnel to ITM
Fix details:
--------------
ELAN_DMAC_TABLE = 51 has been refactored into two
ELAN_LOCAL_DMAC_TABLE = 51 and ELAN_REMOTE_DMAC_TABLE=52
Signed-off-by: Amitesh Soni <amitesh.soni@ericsson.com>
Change-Id: I6ad9d4267ab6b48c28120e7631917ec3038cea9a
Robert Varga [Sun, 5 Jan 2020 17:56:11 +0000 (18:56 +0100)]
Fix odl-genius-api
Multiple things done here:
- use artifacts to import proper upstream versions
- add dependency on odl-openflowplugin-eric-extensions
to prevent duplicate packaging of Ericsson extensions
Change-Id: I79f51be20559798a0539a65e1cf924bbc32f2278
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sun, 5 Jan 2020 17:23:50 +0000 (18:23 +0100)]
Fix odl-genius
Multiple things done to clean up packaging:
- use properly-imported artifacts poms from upstream to eliminate
properties
- add proper dependency on odl-serviceutils-listener to eliminate
duplicate packaging
- eliminate unneeded explicit dependencies, they are pulled in as
needed
- cloudscaler dependency on javax.annotation is optional, as it is
not used in Karaf and should not be pulled into the feature
Change-Id: I1313695530bd7354c2e8f75d45ee8f99525fcb05
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Thu, 16 Jan 2020 21:16:32 +0000 (22:16 +0100)]
Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9
Pick up latest fixes and upgrades.
Change-Id: I8680b14deaec92583bb89a9a10088fc3d34b5b51
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Chandra Shekar S [Thu, 2 Jan 2020 05:53:51 +0000 (11:23 +0530)]
Change the log level to debug
JIRA: NETVIRT-1656
Signed-off-by: Chandra Shekar S <chandra.shekar.s@ericsson.com>
Change-Id: Ic7e501cb82046d2d5077d047ac92956fed66cd17
Robert Varga [Mon, 16 Dec 2019 21:08:16 +0000 (22:08 +0100)]
Remove ARIES-1923 workaround
We now have proper monkey-patch in place and can therefore remove
this workaround.
Change-Id: Ia69518cb72aaa99de9f4eb83c6a295d9715928a0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Thu, 12 Dec 2019 07:07:18 +0000 (08:07 +0100)]
Bump odlparent/yangtools/mdsal to 6.0.3/4.0.3/5.0.6
Pick up the latest upgrades and fixes.
Change-Id: I127543de2d469e3aa626a427450e1db2a37d0dfb
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sun, 15 Sep 2019 18:36:41 +0000 (20:36 +0200)]
Expose TypedReadTransaction.exists(InstanceIdentifier)
This operation was not exposed at the binding layer when these
utilities were conceived and thus is lacking here, even when they
are available as part of MDSAL-389.
The MD-SAL version of TypedReadTransaction has been updated back
then, this patch backports these changes to legacy Genius APIs.
Also converts a few callers to use this capability, potentially
improving performance.
JIRA: MDSAL-389
Change-Id: Ieb870a49c8239ee74e697d7d36dea431b24362ae
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit
7144e4db6a02daf37f8ba055745a359d0a80c21e)
Robert Varga [Fri, 11 Oct 2019 23:44:36 +0000 (01:44 +0200)]
Optimize String formatting
In some hot paths we end up calling String.format() where a simple
concatenation would suffice. Fix that up.
Change-Id: Ib173fc0599899f40012410b75e243dd3873019a3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Mon, 28 Oct 2019 13:44:58 +0000 (14:44 +0100)]
Remove ActionableResource setters
This removes previously-deprecated setters, so that the resource
itself has an immutable surface.
JIRA: GENIUS-281
Change-Id: Ic621cb4afb5567442b29a7c69fdaf0bcb7c5430e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Anushka Bhandari [Tue, 20 Aug 2019 16:53:10 +0000 (22:23 +0530)]
Update docs header to Magnesium in the master
Should be Magnesium on master and Sodium on stable/sodium.
Signed-off-by: Anushka Bhandari <anushka16134@iiitd.ac.in>
Change-Id: I6aa4bcd7742ee5efcdf44ad4f1511f952278e196
Nidhi Adhvaryu [Mon, 18 Nov 2019 06:58:08 +0000 (12:28 +0530)]
Cleaning Oper DS during scalein
- to key tunnel remove events on tunnel name
- uses tombstone api
Signed-off-by: Nidhi Adhvaryu <nidhi.adhvaryu@ericsson.com>
Change-Id: I46470cc20c477c2301e20d2da480a2746d949ee3
Robert Varga [Wed, 6 Nov 2019 16:35:21 +0000 (17:35 +0100)]
Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4
This picks up latest upgrades and fixes.
Change-Id: I8db358b0af81394a561071d98bffea168dda47cc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
xcheara [Mon, 21 Oct 2019 12:18:14 +0000 (17:48 +0530)]
Addessing NPE and UnsupportedOperationException in Genius
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Change-Id: Ie71bd0096968057574a018a04cec0da18b0196c4
Robert Varga [Mon, 28 Oct 2019 13:33:32 +0000 (14:33 +0100)]
Deprecate ActionableResource setters
ActionableResource should not allow modification of parameters
once built. Deprecate setters in the base interface, retaining
them in the implementation.
JIRA: GENIUS-281
Change-Id: Iaf64bb20b67e3d39cfb769bffad0ddbd669cf169
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Wed, 30 Oct 2019 12:24:19 +0000 (13:24 +0100)]
Downgrate aggregator's javadoc plugin
It seems javadoc-verify/merge jobs are affected by some bug in
maven-javadoc-plugin -- it would filter iana-if-type dependency,
leading to a a failure in javadoc:aggregate.
Change-Id: I8efd03b24cffd509de7f44c9ab1c103c1ee50bad
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
xcheara [Sun, 6 Oct 2019 05:09:23 +0000 (10:39 +0530)]
Bump version odlparent->6.0.0,mdsal->5.0.3
This adopts new version of MRI projects and adjusts for the changes
therein. Most notably DPs/DPNs are identified by Uint64, not
BigInteger.
Some drive-by cleanups are performed, mostly to eliminate valid
Eclipse complaints.
Change-Id: Ib73c963d515f55f0865775584de3632e3731820c
Signed-off-by: xcheara <chetan.arakere@altencalsoftlabs.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 14 Sep 2019 16:34:29 +0000 (18:34 +0200)]
Bump mdsal to 4.0.6
This picks up the latest fixes.
Change-Id: I8ec178ca726c1df504efc4f19d2c9c80965d93be
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 14 Sep 2019 16:34:06 +0000 (18:34 +0200)]
Bump yangtools to 3.0.5
This picks up the latest fixes.
Change-Id: Ic37a0d36367f6d89768bec2b92618b4b55aa844f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 14 Sep 2019 16:33:52 +0000 (18:33 +0200)]
Bump odlparent to 5.0.2
This bumps odlparent to pick up latest references.
Change-Id: Ica16c98bd6eeeb2530bdc9cee4bb7f198e4a6ab0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 14 Sep 2019 17:15:05 +0000 (19:15 +0200)]
Fix checkstyle
This fixes up violations reported by upgraded checkstyle.
Change-Id: Ic0db886bc974a31692b228667c93375a958c3618
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Amitesh Soni [Tue, 6 Aug 2019 06:22:25 +0000 (11:52 +0530)]
Retry on DS if read fails
Issue: During the initialization of ID Manager, ID Manager reads
ID pools DS. If this DS read fails, ID Manager does not come up
and other applications dependent on ID Manager do not come up.
Fix: Fix is made to retry on DS until read succeeds.
Change-Id: I627846e1c84f75461f43d9248e98e561b1ffd5bd
Signed-off-by: Amitesh Soni <amitesh.soni@ericsson.com>
Faseela K [Thu, 18 Jul 2019 07:50:13 +0000 (13:20 +0530)]
Fix minor typo in the csit document
Change-Id: Ic832ecd4c8817d8fd4f810d31183d1f2d574e3af
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Ankit Jain [Mon, 3 Sep 2018 09:47:28 +0000 (15:17 +0530)]
GENIUS-202 GENIUS-257 Retry for Unlock() & IdManager calls tryLock()
** LockManager's unlock() method now retries 10 times in case of
exception is thrown. The interval between retries is 1 sec.
** LockManager's lock() method now eternally retries in case of
DataStoreUnavailableException and OptimisticLockFailedException
otherwise it returns FailedRpcResult.
** IdManager no longer calls LockManager's lock() api. It now calls
LockManager's tryLock() api which internally retries for 10 times.
Idmanager returns failure if lock is not acquired after specified lock
attempts. After this change, it is the responsibility of the consumers of
Idmanager to handle these failures.
Change-Id: Ic230e2e4edce4f95ee461ef22ff895368af9663d
Signed-off-by: Ankit Jain <ankit.j.jain@ericsson.com>
Signed-off-by: Amitesh Soni <amitesh.soni@ericsson.com>
Amitesh Soni [Thu, 18 Jul 2019 10:59:56 +0000 (16:29 +0530)]
GENIUS-171 IdManager's releaseId() returns id being released
The releaseId() RPC now returns the release-id upon successful release
so that applications can use the same for cleaning-up their id which
they might have cached in their states.
Signed-off-by: Amitesh Soni <amitesh.soni@ericsson.com>
Change-Id: I811a2b1b4daf7124162b35c5d88159b6a917f8c2
manojna v [Fri, 23 Aug 2019 07:17:39 +0000 (12:47 +0530)]
Adding test classes for scale-in/scale-out functionality
Change-Id: Ie038ca19a4199316b8a5c432e20295cd58e967a1
Signed-off-by: manojna v <manojna.vijayakrishna@ericsson.com>
Nidhi [Mon, 15 Jul 2019 11:32:37 +0000 (17:02 +0530)]
GENIUS-254 GENIUS-119 GRE and L2GW tunnel missing
There is a race condition between creation of tun port and
bridge ref, port not created. When we are trying to add bridge
to ovsdb from SB, NB is trying to add port for same bridge.
during which NB will check DS for bridge entry,which will be
empty as SB is trying to add it in ovsdb at the same time.
So we are adding eventcallback to synchronize both events.
Change-Id: I7f6c0583d90d5aa91e0ae063123680bdece08998
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
Nidhi Adhvaryu [Mon, 12 Aug 2019 11:54:11 +0000 (17:24 +0530)]
GENIUS-251 Documentation for genius event logging using log4j2
Signed-off-by: Nidhi Adhvaryu <nidhi.adhvaryu@ericsson.com>
Change-Id: I5b7ed401482dfda550b450d9beb1e425292e7923
Faseela K [Fri, 2 Aug 2019 07:30:10 +0000 (07:30 +0000)]
"Adding scale-in/scale-out functionality""
This patch has to go in along with the corresponding netvirt patch.
Else genius merge jobs will start failing
Change-Id: I654fbdf30a7e1b5565d238878a5eca7a96960de9
Signed-off-by: Faseela K <faseela.k@ericsson.com>
nithi thomas [Tue, 2 Apr 2019 08:42:21 +0000 (14:12 +0530)]
NETVIRT-1586
OVS Based NA Responder for IPv6 default Gateway
Change-Id: Idd52407babc7bbd9bc45da1910cb0767c4511af3
Signed-off-by: nithi thomas <nithi.t@altencalsoftlabs.com>
Manu B [Tue, 4 Sep 2018 07:29:26 +0000 (12:59 +0530)]
Grep not working for Karaf commands
This cover changes for idmanager.
The fix is to replace Session.getConsole() with System.print().
Change-Id: Ic85a495ae5386b996942b47c7a6b75f281dd3f3c
Signed-off-by: Manu B <amitesh.soni@ericsson.com>
Robert Varga [Thu, 15 Aug 2019 08:42:16 +0000 (10:42 +0200)]
Bump mdsal to 4.0.4
This brings in latest fixes.
Change-Id: I59f89ef4fe76ef16fcdc9b59f8d9b50aa0849847
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
jenkins-releng [Mon, 12 Aug 2019 16:14:12 +0000 (16:14 +0000)]
Bump versions by x.y.(z+1)
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Change-Id: I04a508da4a70d18d5b297b98fbedfed89e0df6d5
Nidhi [Tue, 23 Jul 2019 12:03:35 +0000 (17:33 +0530)]
GENIUS-251: Event Logger for Genius
Separate Logger to capture events for debugging.
Change-Id: I2410d8b336ab83ba2ec186ebcdbfb77e915dbf1b
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
Nidhi [Mon, 15 Jul 2019 09:44:54 +0000 (15:14 +0530)]
ITM recovery is not working
Tunnels were not getting recovered and what we found is
Itm Direct Tunnel support was missing for tep and
transportzone recovery. So this patch is adding that.
Change-Id: Ie0c4df35065384e26dce9d271d229868daa4c8ce
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
Dimple Jain [Thu, 25 Jul 2019 06:17:10 +0000 (11:47 +0530)]
Selective BFD: Fix to enable BFD on both
Source and Destination DPN.
Change-Id: I18ac0ae566c89b9e5f7055d30a36e440abacba31
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
Naveen Kumar Verma [Wed, 31 Jul 2019 06:52:55 +0000 (12:22 +0530)]
Enable interface-manager-stats polling by default
Currently a flag in genius-ifm-config.xlm which controls
the Performance counter is disabled by default.
Making this flag enable by default will make these
counters available to be queried using mbean by default.
As the polling interval is 15 mintues, it should not hurt performance.
Change-Id: I4d904dcacbc13e42e695a1f9d37f20a61cf6eafa
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
Dimple Jain [Thu, 25 Jul 2019 05:42:48 +0000 (11:12 +0530)]
VNF BFD packets are dropped by OVS when Tunnel monitoring
BFD is turned ON.
Adding BFD MAC filter on tunnels to fix this issue.
Change-Id: I5bbb13ac3aae82bffbebdef7e373a10fea9418bc
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
Robert Varga [Thu, 18 Jul 2019 16:19:05 +0000 (18:19 +0200)]
Bump mdsal to 4.0.3
This bumps mdsal to 4.0.3, bringing in latest fixes and improvements.
Change-Id: Ie8e03aab1f31edf80bc9f1d7896c334c9d83be8d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Thu, 18 Jul 2019 16:15:41 +0000 (18:15 +0200)]
Bump yangtools to 3.0.4
This picks up latest fixes and improvements.
Change-Id: I03201927d23762acf641c1ab0ff3419ba8b94969
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Thu, 18 Jul 2019 16:14:18 +0000 (18:14 +0200)]
Bump odlparent to 5.0.1
This picks up the latest upgrades.
Change-Id: If6006a94197839a9718cd5eae16b0a025913af93
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Naveen Kumar Verma [Tue, 30 Jul 2019 07:04:14 +0000 (12:34 +0530)]
Issue: uuid was not coming from interface manager as portNameCache
which pmcounters where using was not getting update.
Solution: EOS of INTERFACE_CONFIG was preventing the
cache to be modified on other Cics. Now adding and removing entries into
portNameCache as moved before the EOS check.
Change-Id: Ia64d56e5dd59ef4044617b77139c35a1ee2a77ea
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
Faseela K [Fri, 2 Aug 2019 06:19:32 +0000 (06:19 +0000)]
Revert "Adding scale-in/scale-out functionality"
This reverts commit
1e7bda5287304ba3a5dde948b5fe38124b56349b.
Change-Id: I6d5c7df57439bacf80f4c540c34dcdd75acdfa1b
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Nobin Mathew [Wed, 17 Apr 2019 10:03:35 +0000 (15:33 +0530)]
Handle Node Repair in IFM
Handle Node Repair in IFM, check for UNKNOWN state.
Change-Id: I48a0e66d396b9072e897f96a71108fc1d4ea8fc6
Signed-off-by: Nobin Mathew <nobin.mathew@ericsson.com>
manojna v [Thu, 25 Jul 2019 11:53:06 +0000 (17:23 +0530)]
Adding scale-in/scale-out functionality
Change-Id: Ic28ed5c32a3d26cfc608cd9c0ad87e37ad1b552a
Signed-off-by: manojna v <manojna.vijayakrishna@ericsson.com>
Apurba Mukherjee [Thu, 4 Jul 2019 06:42:18 +0000 (12:12 +0530)]
GENIUS-266: Deletion of OF-Port along with status
- takes care of deleting of-Port
- takes case of deleting tunnels to and from this port
- takes care of deleting ingress and egress flows
Change-Id: I8d42b02134d163e5ebf6b18aa7734cc65237ab33
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
Apurba Mukherjee [Fri, 14 Jun 2019 10:46:09 +0000 (16:16 +0530)]
JIRA: GENIUS-271-ITM Tunnel state operational DS population
The changes are made to populate the tunnel states for OF tunnels
Change-Id: If77d0311bccc8fb67720d927344adb5c05cdac63
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
Nidhi [Mon, 15 Jul 2019 12:11:23 +0000 (17:41 +0530)]
Fix for NPE in ArpUtil
Change-Id: I9411a1576cf2d74950b685a6bd2d10e5bb3ea0ab
Signed-off-by: Nidhi <nidhi.adhvaryu@ericsson.com>
Dimple Jain [Thu, 25 Jul 2019 05:59:05 +0000 (11:29 +0530)]
Fixing NullPointerException.
ITM was not processing the parked NC event due to a wrong lookup.
As a result all the parked NC events were lost.
Change-Id: Ie6ca0a603c83de778c6741abdeefe3b345a48555
Signed-off-by: Dimple Jain <dimple.jain@ericsson.com>
Robert Varga [Sat, 29 Jun 2019 10:01:58 +0000 (12:01 +0200)]
Remove JSR305 dependencies
JSR305 is not used anywhere, remove it from dependencies.
Change-Id: Ia8129823dc2ac78d967ca23a2502e799d5148d31
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 29 Jun 2019 10:18:54 +0000 (12:18 +0200)]
Remove DataStoreCache
Caches based on this class have been eradicated, with the only
user being CLI trying to find it. Remove the class and references
to it.
Change-Id: I9e4848ae9e491d04d9a2fa771fc8cdf5b3545e2f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Naveen Kumar Verma [Wed, 24 Jul 2019 10:38:59 +0000 (16:08 +0530)]
Skipping tunnel ports when colleting port stats...
Incomplete statistics comes for Tunnel ports from ovs:
"bytes_received": "-1",
"bytes_sent": "-1",
"duration": "65373",
"packets_received": "-1",
"packets_received_drop": "-1",
"packets_received_error": "-1",
"packets_received_on_tunnel": "0",
"packets_sent": "-1",
"packets_sent_on_tunnel": "0",
"port_id": "23",
"tenant_id": "
3ecca858-98cd-4923-9011-
e8c750f7b9a8",
"uuid": "
9e94558d-9dbf-4a82-88df-
b99d8fe056fa"
Change-Id: I116020f735bc341864b3fd80ee6d866deac155f8
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
Naveen Kumar Verma [Wed, 10 Jul 2019 05:03:43 +0000 (10:33 +0530)]
Move Pm Counter processing to new thread
Porcessing of RPC response has been moved to a newer thread.
Change-Id: I2fcf385847924f164d6799d59ea390b2acdcd059
Signed-off-by: Naveen Kumar Verma <naveen.kumar.verma@ericsson.com>
Apurba Mukherjee [Mon, 29 Jul 2019 12:29:08 +0000 (17:59 +0530)]
Issue: Table-0 to Table-36 flow was missing
As part of ITM Yang Models Cleanup, internal flag was removed which
caused the issue. The internal flag is brought back to fix the issue.
Change-Id: Ia069fc52ecbd3223fa7ca7c4713ffb475fb3a353
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
R P Karthika [Mon, 8 Jul 2019 12:56:51 +0000 (18:26 +0530)]
Make Batching size and interval as configurable parameters
in ITM
Define batch size and interval in itm-config.yang and use that instead
of using it from system.properties(),so it can be configured if needed.
Change-Id: I6f292bad3323fd4b5cd1d1be6cb6e575e41c3c1a
Signed-off-by: R P Karthika <r.p.karthika@ericsson.com>
Amitesh Soni [Fri, 19 Jul 2019 07:10:49 +0000 (12:40 +0530)]
Id allocation failing
Issue
=====
While reading data store if data is not present in the DS,
"ExpectedDataObjectNotFoundException" exception is thrown.
Since this exception is not handled, ID allocation is failing.
Fix
====
Added try-catch block to handle the "ReadFailedException" exception
so that the code execution does not stop if read fails.
https://gerrit.ericsson.se/#/c/
4354671/
Change-Id: I00423597e1681085b1fbcd187e11e5f2dd7582cd
Signed-off-by: Amitesh Soni <amitesh.soni@ericsson.com>
Faseela K [Wed, 24 Jul 2019 10:57:37 +0000 (10:57 +0000)]
TSC-181: ITM Yang Models Cleanup
Merge issues are fixed now, hence restoring the actual change
Change-Id: Ie774a5a2c09f901cd4ff58c732acf73effc36827
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Faseela K [Wed, 24 Jul 2019 09:45:40 +0000 (15:15 +0530)]
Ignore flaky ITM test
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Change-Id: I51654fa1b348308215e3c47692729c15fd5caac3
Faseela K [Tue, 23 Jul 2019 09:51:21 +0000 (09:51 +0000)]
Revert "TSC-181: ITM Yang Models Cleanup"
This reverts commit
f32af7aad9875d8aa281ff711f86945c18685318.
Change-Id: I89c0ea7a8b8b5e365d540a8d1987f3a0b191eb76
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Apurba Mukherjee [Thu, 2 May 2019 05:42:03 +0000 (11:12 +0530)]
TSC-181: ITM Yang Models Cleanup
Change-Id: I2d4e79e78c562572d9d91598b489f563eb1f6484
Signed-off-by: Apurba Mukherjee <apurba.mukherjee@ericsson.com>
manojna v [Thu, 20 Jun 2019 10:21:49 +0000 (15:51 +0530)]
Adding compute node scale-in scale-out spec
Change-Id: I00058d5eeac8256e79dcce9a5b2c4502cec826cd
Signed-off-by: manojna v <manojna.vijayakrishna@ericsson.com>
Robert Varga [Sat, 29 Jun 2019 07:38:05 +0000 (09:38 +0200)]
Remove use of powermock from mdsalutil-impl
powermock is not really used in these tests, remove it, allowing
us to simply upgrade mockito.
Change-Id: I26eca39285e2bcca7193c837e69c347c4b5cc878
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Tue, 18 Jun 2019 11:09:16 +0000 (13:09 +0200)]
Migrate to use LoggingFutures
JdkFutures are not taking advantage of various known implementations,
which LoggingFutures does. This migrates code to use new equivalent
method.
Change-Id: Ifd8ae59f82cdb2eb8ea70783bc81288139e5c3d5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Tue, 18 Jun 2019 09:40:13 +0000 (11:40 +0200)]
Make private MDSALManager methods static
These methods are not touching any state, incurring Eclipse warnings.
Fix those warnings by making the methods static.
Change-Id: I0649ee508fffe6d2b83ea55710e405635921812c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Sat, 29 Jun 2019 07:43:59 +0000 (09:43 +0200)]
Remove powermock version overrides
odlparent is providing version 2.0.0, hence there is no point
in us overriding versions (and specifying some artifacts).
Change-Id: Ia006b48a772a98ae9cc8e6fb7abc00dccaba0295
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Wed, 26 Jun 2019 10:30:00 +0000 (12:30 +0200)]
Declare api artifacts in genius-artifacts
This reduces the number of project.version declarations throughout
the project, improving consistency.
Change-Id: Id039b72d27da6d07315cc83d9d3da5d4e1abb6b5
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Robert Varga [Tue, 25 Jun 2019 11:37:33 +0000 (13:37 +0200)]
Import genius-artifacts from binding-parent
This removes duplicate imports and allows us to share versions
at a later point.
Change-Id: Ibd8d2f717975affeb61d51116a1d67c5255d49e7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>