addressed comments

Signed-off-by: Ran Wurmbrand <rwurmbra@redhat.com>

Author: RanWurmbrand

e2e-tests/tests/tier0/mta_852_minimal_rbac_test.go

@@ -12,7 +12,7 @@ import (
 )
 
 var _ = Describe("Cluster-level RBAC export", func() {
-	It("[MTA-852] Should export ClusterRole and ClusterRoleBinding for linked ServiceAccount", Label("cluster-admin"), func() {
+	It("[MTA-852] Should export ClusterRole and ClusterRoleBinding for linked ServiceAccount", Label("tier0"), func() {
 		appName := "simple-nginx-nopv"
 		namespace := "simple-nginx-nopv"
 		serviceName := "my-" + appName

e2e-tests/tests/tier0/mta_853_split_apply_test.go

@@ -12,7 +12,7 @@ import (
 )
 
 var _ = Describe("Namespace-admin cluster-level migration", func() {
-	It("[MTA-853] Should migrate workload with split apply: namespace-admin + cluster-admin", Label("namespace-admin"), func() {
+	It("[MTA-853] Should migrate workload with split apply: namespace-admin + cluster-admin", Label("tier0"), func() {
 		appName := "simple-nginx-nopv"
 		namespace := "simple-nginx-nopv"
 		serviceName := "my-" + appName

e2e-tests/tests/tier1/mta_854_two_clusterroles_test.go

@@ -10,8 +10,8 @@ import (
 )
 
 var _ = Describe("Cluster-level RBAC export", func() {
-	It("[MTA-854] Should export two ClusterRoles and two ClusterRoleBindings for one Deployment", Label("cluster-admin"), func() {
-		appName := "nginx-with-serviceaccount"
+	It("[MTA-854] Should export two ClusterRoles and two ClusterRoleBindings for one Deployment", Label("tier1"), func() {
+		appName := "simple-nginx-nopv"
 		namespace := "simple-nginx-nopv"
 		serviceName := "my-" + appName
 		readClusterRole := "crane-e2e-pod-reader"
@@ -40,10 +40,12 @@ var _ = Describe("Cluster-level RBAC export", func() {
 		readCRB := ClusterRoleBinding{Name: readClusterRoleBindingName, ClusterRoleName: readClusterRole}
 		writeCRB := ClusterRoleBinding{Name: writeClusterRoleBindingName, ClusterRoleName: writeClusterRole}
 		sa := ServiceAccount{Name: "nginx-sa", Namespace: namespace}
+
 		exportOpts := ExportOptions{Namespace: srcApp.Namespace, ExportDir: paths.ExportDir}
 		transformOpts := TransformOptions{ExportDir: paths.ExportDir, TransformDir: paths.TransformDir}
 		applyOpts := ApplyOptions{ExportDir: paths.ExportDir, TransformDir: paths.TransformDir,
 			OutputDir: paths.OutputDir}
+
 		DeferCleanup(func() {
 			if err := ResourceCleanup([]KubectlRunner{kubectlSrc, kubectlTgt}, []Resource{readCR, writeCR, readCRB, writeCRB, sa}); err != nil {
 				log.Printf("Resources cleanup: %v", err)
@@ -54,7 +56,7 @@ var _ = Describe("Cluster-level RBAC export", func() {
 			}
 		})
 
-		By("Deploying app with ServiceAccount on source cluster")
+		By("Deploying app with on source cluster")
 		Expect(PrepareSourceApp(srcApp, kubectlSrc)).NotTo(HaveOccurred())
 
 		By("Creating ClusterRole with pod read permissions")
@@ -69,6 +71,9 @@ var _ = Describe("Cluster-level RBAC export", func() {
 		By("Creating ClusterRoleBinding for write ClusterRole")
 		Expect(writeCRB.Create(kubectlSrc)).NotTo(HaveOccurred())
 
+		By("Creating Service-Account on Namespace")
+		Expect(sa.Create(kubectlSrc)).NotTo(HaveOccurred())
+
 		By("Bind Relevent Service-Account to Read cluster role")
 		Expect(readCRB.AddSubject(kubectlSrc, sa)).NotTo(HaveOccurred())
 

e2e-tests/tests/tier1/mta_855_crd_custom_resource_test.go

@@ -12,7 +12,7 @@ import (
 )
 
 var _ = Describe("Cluster-level RBAC export", func() {
-	It("[MTA-855] Should export CRD to _cluster and custom resource to namespace directory", Label("cluster-admin"), func() {
+	It("[MTA-855] Should export CRD to _cluster and custom resource to namespace directory", Label("tier1"), func() {
 		appName := "simple-nginx-nopv"
 		namespace := "simple-nginx-nopv"
 		serviceName := "my-" + appName