e2e: remove make install-cert-manager for verify_deployment (#102)

- Removes `make install-cert-manager` from verify_deployment. - Removes the windows node taint that was required when using cert-manager Signed-off-by: Anish Ramasekar <[email protected]>
Azure · Jul 20, 2021 · 9d15d92 · 9d15d92
1 parent 7845d70
commit 9d15d92
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 7 deletions.
diff --git a/.pipelines/nightly.yaml b/.pipelines/nightly.yaml
@@ -39,8 +39,6 @@ jobs:
         env:
           SKIP_PREFLIGHT: "true"
           SERVICE_ACCOUNT_ISSUER: $(SERVICE_ACCOUNT_ISSUER)
-      - script: make install-cert-manager
-        displayName: Installing cert-manager
       - script: |
           sed -i "s/AZURE_TENANT_ID: .*/AZURE_TENANT_ID: ${AZURE_TENANT_ID}/" manifest_staging/deploy/aad-pi-webhook.yaml
           sed -i "s/AZURE_ENVIRONMENT: .*/AZURE_ENVIRONMENT: AzurePublicCloud/" manifest_staging/deploy/aad-pi-webhook.yaml

diff --git a/scripts/ci-e2e.sh b/scripts/ci-e2e.sh
@@ -36,9 +36,6 @@ create_cluster() {
     WINDOWS_NODE_NAME="$(${KUBECTL} get node --selector=kubernetes.io/os=windows -ojson | jq -r '.items[0].metadata.name')"
     if [[ "${WINDOWS_NODE_NAME}" == "null" ]]; then
       unset WINDOWS_NODE_NAME
-    else
-      # taint the windows node to prevent cert-manager pods from scheduling to it
-      ${KUBECTL} taint nodes "${WINDOWS_NODE_NAME}" kubernetes.io/os=windows:NoSchedule --overwrite
     fi
 
     if [[ "${REGISTRY}" =~ \.azurecr\.io ]]; then
@@ -88,8 +85,6 @@ main() {
   sleep 120
 
   if [[ -n "${WINDOWS_NODE_NAME:-}" ]]; then
-    # remove the taint from the windows node we introduced above
-    ${KUBECTL} taint nodes "${WINDOWS_NODE_NAME}" kubernetes.io/os=windows:NoSchedule-
     E2E_ARGS="--node-os-distro=windows ${E2E_ARGS:-}"
     export E2E_ARGS
   fi