Merge "Bump Puppet version 4.x to 5.x"

Bump Puppet version 4.x to 5.x

Change-Id: I257ae1b7dcca738ba023fcb289863fcd10a784a7
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update default log size/rollover 10GB/2 to 500MB/4

Update the default logging configuration to set the max log file size to
500MB from 10GB and the maximum number of log rotations to keep to 4
from 2. Meant to address concerns about 10GB being too large to work
with, and that some filesystems don't have 20GB of available space.

Also update log size/rotation test values to reflect new recommendations
and serve as reasonable examples for users.

Change-Id: I2c809e39ac1d050c5327936e73b654a2dda11286
Jira: INTPAK-227
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Add boolean flag to disable the PaxOsgi appender"

Add boolean flag to disable the PaxOsgi appender

Currently the PaxOsgi appender is on the OpenDaylight default
configurations for the logging system. It was found, during a research
of performance issues [1], that this adds pressure to the Java GC and is
a facility that is interesting to have configurable.

This patch adds the configuraton boolean `enable_paxosgi_appender` that
allows packaging OpenDaylight with this setting disabled. In addition,
due to a business requirement [2], this setting is set to `false` by
default.

[1] ODLPARENT-176
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1650576

Change-Id: I358618594a5703cc8a22575dde88e9273e2b8a16
Signed-off-by: Michel Peterson <michel@redhat.com>

Merge "Update to ODL Neon RPM pipelines"

Merge "Remove old inline TODO, wolntfix/not-a-bug"

Update to ODL Neon RPM pipelines

Change-Id: I953c5cddcbb2f4315dbc786d334aea08fe8ac12e
Jira: INTPAK-211
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove old inline TODO, wolntfix/not-a-bug

Remove very old inline TODO that has been copied prolifically.

The NB it accompanies is true: the defaults in the tests need to match
the defaults for the module; but it's not actually important to change.

Change-Id: Idd5c2f7170d703c0c894f4c3e1a78139a21dab7f
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Parametrize the logging pattern to use

Currently the packaging of opendaylight is being done with the current
opendaylight default. This default does not always suits the needs of
every deployment and it is good to have this configurable.

This patch adds a `log_pattern` configuration that allows to specify the
logging pattern that will be used. At the same time, it also defines a
default that is non dependant on PaxOsgi on being enabled [1]. The new
default removes the '%X{bundle.id} - %X{bundle.name} -
%X{bundle.version}' bits that are PaxOsgin related, and to compensate
for that information removed it adds more context to the class name and
expands the clipping of the column to 60 characters.

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1650576

Change-Id: I923cd372c71ba5c04a27f6d4478ca7c511c14d2d
Signed-off-by: Michel Peterson <michel@redhat.com>

Merge "Support multiple argumets for EXTRA_JAVA_OPTS"

Support multiple argumets for EXTRA_JAVA_OPTS

Add double quotes to allow passing multiple arguments to
EXTRA_JAVA_OPTS.

JIRA: INTPAK-217

Change-Id: I5e7224727d72d515b2465a1ccf8bc9b46fc2b1ce
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Install net-tools for netstat in Beaker tests

The current Beaker acceptance tests use netstat to check when an ODL
port opens up. Looking at the log, it seems netstat is no longer
installed by default in our centos:7 container. Install net-tools
package in via nodefile.

JIRA: INTPAK-208
Change-Id: If631ba2f99a2360aeb0790b1d6e2f160b075c8b0
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Adds inactivity probe configuration

Configures inactivity probe timer, which is the time before a probe is
sent to an inactive switch via OVSDB. If the switch does not respond in
the same time as the inactivity probe interval, the switch will be
considered down.

Change-Id: I3b8ae5239841c5412716d9e9be89516c20abc8d9
Signed-off-by: Tim Rozet <trozet@redhat.com>

Binds OpenFlow and OVSDB IPs

