Change default RPM repo, CentOS CBS to ODL Nexus

Default to installing from fully-automated CD repo on ODL infra.

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

Update default RPM repo/path

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

Add support for openSUSE

Change-Id: Id05855f473a2754a4e320bee7e7c21f24873fa12
Signed-off-by: Manuel Buil <mbuil@suse.com>

fix concat issue with akka.conf jinja2 template

Change-Id: I0d6de69497072330555b90dddccf3e37544fbdc8
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

add missing bracket in akka.conf.j2

Change-Id: If92b8adaf732865245708cb17fc01508fc2dde3f
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Move to the nitrogen repo for Debian based OS

Change-Id: If984c6d1f60879a26acd57fa9d9f53a5c184d4d3
Signed-off-by: Manuel Buil <mbuil@suse.com>

Add clustering capabilities to Opendaylight role

When the variable 'cluster' is true then the Opendaylight
controller will be deployed in a clustering mode.

Change-Id: I8a1085bae933ad07eab33fc677d199ef613bc040
Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>
Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>

Spawn nodes & connect network with ansible roles

Refactor for better task encapsulation
Add kill_chown.yml to kill stuck chown processes.
Add restart_stacking.yml to restart compute nodes.
Move additional lab-specific variables into inventory.
Update tags to enble more granular task selection.
Modify container networking to use ansible instead of the bash script.
Add ansible tasks to pull container images from the registry.
Add tasks to purge containers matching a supplied string.
Modify order of infra tasks to install packages first, including
bridge-utils.
Add sampe hosts file.
Update site.yml to include a complete cloud setup.
Changed tabs to spaces for SpaceConsistencyBear.

Enable roles to spawn compute and service nodes
Common task added to start stacking on a compute or service node.
New common variable auto_stack added to control stacking after spawn.
Example ansible scripts to setup infra, spawn compute, & spawn service.
Compute and service roles spawn docker containers & connect them to the
network with a bash script.
Handler for apparmor parser moved to handlers/main.yml.
Site.yml now starts a service node by default.
Removed EOL whitespace.

Change-Id: I892afc76fffdcd85e399c5d2596eb4e5020cc318
Signed-off-by: Matt Welch <matt.welch@intel.com>

Merge "enable automated setup of physical infrastructure"

enable automated setup of physical infrastructure

Ansible role to setup the physical infrastructure servers for the S3P
Scale testing framework.
These roles are refactored to improve structure for upstream.
Source is based on
https://github.com/intel-odl/socets/commit/975f5113e0ce16cd6ef1eeb03176230e4f6445c9
Consolidated bridge management and refactored for clarity & reuse.

Clean up infrastructure role
Improve execution idempotency by registering docker_updated_result and
only restarting docker daemon when needed.
Same with parse_apparmor_profile handler.
Added tags to most tasks for better separation of responsibilities.
Moved lab-specific variables tnd inventory to /etc/ansible/hosts.
Addes top-level site.yml to call infrastructure role.

fix docker proxy and 'restart docker' handler
Infra role will now setup proxies and only restart docker daemon on
change.
Refactor to move most variables into common role.
Updated handler

Removed trailing whitespace

Change-Id: I7956b5acba7d339af2e8abbb588c07f0e809de5c
Signed-off-by: Matt Welch <matt.welch@intel.com>

Merge "Provide initial config files for sfc"

Provide initial config files for sfc

SFC requires that genius uses flow based tunnels and vxlan-gpe. We must
configure that intially in the genius config files

Change-Id: I6a1357a68cf589ddbbeb17a7550ac4a9e724e2ff
Signed-off-by: Manuel Buil <mbuil@suse.com>

Make jetty.xml port substitution version agnostic

Now it will work for Carbon, Nitrogen and Boron releases

Change-Id: I177e85a5eb6e9279550df774535a006309a308ae
Signed-off-by: Manuel Buil <mbuil@suse.com>

Allow long links in commit messages

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

Update RPM repo cfg file URL

Recent refactoring in Int/Pack changed the path, update accordingly.

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

Rm import of not-there add_jessie_backports task

This task file was removed in ee9c04f8bc4 when we moved to Launchpadd
PPA repos vs OpenSUSE OBS, as they don't support Debian.

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

Update supported Fedora versions

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

Update requiements.yml with new repo URL

Use new ODL-managed ansible-opendaylight repo in role install file.

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

Extract rpm_repo_file from rpm_repo_url

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

Add WriteGoodLintBear support

Fix readability, inheritance issues

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

Update ODL RPM version to Carbon

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

Fix typo in README for nb_rest_port

README document states that "odl_rest_port" can be configured to change
the port where ODL listens, but internally "nb_rest_port" is the
variable name used.

Also added an example using it.

Change-Id: Ic5aa035a063adc25e48bd9fc365cdb8ba54bb367
Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.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: Ie0e18e5f370c584f11d2dddd39cdf59e5d141bc7
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Fix markdown linting errors, enable linting

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

Add Coala linting

