Fix path to images

Relative playbook path breaks
in aio configuration.

Issue-ID: MULTICLOUD-684
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Change-Id: Id064157d010438dea33500dd0dc200b1c1b0f0d1

Add support for Network Operator

ovn4nfvk8s plugin now uses operator sdk
and controller runtime. It now includes
support for Network operator. This patch
includes changes needed in KUD for that.

Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Issue-ID: MULTICLOUD-684
Change-Id: I63dc971e257067c69c70a8996eaffd1a9d8a4c2c

Merge "Version update."

Merge "Adding Helm charts of MetalLB"

Version update.

Updating Kubespray version from 2.8.2 to 2.10.4 for KuD offline
support and integration with new add-ons in future.

Signed-off-by: Akhila Kishore <akhila.kishore@intel.com>
Issue-ID: MULTICLOUD-772
Change-Id: I4b7887aae359cd6197e696010acde6e204c41931

Merge "Revert "Version update.""

Revert "Version update."

This reverts commit 5f760c3fb7d0e74833b1a2137e6ff3dadc71b2f5.

Issue-ID: MULTICLOUD-772
Change-Id: I6feffd87545195992fb28e98dcee4038d9b08474
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Version update."

Merge "Add query parameters support to instance API"

Version update.

Updating Kubespray version from 2.8.2 to 2.10.4 for KuD offline
support and integration with new add-ons in future.

Signed-off-by: Akhila Kishore <akhila.kishore@intel.com>
Issue-ID: MULTICLOUD-772
Change-Id: Ib1263e86adb9815e1ee56038507a3c092aad1feb

Merge "Fix bug in tagging podTemplates"

Adding Helm charts of MetalLB

MetalLB is the loadbalancer for Kubernetes

Issue-ID: MULTICLOUD-747
Signed-off-by: Pramod <pramod.raghavendra.jayathirth@intel.com>
Change-Id: Ic07877234f9b0f80cb636f7b4e3c265b8be8a945

Helm charts for Istio deployment

This Chart will deploy istio in SDS configuration
and enables istio sidecar injection in required
namespace

Issue-ID: MULTICLOUD-711
Signed-off-by: Pramod <pramod.raghavendra.jayathirth@intel.com>
Change-Id: I9a4dd781c14c2dc2bb1ee8ca648e3c8911ac1fe1

Add query parameters support to instance API

Add support for query parameters to find instances
with rb-name, rb-version, profile-name as query parameters
eg: curl IP:PORT/v1/instance?rb-name=test&rb-version=v1

Issue-ID: MULTICLOUD-662
Change-Id: If68c87885c8b8dd097cbb3860e8c7320310a6ef7
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Fix bug in tagging podTemplates

Fix a bug in tagging podTemplates
Using the runtime converter to convert
unstructured map data to podTemplateSpec now.

Issue-ID: MULTICLOUD-675
Change-Id: I661d3c63ae39b3533bb9c0a9934fc507284c7074
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Fix istio test scenario endpoint API validation"

Merge "Add top level make"

Add custom label to track created resources

Create a custom label on created resources
Also, create it on pods where pods are being
created.
This will help us later for filtering and querying
pods and resources.

Issue-ID: MULTICLOUD-675
Change-Id: I4b4fce7b67f9f27559d99dcca94a9191b96cb7c6
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Helm chart to deploy Istio-operator"

Merge "Update committers info"

Update committers info

Change-Id: I9f1e94d3b038cffca8b5980eb5f9f209a85021cb
Issue-ID: MULTICLOUD-751
Signed-off-by: Bin Yang <bin.yang@windriver.com>

Merge changes Ia35fac70,I7ffaa3d4,I53e7e4d6

* changes:
  Update plugin.sh with new helper functions
  Provide connection info for plugin testcase
  Correct k8splugin endpoint configuration in KUD test

Helm chart to deploy Istio-operator

Istio-operator is required to manage the
lifecycle and deployments of Istio

Issue-ID: MULTICLOUD-710

