Skip to content

Commit

Permalink
Cleanup after manager kustomize modification
Browse files Browse the repository at this point in the history
  • Loading branch information
mszadkow committed Sep 11, 2024
1 parent 963e44a commit 8daa5f7
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 12 deletions.
2 changes: 2 additions & 0 deletions hack/multikueue-e2e-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,8 @@ function cleanup {
fi
#do the image restore here for the case when an error happened during deploy
restore_managers_image
# Remove the `newTag` for the `kubeflow/training-operator` to revert to the default version
$YQ eval 'del(.images[] | select(.name == "kubeflow/training-operator").newTag)' -i "$KUBEFLOW_MANIFEST_MANAGER/kustomization.yaml"
}


Expand Down
14 changes: 6 additions & 8 deletions test/e2e/config/multikueue/manager/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -1,14 +1,12 @@
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: kubeflow
resources:
- ../../../../../dep-crds/training-operator/manifests/base/crds
- namespace.yaml
- ../../../../../dep-crds/training-operator/manifests/base/crds
images:
- name: kubeflow/training-operator
- name: kubeflow/training-operator
secretGenerator:
- name: training-operator-webhook-cert
options:
disableNameSuffixHash: true
- name: training-operator-webhook-cert
options:
disableNameSuffixHash: true
patchesStrategicMerge:
- patch_crds.yaml
- patch_crds.yaml
4 changes: 0 additions & 4 deletions test/e2e/config/multikueue/manager/namespace.yaml

This file was deleted.

0 comments on commit 8daa5f7

Please sign in to comment.