Some sections are disabled, will enable and fix in follow-up patches.

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

Add .gitreview file

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

Add carbon .deb support for ansible-opendaylight

Ansible-opendaylight made use of OpenSUSE Build Service (OBS) to
install .deb packages. This patch removes support for OBS and
instead adds carbon .deb support for Ubuntu 16.04 using Launchpad
PPA, `ppa:odl-team/carbon`.

This patch temporarily removes support for Debian OS.

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

Install opendaylight .deb only for Ubuntu 16.04

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

Add example playbooks for all testing/rel repos

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

Update docs for ansible-opendaylight

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

Fix RPM URL var default value

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

Add example playbooks

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

Create `requirements.yml` to install opendaylight role

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

Fix ansible-lint errors

Change-Id: I010ce114210a507ca560cfada4c45fefd8a5fc0b
Signed-off-by: srisankethu <sanketh.uppalapati@research.iiit.ac.in>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

CONTRIBUTING.md: Fix minor typo

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

Change `deb_path` to use remote URL instead of local path

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

Add .deb support to ansible-opendaylight

This commit checks the OS and then branches for either RPM or Deb installs.
It allows ODL to be installed in a debian based distribution (Debian or Ubuntu)
using a debian repo or .deb local/remote path.

Ansible version: 2.2.0.0
Tested for Debian 8.0 and Ubuntu 16.04

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

Remove debug statements

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

Add .deb support to ansible-opendaylight

This commit checks the OS and then branches for either RPM or Deb
installs. This patch allows ODL to be installed in a debian distribution
(Debian or Ubuntu) via a deb repo.

Ansible version: 2.2.0.0
Tested for Debian 8.0 and Ubuntu 16.04

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

CONTRIBUTING.md: Set IRC webchat link

Fixes https://github.com/dfarrell07/ansible-opendaylight/issues/25

Change-Id: I0aed5bb3175f9a7ba47411c5d928135a429fa104
Signed-off-by: srisankethu <sanketh.uppalapati@research.iiit.ac.in>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

CONTRIBUTING.md: Add IRC channel information

Closes https://github.com/dfarrell07/ansible-opendaylight/issues/25

Change-Id: I7da3450e0c88c44cf17bd2a867d09565d747818f
Signed-off-by: srisankethu <sanketh.uppalapati@research.iiit.ac.in>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Update default ODL version to Boron

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

Prevent false failures during firewalld checks

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

Add alt text for links, update maintainer info

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

Change README markdown file extension

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

Update default ODL version to Beryllium SR2 4.2

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

Use task conditionals that work with Fedora 22+

Not sure if it was an Ansible or Ruby update that changed this, but
conditionals of the form `x in [1, 2]` fail with errors related to
template parsing. Changed the conditionals to logically equivalent ones
that use simple equality and conjunction.

Fixes #23

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

Change 'yum' module to 'package'

Change-Id: I1a1f12bd2885e0197c544f7fb966b1714da777c1
Signed-off-by: Patrick Laurin <plaurin@inocybe.com>
Signed-off-by: Daniel Farrell <dfarrell@redhat.com>

Remove deprecated update_cache package mod param

When moving from the yum module to the package module as a part of #22
and #19, the `update_cache` param became deprecated.

Deployments against Fedora 23 now work.

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

Add Travis CI badge to README

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

Python line in Travis config is req'd for tox

Both the Python project type line here and installing tox in req.txt is
required for a working tox in Travis CI.

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

Try removing python line from Travis config

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

Add tox to req.txt so it's installed in Travis CI

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

Update Travis cfg to install Tox on build box

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

Add req.txt for installing ansible-lint

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

Add tox dir to gitignore

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

Add Travis cfg for running ansible-lint in CI

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

Add tox cfg for running ansible-lint against role

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

Add example playbook that uses this role

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

Use recommended file-absent method vs rm

The ansible-lint program complained about using rm to directly remove a
file, vs ensuring it's absent via the Ansible file type.

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

Don't use template for Karaf feature config

Templates don't work well now that ansible-opendaylight supports
installing multiple versions of ODL (via rpm_repo*, for example).

The content of the tempalted file is completly replaced, when only the
exact line that needs to be changed should be managed and the rest of
the file should use the version's default content.

Changing the version of ODL installed via params also requires changing
the default Karaf feature repos to match the version. This is confusing
and unnecessary - no one uses this knob that I know of.

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

Update default ODL install to Be SR1 4.1.0

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

Refactor logic for running in containers

New CentOS containers (7.2.1511 and not 7.1.1503) ship full systemd,
instead of using fakesystemd or not including it. It doesn't work
without modifying the Dockerfile and/or passing special params to the
`docker run` command, and it's generally not recommended for use in
containers, so we don't want to use it when provisioning a container.

