Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
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>
|
|
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>
|
|
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>
|
|
|
|
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>
|
|
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>
|
|
|
|
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
|
|
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>
|
|
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>
|
|
|
|
|
|
-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 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>
|
|
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>
|
|
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>
|
|
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>
|
|
|
|
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>
|
|
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>
|
|
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>
|