The odl_bind_ip has been used to bind the northbound HTTP REST server to
a fixed IP, however the OVSDB and OpenFlow services were running on
0.0.0.0. This patch binds those services as well.

Change-Id: I7173706b3e812adc0ad899f2a3b6f4b28b91897f
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix karaf log rotation

Missing a config parm for this to work as desired.

Details in the JIRA.

JIRA INTPAK-199

Change-Id: I0776713d541bf6774da2ea8ee50e8bc6db42cb45
Signed-off-by: Victor Pickard <vpickard@redhat.com>

Add support for IPv6 Deployment

Till now, only IPv4 deployment of ODL was supported.
With this patch IPv6 will be supported. EXTRA_JAVA_OPTS
property is set accordingly java.net.preferIPv4Stack=true
to allow only IPv4 addresses. java.net.preferIPv6Addresses=true
to allow both IPv4 and IPv6 address.

Change-Id: I760681e447f6bec4f249f75501b9363793b41ccc
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Upgrade Beaker 3 to 4

Our unpinned Beaker version went from pulling in 3.39.0 to 4.0.0, which
broke our tests. Need to do the upgrade procedure as documented here:

https://github.com/puppetlabs/beaker/blob/4.0.0/docs/how_to/upgrade_from_3_to_4.md

Change-Id: I29479aa650894a4e00fc4ff38ff14fbfda8efa98
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Signed-off-by: Tim Rozet <trozet@redhat.com>

Disable OF statics polling

ODL polls for OF statics for its properties like
group. This causes unnecessary long logs and affects
performance in large scale deployments. Disable it
with a flag 'stats_polling_enabled=false' which can
be changed to true to enable it again.

Change-Id: I7f659a72e485d6c82ddce6649b2e04c804ceefce
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

INTPAK-185 Binds karaf to ODL IP

Karaf shell was running on 0.0.0.0, port 8101 which is a security risk
(especially with default user/pass karaf/karaf). This patch binds karaf
to the same IP specified that ODL REST and other ports are bound to.

Change-Id: I2a63c02d3469d7fb71739a529810acb8fc1e0213
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Refactor Beaker tests"

Merge "Add "direct" option for logging to console."

Add "direct" option for logging to console.

Log4j 'direct' option for performance improved OpenDaylight
logging.

Change-Id: I308ed47b531313035e62233428e431be4bfc9d7d
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Refactor Beaker tests

* Test against latest from CD repos and latest release/SR
* Latest release/SR tests commented out until Fluorine release
* Name things to make updates easier in the future

Change-Id: If59169c015871b8685f52534e9f9a22550af4c79
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Add flag to enable DSCP marking

Configure files to support QoS DSCP marking feature.
genius-itm-config.xml is configured to support this feature.

default-tunnel-tos is '0' by default and 'inherit' for DSCP
marking support.

Change-Id: If06b1dd41235fb139642a77c0f098598d6d3018b
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>
Signed-off-by: Tim Rozet <trozet@redhat.com>

INTPAK-173 Remove SFC ELAN config

Now that we use genius auto tunnels there is no need to include ELAN
manager config which breaks DHCP when SFC is used.

Change-Id: Iceba4201ac0e98ace88d48ae9796e4e54666966c
Signed-off-by: Tim Rozet <trozet@redhat.com>

INTPAK-164 Disable MD-SAL based trust store

OFP has no support for MD-SAL based trust store. We were trying to
enable it in HA, however it does not work and lacks support. For now we
need to stick with the file based trust store.

Change-Id: I25fd24a71fa2f61b0c54f9601953219749918188
Signed-off-by: Tim Rozet <trozet@redhat.com>

INTPAK-163 Fixes default ODL Java opts

The previous way we were configuring Java options was preventing any
default Java options from being included when ODL started. These
included garbage collection and heap size settings.  This patch modifies
the behavior of specifying Java options to puppet-opendaylight to be in
addition to the default Java options provided by ODL.

