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>

Upgrade mongo driver to 1.0

Mongo driver 1.0 was released recently
Upgrading from 0.2.0 which we were on right now.

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

Merge "Add support for helm templates"

Push latest tag

Push latest tag during the nightly docker build
along with the snapshot version that we already push

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

Add support for helm templates

Add support for evaluating helm templates
The interface provides a way to get a map which
contains a mapping from kubernetes kind to the
corresponding yaml file that defines a resource
of that kind. This map is then provided to the
instantiation code at instantiation time to create
in the kubernetes cluster.
P5: Use filepath.join instead of strings.join
P9: rebase with new folder structure
P10: moved the helm code into its own package.
P12: Add unit tests
     update the go.mod to use latest docker version

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

Merge "Add support for parsing profile yaml files"

Add support for parsing profile yaml files

Add code to parse profile configuration yaml
The parsing function is global and returns a client
which can then be used to get or apply specific parts
of the configuration on top of an extracted helm chart.
P14: Add unit test that covers both ProcessProfileYaml
     and CopyConfigurationOverrides
P15: Adding mock_charts and mock_profiles
     We expect to reuse these files for other unit tests

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

Detect chart name

ChartName should not be mandatory field
If not provided it will be detected by inspection
of the tar.gz archive.

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

Merge "Add test helm chart for testing"

Add test helm chart for testing

Add a test helm chart for testing the end to end
flow. This chart brings up a service and a deployment.
The deployment brings up two containers, vault and consul.
Vault and Consul are brought up in dev mode.
The helm chart also has a subchart, common, which contains
templates that are used within the main chart.
Added an override deployment.yaml which creates only one
container instead of two
Fixed plugin.sh and tar file creation steps

Issue-ID: MULTICLOUD-291
Change-Id: I7eb10c9f43f8a4e09938139b5e2edee5d59feb3b
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
Signed-off-by: Victor Morales <victor.morales@intel.com>

Switch to xenial Docker image

The Ubuntu Xenial Docker image has lower footprint than Trusty,
besides more security patches. This change modifies the Dockerfile
that is used for building the Docker image.

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

Update setup.sh script

This script was changed some instructions which was reported by
shellcheck tool and fix some non-ubuntu instructions.

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

Cleanup kubespray config values

There were some kubespray configuration values that are not needed
and some are causing conflicts with the dashboard. This change
removes them.

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

Updating virtlet version

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

Add cFW scripts folder

The Dockerfiles that are used for building the Docker images during
the testing the Hybrid Firewall ONAP use case are hosted in an
external repo. This change includes the latest working version for
those scripts.

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

Merge "Fix setup.sh sudo permissions"

Change Docker execution permissions

The k8s pluging is running with root user in the Docker image, this
execution can compromise the host system. Therefore, it's necessary
to explicit change those permissions.

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

Merge "Updating kubespray version."

Updating kubespray version.

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

Fix setup.sh sudo permissions

There are some instructions in the setup.sh script file which require
the addition of sudo instruction. This change fixes them.

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

Update committer INFO

Remove those committers who decided to step down as a committer

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

Adding the keyword SNAPSHOT to build script.

This fixes the version tag for K8s plugin modifying the
image name.
Change-Id: Ie88279200607dfe4338b2490647cf02a4e2d056c
Issue-ID: MULTICLOUD-301
Signed-off-by: Akhila Kishore <akhila.kishore@intel.com>

Remove external script references

The integration_* test scripts have some references to external
repositories that are out of the ONAP project scope. This change
removes those references to keep everything defined in this repo.

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

Add vFW scripts folder

The bash scripts that are used for testing the Firewall ONAP use case
are hosted in an external repo. This change includes the latest
working version for those scripts.

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

Add net tag to Makefile

The change that removes duplicate plugins make actions
I0efecdcb70052b4c0d66102c56e0c715481cb34a didn't consider
the flag for net module which it's needed during the runtime.

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

Add support for downloading content

Add support for downloading content for creating
the merged helm charts. This api will be used mainly
by the profile api to create a converged chart which
will then be created by the instantiation code
P2: Add unit tests for archive.go
    Add download method for profile.go
    Update comments for download method
P3: Add unit test for empty files
P4: Add unit tests for Download functions
P5: Rebase against new folder structure

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