Signed-off-by: Pramod <pramod.raghavendra.jayathirth@intel.com>
Change-Id: Ifd2d05e790148096b5c0b454208e233aa5a299f9

Merge "Correct go version installed by k8s addons"

Merge "Add static compile time check to plugins"

Merge changes I46d47e64,I14d8eb6b

* changes:
  Add test case for loading vFW Helm chart
  Correct cni retrieval process from resource name

Merge "Minor improvement to log statements"

Add top level make

Added a top level Makefile which calls
all the sub packages under the src tree.
This will be called in the ci-management golang template
for this project.

Issue-ID: MULTICLOUD-666
Change-Id: I0027fd62e3665d6eae6733227871ed42bab2ca54
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Update plugin.sh with new helper functions

Plugin.sh has been refactored to use new wrapper functions as well
as utilize helm package command for creating resource bundle in
order to test proper handling of this scenario by plugin

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: Ia35fac70153fdb34ba75bfff31f9b2566b986cf1

Correct go version installed by k8s addons

Andrewrothstein.go galaxy role, that was responsible for
go installation was in too old tag to support demanded
(1.12.4) go version. It also blocked ovn-kubernetes addon
installation that's fixed now. Go version has been also
upgraded to 1.12.5

Issue-ID: MULTICLOUD-644

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I926bd061a361b2ae2efa2aecedf4fa6321f04cc8

Add test case for loading vFW Helm chart

This test checks vFW lifecycle using the k8splugin.

Issue-ID: MULTICLOUD-708

Change-Id: I46d47e64562d75d8b1f7552fc073b1e59efa443e
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Co-authored-by: Konrad Bańka <k.banka@samsung.com>

Correct cni retrieval process from resource name

Due to some minor issues, network resource name wasn't being
parsed properly, thus failing removing ovn networks when
deleting VNF instance.

Issue-ID: MULTICLOUD-708

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I14d8eb6b82d0ee74b266272fd8a842b45e6c2b40

Add static compile time check to plugins

Add a static compile time check to
plugins to make sure that any interface implementation
mismatch is caught at compile time rather than runtime.

Issue-ID: MULTICLOUD-666
Change-Id: I7f57774ad9384edb874cf64534f521014aea422f
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Changing Pull Policy for vFw Helm chart"

Changing Pull Policy for vFw Helm chart

Pull policy is "Always" because of that
VM/container images are pulled everytime
for each chart. This slows down the
deployment to several minutes in the
lab environment. Changing pull policy to
"IfNotPresent".

Change-Id: I3c01e681f010481221f1e720e6f09b317fbe9d87
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Issue-ID: MULTICLOUD-708

Merge "Provide ovn-central-address for local deployments"

Provide ovn-central-address for local deployments

K8splugin served with docker-compose doesn't utilize
ovn-central-address configured in host environment.
Function responsible for generating config file has been
modified to provide this information if available.

Issue-ID: MULTICLOUD-708

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: Ie3c27c68c02a0e206f6447e8c4330e18af942cd8

Merge "Add a listallhandler to definitions api"

Merge "Add port to ovn central ip"

Merge "Adapt network plugin to interface restrictions"

Merge "Add a list api for instances"

Merge changes from topics "listprofiles", "tagprofile", "tag_defn"

* changes:
  Add list api for profiles
  Make profile db tags specific
  Make definition tags specific

Merge "Improve provisioning central IP address for ovn tests"

Merge "Provide idempotent multus plugin configuration"

Add port to ovn central ip

Add port to the default OVN central ip
field.

Issue-ID: MULTICLOUD-666
Change-Id: Ib1644aafc00472769422dffd48a83ed7e53b2910
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Minor improvement to log statements

Removing some log statements that are cluttering the output.
Added a log statement to print errors during instantiation.

Issue-ID: MULTICLOUD-666
Change-Id: I76ad59c9cb36b5cb999573afec18e6f1adbed4ad
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Adapt network plugin to interface restrictions

Network plugin was not fully compliant with interface it should
implement due to type of one function argument. Relevant functions
have been adapted to utilize proper types.

