Age | Commit message (Collapse) | Author | Files | Lines |
|
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 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>
|
|
|
|
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
|
|
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
|
|
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>
|
|
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
|
|
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>
|
|
|
|
|
|
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>
|
|
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>
|
|
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>
|
|
|
|
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>
|
|
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>
|
|
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>
|
|
|
|
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 in SingleResult
before throwing any error.
Issue-ID: MULTICLOUD-435
Change-Id: I9d92ff325701c04af278ba7c9e2bf5a55fd4d9bd
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
|
|
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>
|
|
|
|
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>
|
|
This patch includes support for Network Objects through a new plugin.
It also add the first sub-module plugin for OVN4NFVK8s support.
Change-Id: Ia23c42d50f75a5206e1b6a04052c34e940518428
Signed-off-by: Ritu Sood <ritu.sood@intel.com>
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-303
|
|
Upload is a seperate API where it takes
a binary stream and stores it. The api supports
tar.gz file format only.
P2: Check if ID is valid before trying upload
Add test with an invalid ID
Issue-ID: MULTICLOUD-393
Change-Id: Id636a95823a046e1795d3be72d0214e953a8c5fc
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
|
|
k8splugin manages deployment of resource bundles
and these are not restricted to vnfs.
This names' change is to reflect that functionality.
P2: using rb instead of resource bundle
Issue-ID: MULTICLOUD-410
Change-Id: I09e0b92a8fc13562e1c6bb17dc8bc13de97264d7
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
|
|
This change pretends to increase the code coverage creating Unit
Tests for the interactions with the Databases.
Change-Id: I3b78ebe8ddb131e3c06bcee0065ad5eabeed5677
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
Adding APIs for POST, GET, LIST (implemented via GET)
and DELETE commands on /v1/vnfd base for creating,
getting, listing and deleting VNF Definitions.
P2: Added unit tests for vnfdhandler.go
P3: Add unit tests for serialize and deserialize
P4: Integrating review comments
P5: Added customizable mocking for vnfdhandler_test
P6: Added customizablt mocking for vnfd_test
Note that this will soon need to be updated once
the db changes go through in patch 71090
Issue-ID: MULTICLOUD-393
Change-Id: Id509bed370ab3bdc572c6ead22324c1ee3dbf82d
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
Signed-off-by: Victor Morales <victor.morales@intel.com>
|
|
Migrate from dep to go mod which is
native package management functionality in golang 1.11
Issue-ID: MULTICLOUD-392
Change-Id: I9a94cb38bd230738d7d0e586e6956b7a46472dd7
Signed-off-by: Kiran Kamineni <kiran.k.kamineni@intel.com>
|
|
|
|
The build.sh file didn't include the GOPATH variable which in some
cases was causing some building failures. This change includes a
source instruction at beginining of the file.
Change-Id: I718411fd90ede74decbaac7efc5c065aa5a853fc
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
|
|
|
|
|
|
|
|
The current implementation of Unit Tests for Services wasn't
validating sucessful cases properly. This change fix that gap and
increase the coverage number.
Change-Id: Ib5daa607f38e490ced471a129a5fa71b2d4eb133
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The current implementation of Unit Tests for Deployments wasn't
validating sucessful cases properly. This change fix that gap and
increase the coverage number.
Change-Id: I9b891d700c152c9b57d6b5858867a7439ac83f3f
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The cover action target allows the creation of code coverage report
during the Jenkins Job execution, giving an idea of the impact of
the change submitted.
Change-Id: Ia2b8d57a0f9235ace021baaa6f74370b6656e58f
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The current implementation of Unit Tests for Namespaces wasn't
validating sucessful cases properly. This change fix that gap and
increase the coverage number.
Change-Id: Iddc0aca3b52ea8ffebca7dccfc94d4d99e052b06
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The KRD plugins module wasn't cover by Unit Tests that ensure
their functionality. This change create Unit Tests that guarantees
basic use cases.
Change-Id: Idac9179bfb7b805ebadc60d9d1a41e73a6f13be7
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The I1e1eb40f1a18ba33c74069a117462c8df17767ac change standarizes the
Interface for the plugins, this changes affects callers and it's
necessary to update those references. This change fix that issue for
Namespace Kubernetes resources.
Change-Id: I04c957fdd526401523b72e332b6e24385a7ceefe
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
Deployment, service and namespace are plugins which offers CRUD
operations to manage their resources. They haven't implemented
Unit Tests which makes fragile to change/refactor the source code.
This change adds their corresponding Unit Tests and defines a
standard interface.
Change-Id: I1e1eb40f1a18ba33c74069a117462c8df17767ac
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
|
|
The current tests can be separated in two categories: unit test and
integration tests. The criteria depends on the scope or dependencies
required for the test. This change provides a mechanism to test
only one category.
Change-Id: Ib7bac49cb0b2d14544fb9704345436a7ba71aead
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The metadata.yaml was using a list of resources instead of a set. As
result, it's possible to add resources that are duplicated. In order
to avoid this the ResourceTypePathMap field was changed.
Change-Id: Ic454a9c42fa367b58580641b62b3d1c7ac33b1ca
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The no_proxy configuration value was missing in the docker-compose.yml
this was causing a different behaviour when the functional tests were
running and consuming the services on the container.
Change-Id: Ie88e88d50e0ec9c4aa0d03367f2a9d25edccb753
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The current architecture offers the possibility to add more plugins
and extend the its functionality. This flexibility has to be supported
during the compilation process. This change modify the Makefile to find
plugins and compile them.
It also adds a start.sh script which allows to run MultiCloud services
without the need of packaging their binaries in a container.
Change-Id: I9271c51daa4fd9156717e37f1818886c72e36c27
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
The instruction that compiles the Mock plugin is only required during
the testing, therefore this instruction can be moved in different
place in the Makefile
Change-Id: Ib800b2b39992a1020e466aa3f22cada5ac84bcf6
Signed-off-by: Victor Morales <victor.morales@intel.com>
Issue-ID: MULTICLOUD-301
|
|
|
|
This patch makes sure PLUGINS_DIR is not optional since its better
explicitly pass the plugins directory.
Change-Id: I08da5cf1563527d8b713fa106aa05b37a26c95cb
Issue-ID: MULTICLOUD-301
Signed-off-by: Shashank Kumar Shankar <shashank.kumar.shankar@intel.com>
|