Change-Id: Ia46a8b64842bf766543f18678e100f33778f2d76
Signed-off-by: Tim Rozet <trozet@redhat.com>

Update master branch to Fluorine

Update all default/test branch references from Oxygen to Fluorine.

Didn't manually check for possible changes to templates, expecting tests
or config file maintainers to catch problems and fix.

Issue: INTPAK-147
Change-Id: Ia491a9e0eb695882e218484b3d35017cf7295df0
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Disable intermittently failing curl in tests

Have tried to make this stable unsuccessfully. For now disable to make
verify job useful.

Issue: INTPAK-136
Change-Id: Ia1ca5c7db12b50db7894f13e13d58652f6175c22
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove deprecated test fn calls outside of tests

Calls like these outside of test blocks throw deprecation warnings and
fail tests.

> Mocha methods cannot be used outside the context of a test

Just removing them makes the tests pass.

Issue: INTPAK-160
Change-Id: I5a4a94102338454bf3e03e9d6002c6c5b6f8e4f0
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

INTPAK-152 Disable HTTP when TLS is enabled

There are port conflicts when TLS is enabled due to HTTP and HTTPS
starting on the same port.

Changes-Include:
 - Explicitly disables HTTP when TLS is enabled
 - Fix missing match on HTTP Pax port
 - Only configuring HTTP pax port when TLS is not enabled

Change-Id: Ie077c9ff7edeb34c91186ca1f492d607f0e55cc4
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

INTPAK-117 Fixes configuring Java Opts

Migrates configuring Java Options to a more appropriate variable,
"JAVA_OPTS" which is read by karaf as optional settings a user may
provide.  The change also includes moving the configuration by setting
the variable in the karaf file itself, rather than in systemd.  This
allows support for non-systemd OSes and deployment tools.

Change-Id: I2fd3c2a70e8d5f0f0645bc9e354a535d05526fd3
Signed-off-by: Tim Rozet <trozet@redhat.com>

INTPAK-140: Fixes always listening on default port

Pax config file was not being configured with the specified rest port.

Change-Id: I3c711d37335105de647724a0ac4138b8456a13d7
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "INTPAK-123 Move to log4j2"

INTPAK-123 Move to log4j2

With karaf migration to 4.1, log4j migrates to V2. Move all logging
configs to use log4j2.

Referenced https://git.opendaylight.org/gerrit/#/c/66568/

Change-Id: I6d344211d90c36cdee0df8e55e18a04e59c2db45
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Disable deb tests for Oxygen, no pkg available

No one has stepped up to properly maintain the deb pipelines, so we've
done what we can with our cycles and by mentoring interns, but it's
lagging behind the RPM pipelines. At least until we get an Oxygen RPM,
need to disable the tests on master.

INTPAK-135

Change-Id: Ie94c026f4b84bf4721fe9ebd79b81089542fb497
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Use new default REST port 8181 vs old 8080+8181

ODL Oxygen after the recent odlparent version bump uses port 8181 as the
default REST API port now, vs old 8080 and 8181.

https://git.opendaylight.org/gerrit/#/c/51531/
https://lists.opendaylight.org/pipermail/controller-dev/2016-September/012677.html
https://lists.opendaylight.org/pipermail/release/2017-February/009244.html
https://lists.opendaylight.org/pipermail/integration-dev/2017-February/009062.html
https://lists.opendaylight.org/pipermail/release/2018-January/013542.html

Change-Id: I43c9dc45d08b85154fd90ec2e6c3f3f4d10be2ce
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Allow links in git commit messages, coala ignore

This rule is used in most other ODL repos that use Coala.

Change-Id: Ia9159a8862385d714b375ca1558424b9a0561b08
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

INTPAK-134 Updates Jetty configuration

Jetty configuration has been changed to only include 1 connector by
default (previously 2).  The configuration has been changed to operate
on that single connector, as well as update TLS ports configuration to
use the http-default group (now applied to the connector instead of
http-legacy).

