Removing helm/charts from test-infra

The charts repo is no longer supported. It no longer gets new contributions. So, this change is to remove test-infra config that has been used to support the charts repo. See https://helm.sh/blog/helm-2-becomes-unsupported/ Signed-off-by: Matt Farina <[email protected]>
kubernetes · Nov 13, 2020 · 9c2895d · 9c2895d
1 parent 7886a00
commit 9c2895d
Show file tree

Hide file tree

Showing 6 changed files with 0 additions and 116 deletions.
diff --git a/config/jobs/helm/charts/OWNERS b/config/jobs/helm/charts/OWNERS
diff --git a/config/jobs/helm/charts/charts.yaml b/config/jobs/helm/charts/charts.yaml
diff --git a/config/prow/config.yaml b/config/prow/config.yaml
@@ -440,19 +440,6 @@ tide:
     - do-not-merge/invalid-owners-file
     - do-not-merge/work-in-progress
     - needs-rebase
-  - repos:
-    - helm/charts
-    labels:
-    - lgtm
-    - approved
-    - "Contribution Allowed"
-    missingLabels:
-    - do-not-merge
-    - do-not-merge/blocked-paths
-    - do-not-merge/hold
-    - do-not-merge/invalid-owners-file
-    - do-not-merge/work-in-progress
-    - needs-rebase
   - orgs:
     - kubernetes
     - kubernetes-client
@@ -561,7 +548,6 @@ tide:
     - do-not-merge/work-in-progress
 
   merge_method:
-    helm/charts: squash
     kubeflow: squash
     kubernetes-client/csharp: squash
     kubernetes-client/gen: squash

diff --git a/config/prow/plugins.yaml b/config/prow/plugins.yaml
@@ -32,7 +32,6 @@ owners:
   - kubernetes/website
   skip_collaborators:
   - kubernetes-sigs/contributor-playground
-  - helm/charts
 
 approve:
 - repos:
@@ -96,10 +95,6 @@ approve:
   - kubernetes/cloud-provider-gcp
   require_self_approval: false
   ignore_review_state: false
-- repos:
-  - helm/charts
-  require_self_approval: false
-  lgtm_acts_as_approve: true
 
 # Lower bounds in number of lines changed; XS is assumed to be zero.
 size:
@@ -672,17 +667,6 @@ plugins:
   - wip   # Applies a label to PRs with wip in the title to block merge
   - trigger
 
-  helm/charts:
-  - approve
-  - assign
-  - blunderbuss
-  - cla
-  - hold
-  - lgtm
-  - trigger
-  - verify-owners
-  - wip
-
   kubernetes:
   - approve
   - assign

diff --git a/prow/README.md b/prow/README.md
@@ -91,7 +91,6 @@ Prow is used by the following organizations and projects:
 - [Kubeflow](https://github.com/kubeflow)
 - [Azure AKS Engine](https://github.com/Azure/aks-engine/tree/master/.prowci)
 - [tensorflow/minigo](https://github.com/tensorflow/minigo#automated-tests)
-- [helm/charts](https://github.com/helm/charts)
 - [Daisy(google compute image tools)](https://github.com/GoogleCloudPlatform/compute-image-tools/tree/master/test-infra#prow-and-gubenator)
 - [KubeEdge (Kubernetes Native Edge Computing Framework)](https://github.com/kubeedge/kubeedge)
 - [Volcano (Kubernetes Native Batch System)](https://github.com/volcano-sh/volcano)

diff --git a/prow/config/tide_test.go b/prow/config/tide_test.go
@@ -109,8 +109,6 @@ func TestMergeMethod(t *testing.T) {
 	ti := &Tide{
 		MergeType: map[string]github.PullRequestMergeType{
 			"kubernetes/kops":             github.MergeRebase,
-			"kubernetes/charts":           github.MergeSquash,
-			"helm/charts":                 github.MergeSquash,
 			"kubernetes-helm":             github.MergeSquash,
 			"kubernetes-helm/chartmuseum": github.MergeMerge,
 		},
@@ -162,14 +160,6 @@ func TestMergeTemplate(t *testing.T) {
 				TitleTemplate: "",
 				BodyTemplate:  "",
 			},
-			"kubernetes/charts": {
-				TitleTemplate: "{{ .Number }}",
-				BodyTemplate:  "",
-			},
-			"helm/charts": {
-				TitleTemplate: "",
-				BodyTemplate:  "{{ .Body }}",
-			},
 			"kubernetes-helm": {
 				TitleTemplate: "{{ .Title }}",
 				BodyTemplate:  "{{ .Body }}",
@@ -203,14 +193,6 @@ func TestMergeTemplate(t *testing.T) {
 				BodyTemplate:  "",
 			},
 		},
-		{
-			org:  "helm",
-			repo: "charts",
-			expected: TideMergeCommitTemplate{
-				TitleTemplate: "",
-				BodyTemplate:  "{{ .Body }}",
-			},
-		},
 		{
 			org:  "kubernetes-helm",
 			repo: "monocular",