Gobinath [Fri, 24 Nov 2017 06:31:22 +0000 (12:01 +0530)]
System ready integration for OFP
The openflowplugin will start switch connections only on the system
ready event triggered by the infrautils.
Change-Id: Ia60b36cb2446ed92d58d3507d2de9524d6efbc00
Signed-off-by: gobinath <gobinath@ericsson.com>
Anil Vishnoi [Thu, 30 Nov 2017 10:46:03 +0000 (10:46 +0000)]
Merge "Clean up in module reconciliation-framework"
D Arunprakash [Fri, 17 Nov 2017 08:39:02 +0000 (14:09 +0530)]
Spec: Reconciliation framework
Change-Id: I80c8a6ce4b073f59d69ecd01558ce024adce0748
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
David Suarez [Mon, 20 Nov 2017 15:10:45 +0000 (16:10 +0100)]
Clean up in module reconciliation-framework
- Remove some deprecated
- Make some variables final
- Other minor clean up
Change-Id: Ib87f59bf31009d5222c7512fa5d2d29a14454fef
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Anil Vishnoi [Fri, 17 Nov 2017 08:05:46 +0000 (08:05 +0000)]
Merge "Revert "Bug 8902 : Spec for Reconciliation Framework""
D Arunprakash [Fri, 17 Nov 2017 03:51:00 +0000 (09:21 +0530)]
Remove the rst file and add later with proper formatting
Change-Id: Ib92caeb6c7e657b51e76dd169c5569e2bb45bbe7
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
Anil Belur [Fri, 17 Nov 2017 01:56:03 +0000 (11:56 +1000)]
Revert "Bug 8902 : Spec for Reconciliation Framework"
Patch #60283 is preventing patch #65556 from being verified
This reverts commit
f1e058a660cf98d1411ba763c6b0989e2439fabc.
Change-Id: I11c277ae85a366ed83dd240586d5ef8e2ca60d0e
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Anil Vishnoi [Wed, 15 Nov 2017 20:40:06 +0000 (20:40 +0000)]
Merge "Add Docs for Openflowplugin"
Anil Vishnoi [Wed, 15 Nov 2017 20:40:01 +0000 (20:40 +0000)]
Merge "Bug 8902 : Spec for Reconciliation Framework"
Shuva Jyoti Kar [Wed, 15 Nov 2017 14:24:13 +0000 (14:24 +0000)]
Merge "DiagStatus integration for openflowplugin"
D Arunprakash [Wed, 15 Nov 2017 08:29:36 +0000 (13:59 +0530)]
Add Docs for Openflowplugin
This patch just initializes sphinx for Openflowplugin
Change-Id: Ie7d0e2004d48e7ddacaf32ab18d0f94f5bc40c27
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
D Arunprakash [Wed, 12 Jul 2017 09:06:48 +0000 (14:36 +0530)]
Bug 8902 : Spec for Reconciliation Framework
This spec provides the documentation for Reconciliation framework
implementation and usage
Change-Id: I10c312d9890effb317a695450d381ee4d6725b60
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
Stephen Kitt [Wed, 8 Nov 2017 15:01:41 +0000 (16:01 +0100)]
Drop Karaf shell specifiers
We no longer need to support Karaf 3, and the karaf.shell
"[3.0.0,4.1)" declarations prevent us from upgrading to Karaf 4.1
(which is coming with odlparent 3).
Change-Id: I4367b34229246064319ee9e34199623666911f6a
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Faseela K [Thu, 12 Oct 2017 11:43:29 +0000 (17:13 +0530)]
DiagStatus integration for openflowplugin
Openflowplugin should report their initial status once
all switch connection providers are initialized.
Subsequent patches will query the dynamic status of port
6653/6633 to advertise the live status.
Further enhancements can be done if openflowplugin can query
some of the major service status before opening up the
southbound interfaces.
Change-Id: Id57f52d3ad02d59f7469762b1e1f0fb468b785ce
Signed-off-by: Faseela K <faseela.k@ericsson.com>
Anil Vishnoi [Fri, 27 Oct 2017 22:40:11 +0000 (15:40 -0700)]
OPNFLWPLUG-930 Inconsistent flow IDs between flows
in config and operational data stores
Change-Id: I4ee8787e163a6ab64a4574f0dc906c7c91a7d0b2
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Mon, 30 Oct 2017 06:21:37 +0000 (06:21 +0000)]
Merge "Fix issues related to checkstyle enforcement on openflow-impl module"
Anil Vishnoi [Mon, 30 Oct 2017 05:56:01 +0000 (05:56 +0000)]
Merge "Remove deprecated"
Luis Gomez [Fri, 27 Oct 2017 00:59:54 +0000 (17:59 -0700)]
Do not use fix custom version in ofp feature
Change-Id: Ic0a7c69a3f92b2d874fa64b168ccda50c6696351
Signed-off-by: Luis Gomez <ecelgp@gmail.com>
David Suárez [Fri, 20 Oct 2017 11:45:18 +0000 (13:45 +0200)]
Remove deprecated
- CheckedFuture
- Futures.addCallback
- Some other changes regarding checkstyle, sonar and annotations
Change-Id: I6ab7b10a6170bee6e78895b21d40ee1a047d961a
Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>
David Suarez [Sun, 8 Oct 2017 17:04:59 +0000 (19:04 +0200)]
Fix issues related to checkstyle enforcement on openflow-impl module
This is the last commit. It enforces the checkstyle level to error.
Change-Id: I954f0cbabd68134254e3a05216d6f6f126e33d37
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
D Arunprakash [Mon, 16 Oct 2017 09:06:50 +0000 (14:36 +0530)]
Fix checkstyle issues in openflowplugin-impl module
Change-Id: I8d6aa0b35078350bf897c5bf2b9a6cc11d0a4bf2
Signed-off-by: D Arunprakash <d.arunprakash@ericsson.com>
Anil Vishnoi [Fri, 13 Oct 2017 06:21:20 +0000 (06:21 +0000)]
Merge "Cli to display all the connected DPNs"
Gobinath [Tue, 26 Sep 2017 09:37:45 +0000 (15:07 +0530)]
Cli to display all the connected DPNs
Change-Id: I519450772b670750ccb38391a3c633ba572340d1
Signed-off-by: Gobinath <gobinath@ericsson.com>
Signed-off-by: gobinath <gobinath@ericsson.com>
Anil Vishnoi [Thu, 12 Oct 2017 06:06:23 +0000 (06:06 +0000)]
Merge "Fix log message"
David Suarez [Wed, 11 Oct 2017 09:43:22 +0000 (11:43 +0200)]
Fix log message
Change-Id: If70c25133d0a430ebb7b2a2906ac95fdad4d7bde
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Anil Vishnoi [Wed, 11 Oct 2017 07:40:34 +0000 (00:40 -0700)]
Disable the test that's causing merge failure
It's bit of a weird situation. Locally this unit test is passing consistently
but it's failing in merge job. Looks like merge job for the following patch
https://git.opendaylight.org/gerrit/#/c/62822/
failed and it didn't publish the artifacts and looks like that's causing
the merge job failure. This patch is an attempt to get a green merge job
and once all the latest artifacts are publish, this test will be re-enabled.
Change-Id: I99742b756040382b249072cd21782f42b4de3b21
Signed-off-by: Anil Vishnoi <vishnoianil@gmail.com>
Anil Vishnoi [Tue, 10 Oct 2017 23:51:18 +0000 (23:51 +0000)]
Merge "BUG-9218: use packaged inventory model"
Jalpa Modasiya [Fri, 6 Oct 2017 19:58:09 +0000 (12:58 -0700)]
Master: cherry-pick changes for BUG-9223
Cherry-pick changes for BUG-9223 from stable/boron branch for remove hardcoded value of lldp interval
Removed debug log from LLDPLinkAger.java
Change-Id: I95361679b6f12824f45e0ae31972d81485e23fc7
Signed-off-by: Jalpa Modasiya <jmodasiy@luminanetworks.com>
Robert Varga [Mon, 9 Oct 2017 18:51:04 +0000 (20:51 +0200)]
BUG-9218: use packaged inventory model
Rather than packing it here, use controller's dedicated feature
to pull it in.
Change-Id: I14f7acb4577d32eda954406f1cf55f5fff3cb0c7
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
Bertrand [Thu, 28 Sep 2017 05:33:04 +0000 (22:33 -0700)]
This patch implements ct-mark support in nicira extensions.
Specifically:
- support for ct-mark MATCH, including mask.
- support for ct-mark ACTION (set-field), but not including mask
This patch supports implementing ACL spec:
https://git.opendaylight.org/gerrit/#/c/60580
Change-Id: Ib828da39b4470e7172fe7028e8d3123fb184c742
Signed-off-by: Bertrand <bertrand.low@hcl.com>
David Suarez [Thu, 28 Sep 2017 15:09:17 +0000 (17:09 +0200)]
Fix issues related to checkstyle enforcement
This is the first of a series of two or three commits, since the module
is quite big. The latest commit will set the severity to "error".
Change-Id: I7a00bb3038c22ae352b2d5134cc0184ad29dc30f
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Stephen Kitt [Wed, 27 Sep 2017 13:36:03 +0000 (15:36 +0200)]
Bump odlparent 2.0.4 to 2.0.5
Change-Id: I2bafbcaa16edd56d1260ffb184d1568c14a4ef13
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Thanh Ha [Fri, 29 Sep 2017 15:07:01 +0000 (15:07 +0000)]
Merge changes
Thanh Ha [Fri, 29 Sep 2017 15:07:00 +0000 (15:07 +0000)]
Merge "Bump openflowplugin to use yangtools 1.2.0"
Anil Belur [Fri, 29 Sep 2017 02:25:35 +0000 (12:25 +1000)]
Bump openflowplugin to use yangtools 1.2.0
Jira: releng-485
Change-Id: I3a6b2716050b611e1a668a877ff6723f555c6372
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Tomas Slusny [Thu, 28 Sep 2017 14:39:19 +0000 (14:39 +0000)]
Merge "Remove Itemlifecycle"
Tomas Slusny [Thu, 28 Sep 2017 08:52:31 +0000 (10:52 +0200)]
Remove Itemlifecycle
ItemLifecycle was created for one time use case. Devices without
statistics possibility at all
and this workaround was made to store flows, groups, meters to the DS
as a reaction on services (flow, group, meters) when statistics
gathering could not be used.
Resolves: Bug 9010
Change-Id: Ic6eeb79d720284b7f139063ea238f40cef32a11a
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 27 Sep 2017 07:19:19 +0000 (09:19 +0200)]
Sort bucket actions
Sort bucket actions by order when serializing them in
GroupMessageSerializer.
Resolves: bug 9216
Change-Id: Icc5ad3f8b533a9f660e0248efed1b158c52336cf
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Tue, 26 Sep 2017 17:01:22 +0000 (17:01 +0000)]
Merge "Fix issues related to checkstyle enforcement"
David Suarez [Fri, 22 Sep 2017 10:54:30 +0000 (12:54 +0200)]
Fix issues related to checkstyle enforcement
Change-Id: Ia70763a22732dadf6d2f835915f9391c8e101e36
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Jozef Bacigal [Tue, 26 Sep 2017 07:58:55 +0000 (07:58 +0000)]
Merge "Gather statistics in separate thread"
Jozef Bacigal [Tue, 26 Sep 2017 07:57:13 +0000 (07:57 +0000)]
Merge "Async config message"
Tomas Slusny [Thu, 14 Sep 2017 08:04:43 +0000 (10:04 +0200)]
Gather statistics in separate thread
Gather statistics in separate thread to free netty thread from
processing statistics when they arrive.
See also: bug 8787
Change-Id: I2df7de1773db7f4c71a2f8ac33d75b123c12f0c3
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
miroslav.macko [Mon, 19 Jun 2017 14:29:27 +0000 (16:29 +0200)]
Async config message
- Add support for async config message
(Supported from version 1.3)
Resolves: bug 4082
Change-Id: Iae32e8a7433610d689c3d6b7a1c440032c3f2cc6
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
David Suárez [Sat, 23 Sep 2017 20:08:04 +0000 (22:08 +0200)]
Checkstyle enforcement.
This commit [1] left enforcement on warning, this one rise to error.
[1] https://git.opendaylight.org/gerrit/#/c/63234/
Change-Id: I80c896444e065b9f2aea5640775195deceb3ca24
Signed-off-by: David Suárez <david.suarez.fuentes@gmail.com>
Tomas Slusny [Fri, 22 Sep 2017 07:42:25 +0000 (07:42 +0000)]
Merge "Fix checkstyle warnings for impl/connection package and OpenFlowPluginProviderImpl"
Anil Vishnoi [Thu, 21 Sep 2017 21:59:42 +0000 (21:59 +0000)]
Merge "Drop the odlparent.netty property"
Anil Vishnoi [Thu, 21 Sep 2017 21:47:57 +0000 (21:47 +0000)]
Merge "Fix issues related to checkstyle enforcement"
Anil Vishnoi [Thu, 21 Sep 2017 17:08:14 +0000 (17:08 +0000)]
Merge "Fix transaction manager closing."
Tomas Slusny [Thu, 21 Sep 2017 14:16:53 +0000 (14:16 +0000)]
Merge "Fix issues related to checkstyle enforcement"
Jozef Bacigal [Tue, 5 Sep 2017 09:27:57 +0000 (11:27 +0200)]
Fix transaction manager closing.
- the event onTransactionFailed we properly closing chain and creating
a new chain
- changed writeOnlyTransaction to readWriteTransaction
- moved transaction manager to the common module
- topology manager using transaction manager instead of creating a
transaction chain for itself
- added control of unfinished transaction on close
See also: Bug-9038
Change-Id: Idadbb4ed0f4c61e7f80da5e2dbedbd80dece118e
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
David Suarez [Fri, 15 Sep 2017 21:39:14 +0000 (23:39 +0200)]
Fix issues related to checkstyle enforcement
Change-Id: I5d2103eadcb1a20586fb4b78babfb76447981149
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Jozef Bacigal [Tue, 19 Sep 2017 07:40:13 +0000 (07:40 +0000)]
Merge "Terminate SLAVE task before sending role change"
Tomas Slusny [Tue, 19 Sep 2017 07:19:16 +0000 (07:19 +0000)]
Merge "Fix NPE, version 1.0 returns null on auxiliary id."
David Suarez [Mon, 18 Sep 2017 13:43:59 +0000 (15:43 +0200)]
Fix issues related to checkstyle enforcement
Change-Id: I234058f50c2b035ca5d86dbd3a3ddd643c9747fc
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
David Suarez [Mon, 18 Sep 2017 10:23:46 +0000 (12:23 +0200)]
Fix issues related to checkstyle enforcement
Change-Id: I1cbb8d46e74185dbabbfbe74e8a6039ef6573168
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Jozef Bacigal [Mon, 18 Sep 2017 07:43:46 +0000 (09:43 +0200)]
Fix NPE, version 1.0 returns null on auxiliary id.
Change-Id: Id2bce3ec8c03e73d1fcb20a4bde099d5c2a0224d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Tomas Slusny [Thu, 14 Sep 2017 07:16:05 +0000 (09:16 +0200)]
Terminate SLAVE task before sending role change
- Terminate SLAVE task in role context when role change is requested and
not after role change is finished.
- Do not send notification about role failed to set to mastership
service in case future was cancelled
See also: bug 9089
Change-Id: Ia784d3b682b8991fe7a471bbfc7462aebfa6d4ef
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
David Suarez [Thu, 14 Sep 2017 15:44:20 +0000 (17:44 +0200)]
Fix issues related to checkstyle enforcement
Change-Id: I3d96af4e9d5f72921767e2aa06d9daffc20ef26d
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
miroslav.macko [Mon, 28 Aug 2017 12:47:12 +0000 (14:47 +0200)]
Fix checkstyle warnings for impl/connection package and OpenFlowPluginProviderImpl
See also: bug 8607
Change-Id: I6d9e1fcf02d3c282f7e640ab6e30858d79b02289
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Tomas Slusny [Fri, 8 Sep 2017 09:07:11 +0000 (11:07 +0200)]
Move used code from openflowplugin to impl
Move used code that was used only in openflowplugin-impl from
openflowplugin (old Helium design)
See also: bug 8653
Change-Id: I5a425db395601f49625f598057d835ca7a471ac9
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Fri, 8 Sep 2017 10:20:27 +0000 (10:20 +0000)]
Merge "Fix codestyle"
Tomas Slusny [Fri, 8 Sep 2017 10:19:57 +0000 (10:19 +0000)]
Merge "Fix codestyle"
Tomas Slusny [Fri, 8 Sep 2017 10:18:24 +0000 (10:18 +0000)]
Merge "Fix codestyle"
Jozef Bacigal [Fri, 8 Sep 2017 08:35:39 +0000 (10:35 +0200)]
Fix codestyle
- Removed deprecated getLOGValue from deviceInfo
See also:Bug-8607
Change-Id: Ic9c11ea6e2ace4e802ddc8dd3e2445655e2a57fd
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Fri, 8 Sep 2017 07:23:23 +0000 (09:23 +0200)]
Fix codestyle
- Anonymous type replaced by lambda
See also:Bug-8607
Change-Id: I7134315329b860d698927c6eb2c7f94a9476afde
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Fri, 8 Sep 2017 06:42:56 +0000 (08:42 +0200)]
Fix codestyle
- Removed unused import
See also:Bug-8607
Change-Id: Ie5f9e2449bcb072ff0a1a65857e3b371a1722dcd
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Fri, 8 Sep 2017 08:43:03 +0000 (08:43 +0000)]
Merge "Remove dependency of impl on onf-extensions"
Jozef Bacigal [Fri, 8 Sep 2017 08:42:57 +0000 (08:42 +0000)]
Merge "Add missing bundle converters"
Tomas Slusny [Fri, 8 Sep 2017 08:39:30 +0000 (08:39 +0000)]
Merge "Fix codestyle"
Tomas Slusny [Fri, 8 Sep 2017 08:39:20 +0000 (08:39 +0000)]
Merge "Fix codestyle"
Tomas Slusny [Fri, 8 Sep 2017 08:22:35 +0000 (08:22 +0000)]
Merge "Cleanup"
Jozef Bacigal [Fri, 8 Sep 2017 07:35:22 +0000 (07:35 +0000)]
Merge "Move match converter to new converter system"
Jozef Bacigal [Fri, 8 Sep 2017 06:39:50 +0000 (08:39 +0200)]
Fix codestyle
- Optional.isPresent replacesd to functional expression
See also:Bug-8607
Change-Id: Ib9a81b66751db08f93b12232f83da549cf9f2c1b
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Fri, 8 Sep 2017 06:34:37 +0000 (08:34 +0200)]
Fix codestyle
- Arrays.asList() memory save
See also:Bug-8607
Change-Id: I697affb49632c279175488ad35a2fe2111ddc234
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
David Suarez [Sun, 3 Sep 2017 09:22:52 +0000 (11:22 +0200)]
Cleanup
- Remove dead code
- Join if statements
- Private methods called from inner classes should be moved to those
classes
Change-Id: If66b8291e4b9cb237ebc88d1ce362e0aeb32d393
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Jozef Bacigal [Thu, 7 Sep 2017 11:13:30 +0000 (13:13 +0200)]
Fix codestyle
- explicit type
See also:Bug-8607
Change-Id: I5078f0b423942fdec385e9043d5a719cbffb9a6d
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Jozef Bacigal [Thu, 7 Sep 2017 11:11:40 +0000 (13:11 +0200)]
Fix codestyle
- unnecessary boxing removed
See also:Bug-8607
Change-Id: Ia37138f17e4831d38611571b37e74553f06e93af
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Tomas Slusny [Thu, 7 Sep 2017 11:00:12 +0000 (13:00 +0200)]
Move match converter to new converter system
In preparation of removal of Helium design, remove MatchReactor and move
MatchConvertor to new converter manager based system.
See also: bug 8653
Change-Id: I14db62975cd16d07f3fd7a39e36fc38bacb8abf8
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 6 Sep 2017 15:04:56 +0000 (17:04 +0200)]
Remove dependency of impl on onf-extensions
As preparation for removal of helium design:
- Remove dependency of openflowplugin-impl on onf-extensions and move
SalBundleService to openflowplugin-impl, as it is registered and used
only there
- Move onf extension yang models to extension-api
See also: bug 8653
Change-Id: Ib5813b5e7a115f1918d27ea116d7d49b4605799d
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 6 Sep 2017 15:03:47 +0000 (17:03 +0200)]
Add missing bundle converters
Add missing bundle converters about what was device context throwing
warnings.
Change-Id: Ic3d89c386c32f06d80afe1221f2d30fd919d087d
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Jozef Bacigal [Wed, 6 Sep 2017 14:43:31 +0000 (16:43 +0200)]
Fix possible NPE on ContextChainHolderImpl
- deviceConnected method
Change-Id: I7cbd8c3bb2a4d92ecdaa5fc541e01e3d8ac738b6
Signed-off-by: Jozef Bacigal <jozef.bacigal@pantheon.tech>
Tomas Slusny [Tue, 5 Sep 2017 08:04:56 +0000 (10:04 +0200)]
Do not mark device as connecting when closing it
Do not put incoming device connection to connectingDevices map when it
is still in closing state, to prevent having stale entry in
connectingDevices map.
Resolves: bug 9088
Change-Id: Ibee87152cd791f639999097c07a737c64afda990
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Tomas Slusny [Wed, 2 Aug 2017 07:48:37 +0000 (09:48 +0200)]
Redesign statistics context and manager
- To simplify statistics context and statistics manager code,
have better control over statistics polling and to remove
referencing of statistics manager from statistics context,
remove current method of statistics scheduling and replace
it with Guava scheduled service that will handle starting,
periodic gathering and stopping of statistics.
- Simplify chain of steps that continues with statistics initialization
with reconciliation framework enabled
See also: bug 8913
Change-Id: I0f3dcdd5292c6c7180ea80afc6d4d3df16b05ff9
Signed-off-by: Tomas Slusny <tomas.slusny@pantheon.tech>
Anil Vishnoi [Sun, 3 Sep 2017 08:11:07 +0000 (08:11 +0000)]
Merge "BUG8607 Fix checkstyle issues"
David Suarez [Sat, 2 Sep 2017 17:04:02 +0000 (19:04 +0200)]
BUG8607 Fix checkstyle issues
- Formatting
- JavaDoc
- Line lengths
- Name of variables
- Exception catching
- Remove redundant modifiers
- Add default case to switch
- Move variables closer to their
- Enforcement in pom.xml
Change-Id: If17fcdc00e8d22818a5501027a6bb7db3f2dcba3
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
David Suarez [Fri, 1 Sep 2017 08:45:32 +0000 (10:45 +0200)]
Enforce checkstyle
Enforce checkstyle by changing severity to error.
Change-Id: I4a6ae191bfec6df72e17f1c8ca50e92a9fd68ba5
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
Tomas Slusny [Wed, 30 Aug 2017 07:25:38 +0000 (07:25 +0000)]
Merge "Remove CSS artifact remnants"
Anil Vishnoi [Fri, 25 Aug 2017 22:43:24 +0000 (22:43 +0000)]
Merge "Fix checkstyle warnings for impl/protocol test package"
Anil Vishnoi [Fri, 25 Aug 2017 22:42:13 +0000 (22:42 +0000)]
Merge "Fix checkstyle warnings for impl/karaf, lifecycle, common, mastership"
Anil Vishnoi [Fri, 25 Aug 2017 22:30:55 +0000 (22:30 +0000)]
Merge "Fix checkstyle warnings for impl/device package"
Anil Vishnoi [Fri, 25 Aug 2017 22:30:20 +0000 (22:30 +0000)]
Merge "Fix checkstyle warnings for impl/datastore package"
Anil Vishnoi [Fri, 25 Aug 2017 22:24:14 +0000 (22:24 +0000)]
Merge "Fix checkstyle warnings for impl/protocol package"
Tom Pantelis [Fri, 25 Aug 2017 18:46:27 +0000 (14:46 -0400)]
Remove CSS artifact remnants
The project has already been converted to blueprint (except for test-extension)
so removed he remaining CSS remnants (eg use of config-parent, JMXGenerator).
Change-Id: I38c3e16ece82e2f5deada17a3faafa180e11a210
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
miroslav.macko [Fri, 25 Aug 2017 12:46:50 +0000 (14:46 +0200)]
Fix checkstyle warnings for impl/datastore package
See also: bug 8607
Change-Id: I7701b8b611648a3c7af90686847ecefd1970bf8a
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
miroslav.macko [Fri, 25 Aug 2017 11:40:29 +0000 (13:40 +0200)]
Fix checkstyle warnings for impl/device package
See also: bug 8607
Change-Id: I3f29d2ea40989877995b34211ac3e2f3aee5f637
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
miroslav.macko [Thu, 24 Aug 2017 15:32:14 +0000 (17:32 +0200)]
Fix checkstyle warnings for impl/karaf, lifecycle, common, mastership
See also: bug 8607
Change-Id: I8fc6a4677ec53df886877c62c5a98d624c01022f
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
miroslav.macko [Thu, 24 Aug 2017 13:22:41 +0000 (15:22 +0200)]
Fix checkstyle warnings for impl/protocol test package
See also: bug 8607
Change-Id: Icd8914faf38d0d6493dbe633b93cd2442e0bdfd4
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>
Tomas Slusny [Thu, 24 Aug 2017 15:40:19 +0000 (15:40 +0000)]
Merge "Fix checkstyle warnings for impl/role package"
miroslav.macko [Wed, 23 Aug 2017 14:59:42 +0000 (16:59 +0200)]
Fix checkstyle warnings for impl/protocol package
See also: bug 8607
Change-Id: I2adea22e939e8cf9bc87505ca4e130c73fc93a0f
Signed-off-by: miroslav.macko <miroslav.macko@pantheon.tech>