Change-Id: I44ced818b8cddab76bc700ec813e59b7b5ce9c01
Signed-off-by: Tim Rozet <trozet@redhat.com>

Merge "Remove append_on_no_match parameter"

Remove append_on_no_match parameter

append_on_no_match is true by default. This patch removes the
reduntant line. And also TripleO does not recognise that parameter
during installation.  Logging size and rollover is also fixed to
only be configured when using file logging.

Change-Id: Ic0a515dc8f40ec76840bca150805a4f1614b3e32
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Pin nodeenv dep to let tox run more than once

Using the latest version makes tox break on second run, unless .tox dir
is removed.

Change-Id: I8c3129def784e11bb1730565bad8b3db03a7069a
Fixes: INTPAK-74
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Fixes to TLS ODL keystore provider

File exist checks were done in in the odl_keystore type.  Checks within
a type are done before puppet is applied.  Therefore the file has to
exist before puppet is ran, which excludes another puppet resource from
being able to create the files at run time.

The JKS provider for the keystore was incorrectly passing the CA
certificate as a file to create the p12 keystore, instead of as a X509
certificate object.

Changes Include:
 - Checks to ensure key files, ca cert, and public cert are moved from
   the type into the jks provider.
 - CA cert is now correctly created as and OpenSSL:X509 object before
   being passed into the p12 keystore create.

Change-Id: Iaf3e66fa44dee3987817177e2210230ee2d3325c
Signed-off-by: Tim Rozet <trozet@redhat.com>

Make username/password test more robust, faster

Test currently sporadically fails due to timing, and also wastes time
with a hard 180s sleep. Replace sleep with active polling, increase max
wait time.

Change-Id: I01ebe37b3adc106b1eef9bb5c810bddca9b437d2
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

INTPAK-124 Migrates Debian from Carbon to Nitrogen

Migrating to the latest available release.  TLS tests fail in Ubuntu
setups because the jetty file is from an older release and is missing
configuration.  Also disables user/pass acceptance test until it is
fixed by a future patch.

Change-Id: I8424f36df862596ee9a5407a6a0da34295b2214b
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix websockets ip address config

Opendaylight master commit 96914ace4fd625d9a9c89ec49ddeb411fff180b9
uses restconf.cfg to config the websockets ip address.
Opendaylight carbon comiit a6bd56636d246fddcfbbef60277d77d3a7663b46
uses 10-rest-connector.xml to config the websockets ip address.

Commit 168424499efb9c7ed35bf627c417f323164682fa
allow to configure websocket ip address, but for
carbon release. This patch fix it to be
correct to master to configure the restconf.cfg instead
of 10-rest-connector.xml.

Change-Id: I304e15df5c7982283e843285c1910ae3c9a463cc
Signed-off-by: Moshe Levi <moshele@mellanox.com>

Add option to configure karaf logging

Add an option to enable karaf logs to be logged either to file or
console. Default setting is to log to file as karaf.log. There
would be 2 files generated with size 10GB each.

Change-Id: Iba97982e331c4ddd18eafbe65c80acf7f1f5a151
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Adds SSL/TLS support

Allows configuration for requiring TLS communication across Northbound
REST and OVSDB/OF with OVS.  The TLS configuration enables two
keystores, "controller" and "truststore".  The controller keystore is
used to hold the private key and certificate, which may be
auto-generated or provided as input parameters.  The truststore holds
trusted certificates for clients connecting to ODL and may also be
provided via a parameter.  Additionally when providing the private key
and certificate to the controller keystore, a public certificate
authority certificate may be linked.

Change-Id: I079fc0759bb42888472ef95ca239c3ca67db8a56
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Change default RPM repo, CentOS CBS to ODL Nexus"

Change default RPM repo, CentOS CBS to ODL Nexus