Issue-ID: MULTICLOUD-708

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: Ic619ad476dba5e6582759290e18edceaa98c332f

Add list api for profiles

Add a list api for profiles for a specific
definition and version.
GET /v1/rb/definition/name/version/profile
will list all the profiles.

Issue-ID: MULTICLOUD-730
Change-Id: If1b8e6910c276a0f7139ab13340721c6ec8a49e8
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Add a listallhandler to definitions api

GET calls to the v1/rb/definition will
return all definitions and their versions

Issue-ID: MULTICLOUD-715
Change-Id: Ia0951ac83283830e475bf727e7d5ced7aab3add2
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Make profile db tags specific

Tags used in profile such as metadata
are too generic. Instead we are using specific
ones with the type prefixed.

Issue-ID: MULTICLOUD-730
Change-Id: I2f189d35bc202f4a71e0a92ca5b7188ebfd1cef2
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Make definition tags specific

Tags used in db such as metadata
are too generic. Instead we are using specific
ones with the type prefixed.

Issue-ID: MULTICLOUD-730
Change-Id: Ice76cf743f0782a3d7e76747cde1f0e0b116b14e
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Add a list api for instances

curl -X GET /v1/instance
returns all the instances created.
It returns abbreviated instances for improved readability.
For details on what resources were created
for each instance, use the ID with GET.

Issue-ID: MULTICLOUD-715
Change-Id: I05afe0fd2c254acbca4329289c81545f95c9fac5
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Remove kubeconfigdir. Use a tempfile instead"

Remove kubeconfigdir. Use a tempfile instead

Kubeconfigdir does not need to be a configurable
parameter. We just create a local file and use that
to create the config after which it is not needed.

Issue-ID: MULTICLOUD-614
Change-Id: I2df561d50b620e24c5ae5266b7200210e0c11caf
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

vFw Demo Helm chart issue

Fixing a typo in vFw Helm chart

Change-Id: I5e7657dba037e48539ac32a98f1731ec1ff942a8
Issue-ID: MULTICLOUD-708
Signed-off-by: Ritu Sood <ritu.sood@intel.com>

Improve provisioning central IP address for ovn tests

Function responsible for providing central ovn address has been
corrected to utilize already configured address instead of
checking it each time. Error reporting within the same function
has been also corrected and improved as well as iproute package
became used for address determining instead of net-tools' ifconfig

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I156297098b7919471df711f049f0d9439bd81420

Provide connection info for plugin testcase

Plugin.sh testcase was unable to instantiate VNF as it was
missing k8s backend configuration. This configuration is
now provided before instantiation and torn down on test end.

Issue-ID: MULTICLOUD-639

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I7ffaa3d4e2df84b38b50e11900406b55f94c6de8

Correct k8splugin endpoint configuration in KUD test

Plugin.sh was misconfigured to connect at wrong port
of k8splugin service.

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I53e7e4d6fdb727adc755318012ee262dbec811ef

Provide idempotent multus plugin configuration

Blockinfile task overrides marker section in order to provide
valid json content in file. Because of this, generated block is
added each time this playbook is run. This makes CNI config file
contain malformed content when launched more than once.

Issue-ID: MULTICLOUD-676

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: If1c98414be4fb3a5b6c1c63fada0bd934448f040

Merge "Move connectionhandler.go into api package"

Merge changes Id235f454,I0c6dd5e4

* changes:
  Correct plugin_edgex testcase
  Provide wrapper functions for issuing k8splugin API calls

Merge "Add getconnectivityrecordbyname function"

Fix istio test scenario endpoint API validation

Current query issued at the end of testcase to
istio service API didn't validate the case, when
http server responded with 5XX http code. By
reusing common wrapper for cURL, this is now
handled properly.

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: Ifb73494c9a6d98143edccaad095b2de5d456ef4f

Correct plugin_edgex testcase

Plugin_edgex.sh script hasn't been updated to few changes
that happened in k8splugin API and CSAR definitions. This
commit rewrites previous test scenario to be up to date with
current plugin state.

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: Id235f4549e10f57451bf2b6195df8c95230e1c7e