Merge "Remove duplicate plugins make action"

Move api.go functions

CheckEnvVariables, CheckDatabaseConnection, LoadPlugins and
CheckInitialSettings functions are utility functions which are not
related with the API. These functions need to be moved to utils.go
file.

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

Use a standard Go project layout

This project wasn't following some Standard Go Project Layout
guidelines(https://github.com/golang-standards/project-layout). This
change pretends to organize the source code and following those
guidelines.

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

Merge "Add another parameter to the definition"

Add another parameter to the definition

Add a ChartName parameter to definition
This is need to explicitly specify the chart name
to be used within the tar file.

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

Remove duplicate plugins make action

The action which compiles all the plugins is duplicated in the
Makefile. This change remove that duplication to define a
consistent build process.

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

Version update.

Updating the version of galaxy role kubectl to latest.

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

Add support for tags in the mockdb

Add support for tags in mockdb.
Needed for supporting unit tests where a method
does multiple db calls using same uuid but different
tags. Eg: Download methods

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

Merge "Fix sorting with right index value"

Merge "Delete content when a definition is deleted"

Merge "Enable statd service"

Fix sorting with right index value

Fix the less function used in sort.Slice
with the right index.
I am using the same index which obviously will not work.

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

Delete content when a definition is deleted

When a definition is deleted, delete the content too.
Not deleting the content would leave blobs of data that is not
connected to any metadata in the mongo database.

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

Account for strings when reading from db

Strings are not returned unmodified by the
mongo driver and seem to have a size value prepended to them.
Using the built in string reader resolves this issue.

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

Merge "Add resource bundle profile api"

Add resource bundle profile api

Add CRUD api for uploading profiles for specific definition
resource bundles.
- Adding unit tests

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

Minor refactoring of definition unit tests

The definition unit tests needed some small changes to
ensure consistency between the source and the tests.
P3: Sort both the slices before running DeepEqual

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

Fix order of db create call

definition upload code call db create in wrong order
This patch fixes that.

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

Merge "Check if no document is returned before error"

Bump mongodriver to 0.2.0

Bumping mongodriver version to 0.2.0
This is a driver that has frequent updates
and we need to keep on top of those changes

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

Check if no document is returned before error

Check if no document is returned in SingleResult
before throwing any error.

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

Enable statd service

NFS uses rpc-statd service to provide locks. These service needs to
be started separately. This fix enables that service to prevents
'no locks available' message errors.

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

Merge "Refactor docker-compose variables"

Bump vagrant version to 2.2.3

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

Refactor docker-compose variables

The default environment variables required by docker-compose.yml
file was exported to .env file to have better control on them.

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

Merge "Bump kubernetes client to 1.12.3"

Bump kubernetes client to 1.12.3

Bumping up the kubernetes client library used
to kubernetes 1.12.3
We will move it to v10.0.0 once our k8s deployment
moves to 1.13.0
Bumping kubectl version to 1.12.2 as andrewrothstein/ansible-kubectl
does not have 1.12.3 in its main.yml

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

Merge "Bump virtlet to 1.4.3 and criproxy to 0.14.0"

Merge "Add Network and OVN4NFV Plugins"

Merge "Migrating from consul to mongodb for backend"

Migrating from consul to mongodb for backend

Migrating to mongodb from consul.
The main reason being the value size limitation of 512kb in consul.
See https://jira.onap.org/browse/MULTICLOUD-426 for details.
This requires a little bit of hierarchy management
and data management.
We are no longer converting structs to json encoded
strings. The underlying db supports structs without any modifications.
Also, since Mongo has the concept of collections, each submodule can
use its own collection for storage as needed.
Definition uses a collection called rbdef right now.
P10: Enabling unit tests for mongo.go. This requires the usage
     of aliased functions.
P11: Expanded unit tests for all functions in mongo.go
P12: Refactored parameter validation.
     Removed TestHealthCheck as we are not mocking any of the
       db commands right now
     Checking return value of read with an expected value
P13: Adding back consul support.
     Fixing functional test
     Full consul implementation check and modifications is
     being tracked by MULTICLOUD-427
P15: Fix ReadAll unit test and corresponding code
     ReadAll now returns error when no objects are found

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

Fix issue with pip install

Pip install not preserving environment

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