Move to our fully-automated CD repos as the default RPM install source.

Change-Id: Iea24629c18699b2b793d0a55d88b3516a3c5b75b
Fixes: INTPAK-112
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove unused link from README

Change-Id: I179e2c21892776bc1ab20dae5706f4080c23f079
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update contributing guide

Update contributing guide to reflect updated Centos VM names
for beaker run examples.

Change-Id: I26ddbfade66d8f4cd9e23107533b373eb3625e12
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Merge "Cleanup Gemfile"

Cleanup Gemfile

* Remove old groups that were used for Travis CI
* Rorg ABC order
* Add semantic_puppet gem as suggested by warnings in logs

Change-Id: Icbdd309c8567baff4f23dc61cbd5aac745d680f7
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Configures websocket listener IP

Will use the already provided bind ip to also bind websocket listener.

Change-Id: Ia2084fd3e064f579d34ba7a60fec0df5b910e411
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Allow full customization of ODL RPM repo

Instead of assuming ODL's RPM should be installed from a CentOS CBS
repo, allow installing from any repo. This will allow us to install from
ODL's new fully-CD Nexus repos.

Param rpm_url is changed to be a baseurl var to configure ODL RPM .repo
config file to install from.

This is a backwards-incompatible API change.

Also now defaults to use Oxygen ODL and fixes an issue where auth tests
were failing due to odl-restconf not being available.  Although,
odl-restconf was specified in the test, it was not being loaded because
ODL does not respect subsequent changes to the karaf features file.
Therefore we need to load it during the initial ODL install.

Closes-Bug: https://jira.opendaylight.org/browse/INTPAK-11

Change-Id: Ibf49be1bf3ee780f9ca69894ffb0a1f3f5ac385c
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Signed-off-by: Tim Rozet <trozet@redhat.com>

Bug 9266 Remove all security group related configs

From Carbon and beyond, only supported sg mode is stateful.
Hence all of the related configs can be removed.

Change-Id: Id11ce69233c1a0d6982f53cf6a1c0f47bda2e679
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Fixes default features for Karaf 4

Only 'standard', 'wrap', and 'ssh' are now
required with Karaf 4 releases.

Change-Id: Ieecc1e45712c918e8da602241f7f7d6a695c7afb
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix bind IP config for REST

Changes bind ip to be set as a default attribute rather than value and
also configures pax web cfg as recommended in
https://bugs.opendaylight.org/show_bug.cgi?id=9158

Note explicitly configuring 0.0.0.0 does not work so if default we skip
configuring bind IP and it defaults to 0.0.0.0.

Also increases sleep before curling REST in acceptance tests because
Karaf 4 takes longer to boot.

Change-Id: I23e7b1d501c782d5d1df0aa6d8b3e6cac9540de7
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fixes configuring jetty.xml

In Nitrogen and later the template configuration for jetty.xml is
incorrect and ODL does not start correctly.  This patch removes the
template in favor of using augeas.

Also updates acceptance to use nitrogen (since oxygen repo not
available).

Change-Id: Ia5cf466cb8ba2596d1df0fa4f2d5956827206e46
Signed-off-by: Tim Rozet <trozet@redhat.com>

Update contributing docs

Change-Id: I71c71ad42a881a133acd534359c4388de507d1a3
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Adds VXLAN+GPE config when using SFC feature

Config is necessary for Genius to create VXLAN tunnels as VXLAN+GPE when
using the odl-netvirt-sfc feature.

Disable XML Coala linting because of false-positive regarding XML
version. ODL uses files like this, should copy. Can reevaluate enabling
later.

Change-Id: I02314dac7c779d00d54cd5c47448822792e353ab
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Remove unnecessary rspec ignore rules"

Merge "Remove unnecessary puppetlabs-java dependency"

Merge "Remove unnecessary puppet-lint config"

Merge "Remove unncessary null->none RPM repo Beaker logic"

