Variable used before intialization

Initialize variable before using

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

Change format of the network file

Format of network file is different
between KUD and Plugin. Change the
format to match the format in KUD

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

Merge "Make aio.sh rerunnable"

Make aio.sh rerunnable

aio.sh script can be run only ones.
Cloning of k8s on top of existing dir does not work
and ssh key generation on top of existing key does not work.

Issue-ID: MULTICLOUD-620

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

vFW test case wrong urls and version

In vFw test case some urls are incorrect
and the vFW scripts release version
doesn't exist anymore

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

Merge "Move OVN to the latest version"

Merge "Disable building of K8s plugin in KUD deployment"

Source /etc/environment for tests

After KUD deployment (at least aio.sh),
if running related tests immediately in the same shell session,
system wide environment variables (/etc/environment) are not effective.

This commit sources /etc/environment in commonly used _functions.sh.

Issue-ID: MULTICLOUD-619

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

Disable building of K8s plugin in KUD deployment

K8S plugin is part of ONAP and should not be instantiated by KUD.
It only needs to be instantiated by OOM of ONAP.
Changing the default value.

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

Check always returns true for install_plugin

The [[ test for KUD_PLUGIN_ENABLED always returns true.
Removed [[ to use the boolean operation instead.

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

Update Readme to reflect the new golang version

golang version used is 1.12.4. This patch updates
the README.md to reflect that.

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

Merge "update golang version for kud"

Merge "Fix pip install/upgrade logic"

update golang version for kud

Use the 1.12.4 version of golang for kud instead of
the older 1.11 version.

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

Make service port configurable

Service port should be configurable.
This patch removes the hardcoded value.
The default value is 9015

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

Fix pip install/upgrade logic

installer.sh has reverse logic for install/upgrade pip
after checking if pip is installed.

Issue-ID: MULTICLOUD-607

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

Create kubeconfig files in kubeconfig dir

The connectivity api should allow the creation
of kubeconfig files in the kubeconfig dir.

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

Merge cloudregionid and connectionname

Merge cloudregionid and connectionname to form cloudregion

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

Merge "Fix multus and ovn tests"

Merge "Use the unstructured type for decode"

Merge "Adding healthcheck for OOM testing"

Merge "Update info.yaml to add new committer"

Adding healthcheck for OOM testing

Add a heathcheck endpoint for OOM testing

Issue-ID: ONAPARC-363
Change-Id: Id5f77478d331fedb19ac162389d95852e10c5022
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Use the unstructured type for decode

Use the unstructured type in the into field.
This allows us to remove the convert part of the
code and also allows decoding for CRDs.

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

Updated go.sum created from 1.12.4

Updated go.sum generated from build based on 1.12.4

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

Update info.yaml to add new committer

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

Merge "Remove deployment plugin"

Remove unused variables dockerfile

Remove unused environment variables from
dockerfile

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

Remove deployment plugin

The deployment plugin does not support managing
resources based on the old v1beta1 resource path.
This is supported by the generic plugin.
We will re-enable this plugin if there is a need for
special actions for deployments.

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

Move OVN to the latest version

Chnage commit id for OVN

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

Fix multus and ovn tests

The tests were reporting success even when the
interfaces were not created correctly

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

Merge "Add plugin to the folder names"

Merge "Move configuration into config files"

Merge "Updating Multus to v3.3-tp"

Add plugin to the folder names

Make the folder name as k8splugin in keeping
with the repository name and other references.

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

Move configuration into config files

Move k8splugin configuration into config files
instead of using environment variables.

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

Merge "Virtlet Image link broken"

Merge "Support non-namespaced objects in generic plugin"

Virtlet Image link broken

Fixed link

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

Updating Multus to v3.3-tp

Multus team created a tech preview for
Virtlet issue. Moving KUD to that release.

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

Support non-namespaced objects in generic plugin

Generic plugin is not generic enough right now.
It was assuming that objects are namespaced.
This patch supports both types based on the
Scope field in the mapping.

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

Fix docker build

Docker build is broken. This patch fixes it.

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

Replace Kind with GroupVersionKind

Kind is not unique to track resources in Kubernetes
GroupVersionKind is unique. We are just using that to
track our data. It is abstracted behind a couple of new
types for templates and resources.
This change makes a lot of the old kind based operations
redundant and simplified.

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

Rename plugin reference interface

Rename the plugin interface to something
more relevant. KubernetesResource will be
used to describe resources for creation.

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

Fix build error

Build is broken because of some changes that were
merged without being rebased on dependent patches.

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

Merge "Add Connectivity Info API"

Merge "Day 2 Configuration API's"

Add Connectivity Info API

API to add connectivity information for
Cloud Regions to K8s Plugin
https://wiki.onap.org/display/DW/MultiCloud+K8s-Plugin-service+API%27s

Change-Id: I0a9166df9b076a7fdacf0b2f9e51a9cdebe4b621
Signed-off-by: rsood <ritu.sood@intel.com>
Issue-ID: MULTICLOUD-292
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>

Merge "Provider network support when OVN is used"

Merge "Add support for so integration"

Add support for so integration

Add support for the multicloud-api
that is called by SO for instantiation.

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

Merge "Add option to start https listener"

Merge "Using the --chown flag to reduce image size"

Add option to start https listener

Added support for tls connection for k8splugin.
It will load the certificates from its current folder.
Any installer needs to just put these files in the same
folder as the executable and it will pick them up and start
in https mode.

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

Day 2 Configuration API's

This patch adds Configuration API's
https://wiki.onap.org/display/DW/MultiCloud+K8s-Plugin-service+API%27s

Change-Id: I52ebfc5aa980ec8af4a31569d569216e9a2a760c
Issue-ID: MULTICLOUD-464
Signed-off-by: rsood <ritu.sood@intel.com>

Using the --chown flag to reduce image size

Patch to reduce the size of the docker image
for multicloud/k8s image
Reduces size by about 300mb

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

Provider network support when OVN is used

Example script is provided to create
OVN provider networks. Test case is
provided to place workloads on the OVN
provider network using ovn4nfv-k8s-plugin

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

Add support for generic plugin

Add support for generic kinds
We are currently using a known map to find the
group version resource for a kind.

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

Export kubernetesClient

Export kubernetesClient. There are situations where
this will need to be accessed in the plugins and we need
the definition to be exported for that.

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

Merge "Fix integration test plugin.sh"

Use consistent naming for Name and Version

Definition Name and Definition Version are now
using the consistent naming similar to the Profile
and instance structure.
Name becomes RBName
Version becomes RBVersion

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

Fix integration test plugin.sh

Update the integration test plugin.sh
with the latest api changes to instance
management.

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

Use common code for executeRequest

Use the common executeRequest function instead
of multiple copies of the same lines for test requests

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

Merge "Add url path tests to definition handler"

Merge "Add url path tests to profile handler"

Add url path tests to profile handler

Improve the profile tests by testing the
url path parameters as well.

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

Add url path tests to definition handler

Url path parameters are not tested in the current
unit tests. This is fixed with the use of the same
router that is used by the main program.

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

Refactor instance code

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

Minor update KRD to KUD

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

Merge "Make profile key explicit"

Remove build tags

Remove build tags as this is causing issues where
mock plugins need to be loaded into files that do
unit testing as well.
Full integration testing is being done in the CSIT.

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

Make profile key explicit

Using profilename instead of name in the db key
protects it from getting overwritten by anything
else that might use name as a key.
Using explicit key names makes sure that we don't
have to add bson structure tags in the code.

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

Fix merge issue

The TestUpdate method was updated properly with the
merge on to master. This fixes that issue.

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

Merge "Fix document typo in readme.md"

Merge "Add update method to db interface"

Fix document typo in readme.md

Fix some small typos in the readme.md file

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

Add update method to db interface

Add update interface to the db.
This will allow us to support PUT http methods
in the future.

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

Update definition and profile to latest spec

Bringing all the definition and profile code upto the latest spec.
Integrated the end to end instance code changes that were made.
P9: Added updated plugin.sh with updated uri paths based on spec

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

Merge "Fix namespace creation"

Merge "Add support for composite keys"

Merge "Enable random_hostname vagrant-libvirt value"

Fix namespace creation

Namespace is being created by k8splugin
if it doesn't exist. This is done  before
other resources are created in kubernetes.
This patch removes check that failed if
namespace didn't exist

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

Add support for composite keys

Composite keys help us store objects which are unique
for a given set of pre-existing objects.
Eg: Many profiles can exist for a definition and its
key will have a definition name as a part of the composite key.
P2: Use a predefined interface for keys instead of generic interfaceP{}
P3: Add check for empty strings in stringer interface
P5: Add appropriate keys in other packages.

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

Updating the path inside README.

Correcting the path to diagram.png on the Readme in KUD folder.

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

Enable random_hostname vagrant-libvirt value

This configuration value allows to generate Libvirt instances with
randomly domain names generated. As result, it's possible to use this
project in the same server but in different folders at the same time.
As part of the change, it was necessary to centraliza some Libvirt
common configurations.

Change-Id: Ie6adf3f40de7a8cced875b0c899738f9f6449181
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301

Merge "Use httptest instead of http in unit tests"

Merge "Restructuring the repo."

Use httptest instead of http in unit tests

Use httptest instead of http in unit tests similar to:
https://golang.org/pkg/net/http/httptest/#example_ResponseRecorder
Update empty body checking to account for change

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

Restructuring the repo.

The idea is to restructure the existing repo create a deployment
independent of Vagrant or other hosting providers.
Renamed KRD to KUbernetes Deploy(Kud) including the ansible scripts
Added new path to functional tests.
Moved samples pdfs to sites.
Minor changes to Readme.
Updated aio.sh, moved sample config
Corrected other nits. Updated and verified test cases.
Addressed comments and changes associated with it.
Updated Readme and minor change in Vagrantfile.
Validated test cases again. Moved aio.sh into vagrant folder.
Added new README for each hosting provider and project on the whole.
Updated the installer script with relative path.
Updated the name to deployment_infra, moved the cFW sripcts to tests.
Updated the gitignore file.

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

Merge "NewRouter supports custom backend clients"

Merge "Remove -a build flag"

Merge "Bring in all the other helper code"

Merge "Cleanup kubespray config values"

Merge "Update setup.sh script"

Remove -a build flag

-a forces rebuilding of packages
that are already up-to-date. This is
slowing down builds to several minutes

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

NewRouter supports custom backend clients

NewRouter needs to support custom clients
This is needed where the backend clients are mocked
and we need url path parameters to be available in our
unit tests.
Using the same router code allows us to do this.

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

Workaround for Kernel Package Issue

Due to Ubuntu Kernel package issue
KRD is not installing.
https://bugs.launchpad.net/ubuntu/+source/linux-base/+bug/1820755

This is a workaround to not install
those kernel packages

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

Merge "Switch to xenial Docker image"

Merge "Add cFW scripts folder"

Bring in all the other helper code

Bring in all the helper functions added for end to end
integration. This allows the api level upload of helm charts,
profiles and also instantiation of said helm charts.
P3: Plugin index is based on lowercase kind name
    whereas the map contains the correct case for kind.
    Convert to lower case before loading the plugin.
Changes after rebasing on the new folder structure.
Rebasing over the new folder structure
P8: Add unit tests for Resolve function
    Fix the integration tests for createvnf
    I had to add a huge blob of base64 encoded data
    based on the profile and sample helm chart to test
    the flow.
P12: Update the integration test with the rb_profile_id parameter

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