Merge "Update package path to use github"

Update image version of k8splugin

Update image version of k8splugin to 0.5.0-SNAPSHOT
This will be then be used to create the release image

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

Remove GOPATH

Remove GOPATH in docker build script

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

Update package path to use github

Update the package path to use the standard github.com
path instead of making it a local path.
This allows import from other go projects.

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

Merge "Remove ovn4nfvk8s network plugin"

Merge "Add a namegenerator package"

Remove ovn4nfvk8s network plugin

As ovn4nfv networks are now handled on k8s side by created CRD, this
plugin is no longer necessary. Removed all references to ovn

Issue-ID: MULTICLOUD-733
Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I7ad6c6d9b3b4fd8f249796f437e69c7df4e701cc

Fix test failure in threaded tests

Fix bug in tests where they can fail in highly
threaded environments where a file can be accessed by two test
processes.
Fixed it by duplicating the test case.

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

Remove dependency on git.apache.org

Remove dependency on git.apache.org

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

Merge "Use Multus Daemonset for installing Multus in KuD"

Merge "Remove double thread restriction on build"

Merge "Remove gopath from builds"

Merge "Adding informers for CRD"

Merge "Adding listers for CRD"

Use Multus Daemonset for installing Multus in KuD

Currently KuD uses Ansible scripts for installing Multus.
Multus has a daemonset that should be used for installing the multus
as part of an add-on.
This is also helpful for KuD offline deployment in the future.
Removed the comment. Updated the images path and removed
error supression addressed by comments.

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

Remove gopath from builds

Remove gopath from builds as go mod
does not require them.

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

Remove double thread restriction on build

The double thread restriction on the build was introduced
to alleviate runtime memory exceptions.
Now that we have a larger VM for builds, this is not required.

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

Merge "Adding Helm charts of Multicloud-K8s"

Merge "Add helm chart for Keycloak server"

Add helm chart for Keycloak server

Keycloak is an Authentication and Authorization
oauth2.0 server

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

Adding informers for CRD

Adding generated informers for CRD
These informers will be needed when we want to
watch these CRDs from the ONAP cluster
Generated using generate-groups.sh
from the kubernetes/code-generator repo

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

Adding listers for CRD

Adding generated listers for CRD
Generated using generate-groups.sh
from the kubernetes/code-generator repo

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

Adding typed client for CRD

Adding a typed client for CRD
Generated using the generate-groups.sh script from
the kubernetes/code-generator repo

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

Merge "Adding monitor operator to monitor edge resources"

Adding monitor operator to monitor edge resources

Add an operator to monitor resources at the edge location.
The operator listens to pods and services right now and stores
their information in a CustomResource

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

Merge "Remove unsused variables from kud-vars"

Remove unsused variables from kud-vars

ovn4nfv url and other information not
required now after the patch
https://gerrit.onap.org/r/#/c/multicloud/k8s/+/93602/

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

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

Adding Helm charts of Multicloud-K8s

To have seperate copy for ONAP4K8s

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

Add a namegenerator package

Adds a namegenerator package which generates
readable names for instances.
This will go with a small change in the API
where we expect instance names and will generate
the names if they are not provided.

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

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"