Merge "Update puppet-opendaylight README"

Remove unnecessary rspec ignore rules

The code ignored by these rules now isn't picked up by rspec tests as
untouched in coverage reports, either because of enhancements to the
tooling or because we no longer import the code.

Change-Id: I411524ec898c9687631d318dd807226ebdbdf205
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove unnecessary puppetlabs-java dependency

This was necessary when we did from-tarball installs, but now that
packages always manage installing Java it's unnecessary.

Change-Id: I47096ec42acbae4d7b37918172353e7a044b68c2
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove unnecessary puppet-lint config

No longer seems to do anything useful.

Change-Id: Ibf951439287a42283cc0f4c1d0b62c29b7c7e7e2
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update puppet-opendaylight README

Various age-based refactoring and changes related to moving to
ODL-managed repos.

Change-Id: Ieef66e3743608e5424ba008005ef3cf0dd957c79
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove unncessary null->none RPM repo Beaker logic

Change-Id: Ic0c04219fe03352051a9508b6e950a0f00580e4b
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove "temporary" work-around from ages ago

We no-longer support Puppet 3, so this is definitely no longer an issue.

Change-Id: I2dd18737fb4a678a7acebd32be7ec6472be83908
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Bug 8985 - Configure SNAT to use conntack

Configure SNAT to use conntrack by default. controller mechanism
can be used by changing 'snat_mechanism' parameter to 'controller'.

Change-Id: I21a1f4e0ddcaa1d434cb91cb15d987a4c2c0f8b7
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>

Remove last traces of puppet-archive dep

We were using this to do straight-from-tarball installs to provide
non-RPM-OS support before we had .debs. When we added .debs, we removed
the ability to install from tarballs, which makes this unnecessary.

Change-Id: Iafb586648c57d1c1aed7f9d0f8e669a367f67fd4
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Fix Vagrantfile, both CentOS and Fedora machines

* Don't do bundle installs, only updates. Fixes problems caused by old
gemfile.lock files getting sync'd to VM, perhaps other problems.
* Create required docker group, not sure how it worked without this.
* Add vagrant user to rvm group to fix gem install error.

Both machines now `vagrant up` and can do `bundle exec rake
acceptance_dock`

Change-Id: I16f4c22a3863293058d1c8409591bd2759d3e9dc
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Support Fedora 26, EoL Fedora 24

Fedora 26 is out and Fedora 24 will go End of Life on August 8th.

Change-Id: Ie3bf0f30c405e0fce399937270ae7075ce872c8c
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Version bump to 4.1.0 for loging config params

Change-Id: I834cd4f05039162214dad87ce41a394fd4d67c08
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Bug 8854: Add config of ODL log file size/rollover

The default ODL logging config uses 10 small log files. This makes
searching for errors difficult. Add the ability to configure the maximum
size of karaf.log and the number of rollover logs to keep. Default to
values that make it normal to end up with a single large log file but
still allow pruning from the tail when logs rollover.

Change-Id: Ibcdd0c6634fc37180a9880e03abfd24f1eb84a10
Signed-Off-By: Janki Chhatbar <jchhatba@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Add WriteGoodLintBear support

Fix readability, inheritance issues in coafile

Change-Id: Ibf9bcef436b2c80ee48cdbcbb05a490dccc6651a
Signed-off-by: Sri Sanketh U <sanketh.uppalapati@research.iiit.ac.in>

Fixes clustering to use template instead of exec

Adds ability to specify modules to use when breaking up the datastore
into shards.  By default we just use the 'default' module which contains
all of the datastore, but a user is able to pass a hash of modules and
namespaces if they desire to break up the datastore into separate
shards.

Deprecates any need to use ha_node_index param.

Change-Id: I05c88741c16c433baa11fe4cbc43bc04fc8c2625
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fix metadata-json-lint fail by updating Puppet

