3 files changed, 20 insertions, 15 deletions

diff --git a/src/k8splugin/plugins/generic/plugin.go b/src/k8splugin/plugins/generic/plugin.go
index b20cbe37..cc5fcb7a 100644
--- a/src/k8splugin/plugins/generic/plugin.go
+++ b/src/k8splugin/plugins/generic/plugin.go
@@ -14,8 +14,6 @@ limitations under the License.
 package main
 
 import (
-	"log"
-
 	pkgerrors "github.com/pkg/errors"
 	"k8s.io/apimachinery/pkg/api/meta"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -97,8 +95,6 @@ func (g genericPlugin) Get(resource helm.KubernetesResource,
 	}
 
 	gvr := mapping.Resource
-	log.Printf("Using gvr: %s, %s, %s", gvr.Group, gvr.Version, gvr.Resource)
-
 	opts := metav1.GetOptions{}
 	var unstruct *unstructured.Unstructured
 	switch mapping.Scope.Name() {
@@ -144,8 +140,6 @@ func (g genericPlugin) Delete(resource helm.KubernetesResource, namespace string
 	}
 
 	gvr := mapping.Resource
-	log.Printf("Using gvr: %s, %s, %s", gvr.Group, gvr.Version, gvr.Resource)
-
 	deletePolicy := metav1.DeletePropagationForeground
 	opts := &metav1.DeleteOptions{
 		PropagationPolicy: &deletePolicy,
diff --git a/src/k8splugin/plugins/network/plugin.go b/src/k8splugin/plugins/network/plugin.go
index 49aedc53..aa0d584b 100644
--- a/src/k8splugin/plugins/network/plugin.go
+++ b/src/k8splugin/plugins/network/plugin.go
@@ -35,14 +35,16 @@ var ExportedVariable networkPlugin
 type networkPlugin struct {
 }
 
-func extractData(data string) (cniType, networkName string) {
+func extractData(data string) (cniType, networkName string, err error) {
 	re := regexp.MustCompile("_")
 	split := re.Split(data, -1)
-	if len(split) != 3 {
+	if len(split) != 2 {
+		err = pkgerrors.New("Couldn't split resource '" + data +
+			"' into CNI type and Network name")
 		return
 	}
-	cniType = split[1]
-	networkName = split[2]
+	cniType = split[0]
+	networkName = split[1]
 	return
 }
 
@@ -86,7 +88,11 @@ func (p networkPlugin) List(gvk schema.GroupVersionKind, namespace string,
 
 // Delete an existing Network
 func (p networkPlugin) Delete(resource helm.KubernetesResource, namespace string, client plugin.KubernetesConnector) error {
-	cniType, networkName := extractData(resource.Name)
+	cniType, networkName, err := extractData(resource.Name)
+	if err != nil {
+		return pkgerrors.Wrap(err, "Error extracting CNI type from resource")
+	}
+
 	typePlugin, ok := utils.LoadedPlugins[cniType+"-network"]
 	if !ok {
 		return pkgerrors.New("No plugin for resource " + cniType + " found")
diff --git a/src/k8splugin/plugins/network/plugin_test.go b/src/k8splugin/plugins/network/plugin_test.go
index 586bccb8..33cae1c7 100644
--- a/src/k8splugin/plugins/network/plugin_test.go
+++ b/src/k8splugin/plugins/network/plugin_test.go
@@ -130,18 +130,23 @@ func TestDeleteNetwork(t *testing.T) {
 	}{
 		{
 			label:         "Fail to load non-existing plugin",
-			input:         "test",
-			expectedError: "No plugin for resource",
+			input:         "non-existing-cni_test",
+			expectedError: "No plugin for resource non-existing-cni",
 		},
 		{
-			label:         "Fail to delete a network",
+			label:         "Fail to extract cni from network name",
 			input:         "1_ovn4nfvk8s_test",
+			expectedError: "Error extracting CNI type from resource: Couldn't split resource '1_ovn4nfvk8s_test' into CNI type and Network name",
+		},
+		{
+			label:         "Fail to delete a network",
+			input:         "ovn4nfvk8s_test",
 			mockError:     "Internal error",
 			expectedError: "Error during the deletion for ovn4nfvk8s plugin: Internal error",
 		},
 		{
 			label: "Successfully delete a ovn4nfv network",
-			input: "1_ovn4nfvk8s_test",
+			input: "ovn4nfvk8s_test",
 		},
 	}