Provide wrapper functions for issuing k8splugin API calls

In plugin.sh and plugin_edgex.sh test scenarios, there are
executed many curl calls to API endpoint. As curl's default
behavior for handling 4xx and 5xx status code is not perfect,
wrapper functions have been created to easily ensure correctness
of issued API call.

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I0c6dd5e4f896972245a6fa6c8c2a10d8df8ec406

Create dir before extracting archive

Some archives don't include a directory entry.
Eg: tgz archives generated by helm package.
This bug fix checks that a directory exists before
an extracted file is created there.

Issue-ID: MULTICLOUD-705
Change-Id: If6720948d470b83786901574f5d8d3227835a047
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Reduce process count in test

CI seems to be failing with out of memory issues
This reduces the multiprocess to 2 for the go test
command.

Issue-ID: MULTICLOUD-666
Change-Id: I3f70ecad51cc16f8972346882d5d98fc803427ef
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Fix helm installation conflict

Kubespray installs helm client on kubernetes master. The same client
has to be installed on host running installation playbooks. In single
node scenario, local host already has helm client installed by
kubespray, thus leading to conflict due to way of provisioning.
This helm installation has been moved to global configure playbook, as
well as corrected, not to fail on single host deployments.

Issue-ID: MULTICLOUD-690

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I1ef779ed0f2fde82758ce9e229c3f5bb015b2aeb

Move connectionhandler.go into api package

Move connectionhandler.go into api package
This brings it inline with the other API handlers

Issue-ID: MULTICLOUD-666
Change-Id: Ia3b832159f537c6d8d8ecc93b56bc9b085649f7e
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Add getconnectivityrecordbyname function

Plugins need this connectivity information
Adding code to retrieve it

Issue-ID: MULTICLOUD-688
Change-Id: I5fa1b69cdad754a432316edc4188c19b7fe10f84
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Change the import paths for go get access"

Change the import paths for go get access

This allows other projects to do a go get and use this repo.
go get github.com/multicloud-k8s/src/k8splugin
will do the job.
import github.com/multicloud-k8s/src/k8splugin
will import the root package

Issue-ID: MULTICLOUD-666
Change-Id: I2f41c88c5a7fb3305b78cce8a83a0e102baa444c
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Correct installer.sh to enable testing during deployment

Issue-ID: MULTICLOUD-686

Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I02724ca72a217cbb5be8cfee5edf136f4bf8ce2c

Make aio.sh preserve host's ssh keys if possible

aio.sh script creates rsa key that is also added to host's authorized_keys.
Currently, existing id_rsa keys are removed in order to generate fresh key each time.

This change makes setup script reuse existing key when possible.

Issue-ID: MULTICLOUD-676

Change-Id: I3b6b837f64dabd3cb6a14aa5ea65e67c6b8db6a8
Signed-off-by: Konrad Bańka <k.banka@samsung.com>

Unify resolving current directory in KUD scripts

Issue-ID: MULTICLOUD-685

Change-Id: I62199f64da360627a13c467139a39430ae389167
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Adding ci-kud-installer wrapper

this wrapper is new entry point for CI job triggering aio.sh
execution.

Change-Id: I650675c9cfff396010bf3a1d36e9840dedf9d403
Issue-ID: MULTICLOUD-635
Signed-off-by: Michal Ptacek <m.ptacek@partner.samsung.com>
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Merge "Multus error when the namesapace is not default"

Merge "Add ovn-networkobj for Multus"

Merge "Plugin code refactoring"

Merge "Make ovn-central network interface configurable"

Merge "Enable mitogen to speed up Ansible execution"

Multus error when the namesapace is not default

Multus has a limitation that namespace
in which network is defined has to be
explicitly provided. As we are defining
network in "default" namespace, that needs
to be provided in all deployment files

Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Issue-ID: MULTICLOUD-663
Change-Id: Iefc17691fa909843f12fa35693900a063a6d20ae

Add ovn-networkobj for Multus

Add ovn custom resource for Multus as part
of installation

Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Change-Id: I4e01a06ba76515fa271790b461f473045eb174a2
Issue-ID: MULTICLOUD-670