We were pinned to 4.8, which metadata-json-lint was failing with due to
a (new) missing dep on semantic_puppet. Using >=4.9 doesn't require
specifying that dep explicitly. Pin to the latest Puppet 4.x version.

Also update the Puppet module metadata to reflect version change.

Change-Id: I26535cc4799e16d7088efcd3099b40a70c34e433
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Fixes idempotency and maintence with logging"

Disables modifying password of user

Until we have support for detecting if an existing user's password
matches the desired password, we need to disable support for setting it.
The current behavior is we don't know what the current password is so we
always configure it.  This means our puppet module is not idempotent and
triggers ODL to restart everytime puppet is run, which is bad.

The request to add support for the idm tool to be able to check password
is:  https://bugs.opendaylight.org/show_bug.cgi?id=8721

Note this does not affect odl usernames.  We still detect if a user
already exists and if not create it with the desired password.

Change-Id: I64a23ad6fe68d1afb765f46c6b54f7291d25b47e
Signed-off-by: Tim Rozet <trozet@redhat.com>

Fixes idempotency and maintence with logging

ODL will modify the loggin file at run time to add qos and other
configs.  This causes a problem where when puppet runs again it uses a
template so it overrides the file and restarts ODL.  This change
abandons using a template and instead just makes sure each logger is set
in the file using file_line.

Change-Id: Idd7d565be61a73985203b64c200281c9242c87bb
Signed-off-by: Tim Rozet <trozet@redhat.com>

Use Puppet 4 in Beaker tests

Change-Id: I2900fdc2dd1f0f05f053e0eb07c27bf8ef337a01
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Use username opendaylight in metadata

Puppet Forge requires this field to be the same as the username that
uploads the module package. To host puppet-opendaylight on the offical
Puppet Forge account, needs to be opendaylight-opendaylight.

Change-Id: I6eaaff9148799dc78a1da64a091e8cd8959f2ee3
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update idm db location

This was changed in ODL to the data dir:
https://git.opendaylight.org/gerrit/#/c/56337/2

Change-Id: I7908679a8a4514874384833b56c4149125bcab4e
Signed-off-by: Tim Rozet <trozet@redhat.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Pin requests to <2.16

The new requests library pulls in a chardet dependency that is in
conflict with coala packages. Pin the version of requests to ensure
we're pulling in a supported version.

Copied change by Thanh.

Change-Id: I83f780b6ef93e1a81f5273baacf762533fd7ec7f
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Pin coala-bears version

Change-Id: Ib53531ee961d402d3fb9f0e3168b75d6a34c3f64
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Remove Travis CI config/docs, no non-GH support"

Update metadata for new ODL-managed repo

Change-Id: Iee6d531e23dcf6e7ca337bcbfe968672c3e2383a
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove Travis CI config/docs, no non-GH support

I don't think it's possible to configure Travis CI to run on repos that
aren't hosted on GitHub, so now that we've moved to ODL-manged infra
this does nothing.

Change-Id: I7b58360b6841b15b48d7c9f93e5b7d0e750d4d67
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Pin Coala to <0.11 to avoid bug

Change-Id: I5190be4f2e78ac9dfa828d57a36f400af2892f9d
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Add carbon .deb to puppet-opendaylight

Change-Id: I76565b53f310c5004ae6683a21cfb53a4a079340
Signed-off-by: Akshita Jha <zenith158@gmail.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Only do Carbon tests on master/carbon branches

Will move master to Nitrogen once we have builds.

Change-Id: I3eb0f613bc54093c02e6766f7064fd35e92e0210
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Merge "Revert "Only test Boron in Rakefile suites on ...""

Merge "Revert "Revert "Adds ability to configure ODL.."""

Revert "Only test Boron in Rakefile suites on ..."

This reverts commit 8e3cd6ee9a0c484e9eb28f26f40fe8b909e8b974.

Change-Id: Ia93f05cbe1209d710266514524559b33b4332d4c
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>