Switch instantiation parameters source from user to sdnc parameters 49/98849/1
authorKonrad Bańka <k.banka@samsung.com>
Tue, 26 Nov 2019 10:21:39 +0000 (11:21 +0100)
committerKonrad Bańka <k.banka@samsung.com>
Tue, 26 Nov 2019 10:21:39 +0000 (11:21 +0100)
Issue-ID: MULTICLOUD-941
Signed-off-by: Konrad Bańka <k.banka@samsung.com>
Change-Id: I3a6c67eb442023f64600b65f5af54a49117a8c51

src/k8splugin/api/brokerhandler.go
src/k8splugin/api/brokerhandler_test.go

index 669b539..b377baf 100644 (file)
@@ -134,21 +134,21 @@ func (b brokerInstanceHandler) createHandler(w http.ResponseWriter, r *http.Requ
                return
        }
 
-       rbName := req.getAttributeValue(req.UserDirectives, "definition-name")
+       rbName := req.getAttributeValue(req.SDNCDirectives, "k8s-rb-definition-name")
        if rbName == "" {
-               http.Error(w, "definition-name is missing from user-directives", http.StatusBadRequest)
+               http.Error(w, "k8s-rb-definition-name is missing from sdnc-directives", http.StatusBadRequest)
                return
        }
 
-       rbVersion := req.getAttributeValue(req.UserDirectives, "definition-version")
+       rbVersion := req.getAttributeValue(req.SDNCDirectives, "k8s-rb-definition-version")
        if rbVersion == "" {
-               http.Error(w, "definition-version is missing from user-directives", http.StatusBadRequest)
+               http.Error(w, "k8s-rb-definition-version is missing from sdnc-directives", http.StatusBadRequest)
                return
        }
 
-       profileName := req.getAttributeValue(req.UserDirectives, "profile-name")
+       profileName := req.getAttributeValue(req.SDNCDirectives, "k8s-rb-profile-name")
        if profileName == "" {
-               http.Error(w, "profile-name is missing from user-directives", http.StatusBadRequest)
+               http.Error(w, "k8s-rb-profile-name is missing from sdnc-directives", http.StatusBadRequest)
                return
        }
 
index 8ef5e18..00ca3b7 100644 (file)
@@ -54,11 +54,11 @@ func TestBrokerCreateHandler(t *testing.T) {
                                "user_directives": {
                                        "attributes": [
                                                {
-                                                       "attribute_name": "definition-name",
+                                                       "attribute_name": "k8s-rb-definition-name",
                                                        "attribute_value": "test-rbdef"
                                                },
                                                {
-                                                       "attribute_name": "definition-version",
+                                                       "attribute_name": "k8s-rb-definition-version",
                                                        "attribute_value": "v1"
                                                }
                                        ]
@@ -67,9 +67,9 @@ func TestBrokerCreateHandler(t *testing.T) {
                        expectedCode: http.StatusBadRequest,
                },
                {
-                       label: "Succesfully create an Instance",
+                       label: "Deprecated parameters passed (user_directives)",
                        input: bytes.NewBuffer([]byte(`{
-                               "vf-module-model-customization-id": "84sdfkio938",
+                               "vf-module-model-customization-id": "97sdfkio168",
                                "sdnc_directives": {
                                        "attributes": [
                                                {
@@ -81,15 +81,42 @@ func TestBrokerCreateHandler(t *testing.T) {
                                "user_directives": {
                                        "attributes": [
                                                {
-                                                       "attribute_name": "definition-name",
+                                                       "attribute_name": "rb-definition-name",
+                                                       "attribute_value": "test-rbdef"
+                                               },
+                                               {
+                                                       "attribute_name": "rb-definition-version",
+                                                       "attribute_value": "v1"
+                                               },
+                                               {
+                                                       "attribute_name": "rb-profile-name",
+                                                       "attribute_value": "profile1"
+                                               }
+                                       ]
+                               }
+                       }`)),
+                       expectedCode: http.StatusBadRequest,
+               },
+               {
+                       label: "Succesfully create an Instance",
+                       input: bytes.NewBuffer([]byte(`{
+                               "vf-module-model-customization-id": "84sdfkio938",
+                               "sdnc_directives": {
+                                       "attributes": [
+                                               {
+                                                       "attribute_name": "vf_module_name",
+                                                       "attribute_value": "test-vf-module-name"
+                                               },
+                                               {
+                                                       "attribute_name": "k8s-rb-definition-name",
                                                        "attribute_value": "test-rbdef"
                                                },
                                                {
-                                                       "attribute_name": "definition-version",
+                                                       "attribute_name": "k8s-rb-definition-version",
                                                        "attribute_value": "v1"
                                                },
                                                {
-                                                       "attribute_name": "profile-name",
+                                                       "attribute_name": "k8s-rb-profile-name",
                                                        "attribute_value": "profile1"
                                                }
                                        ]