Make ovn-central network interface configurable

installer.sh script is configuring ovn-central address <ip>:<port>
to environment variable.
Ip address is detected from the ovn-central host through ansible
by grepping ifconfig output, but in this step detected network
interface is hard-coded in the code to be "eth1".

This commit makes network interface configurable in aio/vagrant
deployment.

Issue-ID: MULTICLOUD-621

Change-Id: Icbe22ffdcb45c5f16067ea609eec76fb70c0ea78
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Merge changes I798b9dd9,I36b65dd2

* changes:
  Remove the requirement to run aio.sh as root
  Documentation update for aio.sh CI change

Merge "Make aio.sh suitable for CI"

Merge "Starting of k8splugin with docker-compose is broken"

Enable mitogen to speed up Ansible execution

Signed-off-by: Victor Morales <victor.morales@intel.com>
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>
Issue-ID: MULTICLOUD-301
Change-Id: I427145e26eabc92ab19b0a143b759113dbfbeeea

Plugin code refactoring

The plugin code has been refactored to
implement a common interface.
This will allow us to do plugin validation
at loadtime of the plugin instead of at runtime.
This also makes the code calling the plugins cleaner
and easier to read.

Issue-ID: MULTICLOUD-557
Change-Id: Ice2bcc9b850d7c0e1707dcc42132c63dd77472a7
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Remove the requirement to run aio.sh as root

However user needs to be in sudoers without password
needed to able to run needed steps with sudo.

Issue-ID: MULTICLOUD-672

Change-Id: I798b9dd9dad4175c2b43700bfe801d34b578fe97
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Documentation update for aio.sh CI change

Update documentation accordingly.

Issue-ID: MULTICLOUD-629

Change-Id: I36b65dd270a25d63884d161163cfa8e7a813d241
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Make aio.sh suitable for CI

aio.sh script currently clones multicloud/k8s repo itself.
That is unnessary and makes it impossible to hook k8s repository
to CI system where commit changeset is used to define repository
files to be tested.

This commit will remove cloning of multicloud/k8s repository
and also move aio.sh to baremetal folder which seems to be
the intended place for it.

Issue-ID: MULTICLOUD-629

Change-Id: Ifc54bbc1ec06bc4c3fbbecca7e676d5ffa392d78
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Fixing kubectl conflict on KUD addons

Removing kubectl (andrewrothstein.kubectl) dependency
that conflicts with kubectl installed by Kupespray.

Copy kubectl installed by Kubespray also to host
running Ansible.
That needs kubectl_localhost: true in Kubespray configuration
to make it copy binary to localhost.

Issue-ID: MULTICLOUD-667

Change-Id: I8c5f56488a9f559c4358cea5ad56fa23b26ec6aa
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Remove andrewrothstein.kubectl role

Removing andrewrothstein.kubectl role
which was causing issues in aio
baremetal scenario

Change-Id: If3c8c71319c4b14dedfa5997881307e5424fd453
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Issue-ID: MULTICLOUD-301

Starting of k8splugin with docker-compose is broken

docker-compose.yml and start.sh script cannot be used
in installer.sh to start k8splugin and mongo.

This commit modifies start.sh script to be usable in
installer.sh to start plugin before tests.

Also fixed kud_tests variable in installer.sh.

Issue-ID: MULTICLOUD-640

Change-Id: I60cc850d6f5b58df574f79ad3c65b801c59b3a14
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

Fix IP address for sink pod

IP address was incorrect for
sink

Change-Id: I28962cedeecd25b136eecf39786b4daed6ae9c26
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Issue-ID: MULTICLOUD-653

Merge "Remove env variable"

Merge "Update vFW Helm chart"

Merge "Install jq as an dependency for yq"

Remove env variable

Remove ENV variable. Somehow I missed
removing this variable when we moved to configuration
files.

Issue-ID: MULTICLOUD-653
Change-Id: Ibf2238791dfcb1e66c7b9cca7b6d25e3d30bb0e4
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>