The old checks were seeing systemd and (rightly so, it's installed) and
attempting to use it in containers. The new one directly checks if
running in a container instead of inferring from presence or absence of
systemd.

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

Update ODL to Beryllium

ODL Li SR3 to Beryllium 4.0.0

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

Add var to select .repo used for installing ODL

Defaults to Li SR3 repo config provided by upstream Int/Pack.

Can be overridden to install different ODL repo configs, and therefore
different ODL versions.

Other vars, like the karaf feature repos, would also need to be updated
if a different version of ODL is to be installed.

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

Update default rpm_path var to Li SR3

Closes #21

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

Update ODL Lithium Karaf feature repos for SR3

The 3-candidate RPM repo now installs ODL SR3, so the templated Karaf
feature repos need to be updated to find Li SR3 artifacts.

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

Update ODL Lithium Karaf feature repos for SR2

The 3-candidate RPM repo now installs ODL SR2, so the templated Karaf
feature repos need to be updated to find Li SR2 artifacts.

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

Minor inline docs updates for default vars

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

Extract ODL install to a single task for clarity

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

Officially support Fedora 21

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

Remove old, unused task

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

Confirm systemd intalled b4 using it in handlers

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

Bugfix in check if systemd is installed

When systemd isn't installed, the `rpm -q` check exists with
a non-zero exit status (1), which is expected but interpreted
by Ansible as a failure. Tell Ansible that RC 0 and 1 are okay.

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

Add skips for firewalld when systemd is missing

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

Add logic for not starting ODL w/ sysd w/o sysd

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

Update port 8080 remap for Lithium

Lithium changed how ODL opens port 8080, so the config needed to be
updated to match.

This is needed to support running ODL with other apps that rely on
8080.

Ideally, we'll eventually support remapping 8181 as well.

We also need to clarify our API for doing port remapping,
as the var name is horrible.

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

Update ODL to Lithium, install RPM via CentOS CBS

Updates OpenDaylight's version from Helium SR3 to Lithium.

Changes ODL's NB REST port 8080->8282 (following Lithium change).

Updates Karaf 3.0.1->3.0.3.

Updates support Java versions to >=1.7.

Installs OpenDaylight via the CentOS Community Build System (CBS),
vs Fedora Copr. Upstream ODL has move to the CBS for Lithium.

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

Add local/remote path-based ODL RPM install

Introduces a number of API changes, but the default
is still a remote Yum repo-based install.

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

Refactor README for format, bugfixes and clarity

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

Remove unnecessary and confusing task conditions

Some task and handlers were marked to only run on
Red Hat family OSs. We currently don't support any
other OS, and running against another OS (so with
only the not-excluded tasks and handlers) would
break. They add unnecessary complexity, may mislead
people and add no benefit with our current feature
set.

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

Add handlers to clean start ODL after #10 test

This is mostly an experiment, may or may not be helpful.

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

Revert "Include experiment task in tasks/main"

This reverts commit a2c96d1f366cca1fba7f9eb9f23b5868e8425699.

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

Include experiment task in tasks/main

This should not be merged into master. It's an experiment.

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

Add task to copy #10 test file to correct path

This is an experiment, may or may not be helpful/work.

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

Add file related to test in #10

This may or may not be helpful/required for installing
the Jolokia bundle non-interactively (contracted with
our interactive Karaf shell command task, which works
fine for the record). It's basically an experiment.

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

Document that NB port will be opened by firewall

Relevant to #15, closes #9

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

Include task to open NB REST port in tasks/main

Implies that it will be used by default when the role
is applied.

Relevant to #9

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

Add task to open ODL's NB REST port via FirewallD

Relevant to #9

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

Fix trivial typo in README

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

Document the Karaf feature repos params in README

Relevant to #14 and #15

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

Update features template to use Karaf repo vars

Closes #14

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

Add vars to configure ODL Karaf feature repos

Relevant to #14

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

Document overriding role vars to config ODL

Relevant to #15

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

Update contrib docs name/link to new extension

Was recently forced by Ansible Galaxy to use .md vs the
proper .markdown extension. Reflect that change in the README.

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

Ansible Galaxy requires .md extension

Even though .markdown is the best practice recommended
by the upstream Markdown people, Ansible Galaxy requires
a README.md for all roles.

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

Use start/stop handlers vs restart handler

To get proper behavior with both the ODL NB config
and Karaf feature config changes notifying sets of handlers,
need to use the same ones when possible. Else, we might
stop ODL, make Karaf changes, start ODL and then restart ODL.

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

Notify handlers to clean-start ODL after Karaf cfg

Calls the handers to stop ODL, clean up Karaf's cached
config (as recommended by Karaf docs) and then start
the ODL service after a Karaf features config change.

The order these handlers are listed in isn't what actually
causes them to be ordered correctly! Ansible defines the order
of handlers based on the order they are included in handlers/main.

Closes #7

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

Include currently used set of handlers in main

The order of the handlers is critical! Per the Ansible docs,
handlers are run in the order they are included. Changing
the order may result in event orders other than the desired
[stop ODL, cleanup Karaf, start ODL].

Relevant to #7

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

Add handler to start ODL's systemd service

Relevant to #13

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