Helmv3 migration of the chart

charts/kubefed/.gitignore

@@ -0,0 +1 @@
+requirements.lock

charts/kubefed/README.md

@@ -25,38 +25,8 @@ group](https://groups.google.com/forum/#!forum/kubernetes-sig-multicluster).
 ## Prerequisites
 
 - Kubernetes 1.13+
-- Helm 2.10+
+- Helm 3.1.2+
 
-## Configuring RBAC for Helm (Optional)
-
-If your Kubernetes cluster has RBAC enabled, it will be necessary to
-ensure that helm is deployed with a service account with the
-permissions necessary to deploy KubeFed:
-
-```bash
-$ cat << EOF | kubectl apply -f -
-apiVersion: v1
-kind: ServiceAccount
-metadata:
-  name: tiller
-  namespace: kube-system
----
-apiVersion: rbac.authorization.k8s.io/v1
-kind: ClusterRoleBinding
-metadata:
-  name: tiller
-roleRef:
-  apiGroup: rbac.authorization.k8s.io
-  kind: ClusterRole
-  name: cluster-admin
-subjects:
-  - kind: ServiceAccount
-    name: tiller
-    namespace: kube-system
-EOF
-
-$ helm init --service-account tiller
-```
 
 ## Installing the Chart
 
@@ -71,13 +41,14 @@ kubefed-charts   https://raw.githubusercontent.com/kubernetes-sigs/kubefed/maste
 
 With the repo added, available charts and versions can be viewed.
 ```bash
-$ helm search kubefed --devel
+$ helm search repo kubefed
 ```
 
 Install the chart and specify the version to install with the
 `--version` argument. Replace `<x.x.x>` with your desired version.
 ```bash
-$ helm install kubefed-charts/kubefed --name kubefed --version=<x.x.x> --namespace kube-federation-system --devel
+$ kubectl create namespace kube-federation-system
+$ helm --namespace kube-federation-system install kubefed kubefed-charts/kubefed --version=<x.x.x>
 ```
 
 **NOTE:** For **namespace-scoped deployments** (configured with the `--set
@@ -95,26 +66,16 @@ namespace with this label by default.
 
 ## Uninstalling the Chart
 
-Due to this helm [issue](https://github.com/helm/helm/issues/4440), the CRDs cannot be deleted
-when delete helm release, so before delete the helm release, we need first delete all
-of the CR and CRDs for KubeFed release.
-
 Delete all KubeFed `FederatedTypeConfig`:
 
 ```bash
 $ kubectl -n kube-federation-system delete FederatedTypeConfig --all
 ```
 
-Delete all KubeFed CRDs:
-
-```bash
-$ kubectl delete crd $(kubectl get crd | grep -E 'kubefed.io' | awk '{print $1}')
-```
-
 Then you can uninstall/delete the `kubefed` release:
 
 ```bash
-$ helm delete --purge kubefed
+$ helm --namespace kube-federation-system uninstall kubefed
 ```
 
 The command above removes all the Kubernetes components associated with the chart

charts/kubefed/charts/controllermanager/templates/crds.yaml → charts/kubefed/charts/controllermanager/crds/crds.yaml

@@ -1,12 +1,7 @@
-{{ if (or (or (not .Values.global.scope) (eq .Values.global.scope "Cluster")) (not (.Capabilities.APIVersions.Has "core.kubefed.io/v1beta1"))) }}
----
-
 ---
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: clusterpropagatedversions.core.kubefed.io
 spec:
@@ -89,8 +84,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: federatedservicestatuses.core.kubefed.io
 spec:
@@ -174,8 +167,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: federatedtypeconfigs.core.kubefed.io
 spec:
@@ -350,8 +341,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: kubefedclusters.core.kubefed.io
 spec:
@@ -492,8 +481,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: kubefedconfigs.core.kubefed.io
 spec:
@@ -625,8 +612,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: propagatedversions.core.kubefed.io
 spec:
@@ -709,8 +694,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: dnsendpoints.multiclusterdns.kubefed.io
 spec:
@@ -799,8 +782,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: domains.multiclusterdns.kubefed.io
 spec:
@@ -854,8 +835,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: ingressdnsrecords.multiclusterdns.kubefed.io
 spec:
@@ -953,8 +932,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: servicednsrecords.multiclusterdns.kubefed.io
 spec:
@@ -1090,8 +1067,6 @@ status:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   creationTimestamp: null
   name: replicaschedulingpreferences.scheduling.kubefed.io
 spec:
@@ -1195,4 +1170,3 @@ status:
   conditions: []
   storedVersions: []
 ---
-{{ end }}

charts/kubefed/templates/crds.yaml → charts/kubefed/crds/crds.yaml

@@ -1,10 +1,7 @@
-{{ if (or (or (not .Values.global.scope) (eq .Values.global.scope "Cluster")) (not (.Capabilities.APIVersions.Has "types.kubefed.io/v1beta1"))) }}
 ---
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedclusterroles.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -133,8 +130,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedconfigmaps.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -265,8 +260,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federateddeployments.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -399,8 +392,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedingresses.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -531,8 +522,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedjobs.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -661,8 +650,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatednamespaces.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -793,8 +780,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedreplicasets.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -927,8 +912,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedsecrets.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -1057,8 +1040,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedserviceaccounts.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -1189,8 +1170,6 @@ spec:
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  annotations:
-    "helm.sh/hook": crd-install
   name: federatedservices.types.kubefed.io
 spec:
   group: types.kubefed.io
@@ -1317,4 +1296,3 @@ spec:
       required:
       - spec
   version: v1beta1
-{{ end }}

scripts/delete-kubefed.sh

@@ -25,15 +25,15 @@ source "$(dirname "${BASH_SOURCE}")/util.sh"
 
 function delete-helm-deployment() {
   # Clean kubefed resources
-  ${KCD} -n "${NS}" FederatedTypeConfig --all
-  if [[ ! "${NAMESPACED}" || "${DELETE_CLUSTER_RESOURCE}" ]]; then
-    ${KCD} crd $(kubectl get crd | grep -E 'kubefed.io' | awk '{print $1}')
-  fi
+  # ${KCD} -n "${NS}" FederatedTypeConfig --all
+  # if [[ ! "${NAMESPACED}" || "${DELETE_CLUSTER_RESOURCE}" ]]; then
+  #   ${KCD} crd $(kubectl get crd | grep -E 'kubefed.io' | awk '{print $1}')
+  # fi
 
   if [[ "${NAMESPACED}" ]]; then
-    helm delete --purge kubefed-${NS}
+    helm -n ${NS} uninstall kubefed-${NS}
   else
-    helm delete --purge kubefed
+    helm -n ${NS} uninstall kubefed
   fi
 }
 
@@ -62,7 +62,10 @@ done
 # Deploy kubefed resources
 delete-helm-deployment
 
-${KCD} ns "${NS}"
+${KCD} ns "${NS}" -v8 
+
+echo "Helm uninstall finished"
+exit 0
 
 # Wait for the namespaces to be removed
 function ns-deleted() {

scripts/deploy-kubefed.sh

@@ -38,34 +38,6 @@ set -o pipefail
 source "$(dirname "${BASH_SOURCE}")/util.sh"
 
 function deploy-with-helm() {
-  # Don't install tiller if we already have a working install.
-  if ! helm version --server 2>/dev/null; then
-    # RBAC should be enabled to avoid CI fail because CI K8s uses RBAC for Tiller
-    cat <<EOF | kubectl apply -f -
-apiVersion: v1
-kind: ServiceAccount
-metadata:
-  name: tiller
-  namespace: kube-system
----
-apiVersion: rbac.authorization.k8s.io/v1
-kind: ClusterRoleBinding
-metadata:
-  name: tiller
-roleRef:
-  apiGroup: rbac.authorization.k8s.io
-  kind: ClusterRole
-  name: cluster-admin
-subjects:
-  - kind: ServiceAccount
-    name: tiller
-    namespace: kube-system
-EOF
-
-    helm init --service-account tiller
-    util::wait-for-condition "Tiller to become ready" "helm version --server &> /dev/null" 120
-  fi
-
   local repository=${IMAGE_NAME%/*}
   local image_tag=${IMAGE_NAME##*/}
   local image=${image_tag%:*}
@@ -86,6 +58,13 @@ EOF
   ${cmd}
 }
 
+function create-namespace {
+  # Required arguments
+  local ns="${1}"
+  echo "kubectl create namespace ${ns}" 
+  kubectl create namespace ${ns}
+}
+
 function helm-deploy-cmd {
   # Required arguments
   local name="${1}"
@@ -94,7 +73,7 @@ function helm-deploy-cmd {
   local image="${4}"
   local tag="${5}"
 
-  echo "helm install charts/kubefed --name ${name} --namespace ${ns} \
+  echo "helm --namespace ${ns} install ${name} charts/kubefed \
       --set controllermanager.repository=${repo} --set controllermanager.image=${image} \
       --set controllermanager.tag=${tag}"
 }
@@ -173,6 +152,9 @@ cd "$(dirname "$0")/.."
 make kubefedctl
 cd -
 
+# Create namespace first before deploy with helm
+create-namespace ${NS}
+
 # Deploy KubeFed resources
 deploy-with-helm