Skip to content

Commit

Permalink
test: deduplicate test manifests and fix "stress" profile (#14001)
Browse files Browse the repository at this point in the history
Signed-off-by: Mason Malone <[email protected]>
  • Loading branch information
MasonM authored Dec 31, 2024
1 parent 2e0f2f7 commit 81993b5
Show file tree
Hide file tree
Showing 21 changed files with 50 additions and 106 deletions.
4 changes: 0 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -484,10 +484,6 @@ install: githooks
ifeq ($(PROFILE),stress)
kubectl -n $(KUBE_NAMESPACE) apply -f test/stress/massive-workflow.yaml
endif
ifeq ($(RUN_MODE),kubernetes)
kubectl -n $(KUBE_NAMESPACE) scale deploy/workflow-controller --replicas 1
kubectl -n $(KUBE_NAMESPACE) scale deploy/argo-server --replicas 1
endif
ifeq ($(UI_SECURE)$(PROFILE),truesso)
KUBE_NAMESPACE=$(KUBE_NAMESPACE) ./hack/update-sso-redirect-url.sh
endif
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ kind: Deployment
metadata:
name: argo-server
spec:
replicas: 0
template:
spec:
containers:
Expand Down
17 changes: 17 additions & 0 deletions test/e2e/manifests/components/base/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
apiVersion: kustomize.config.k8s.io/v1alpha1
kind: Component

resources:
- argo-server.service-account-token-secret.yaml

patches:
- path: argo-server-deployment.yaml
- path: workflow-controller-configmap.yaml
- path: workflow-controller-deployment.yaml
- path: workflow-controller-cluster-workflow-template-rbac.yaml
- path: minio-deployment.yaml

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ kind: Deployment
metadata:
name: workflow-controller
spec:
replicas: 0
template:
spec:
containers:
Expand Down
9 changes: 9 additions & 0 deletions test/e2e/manifests/components/local-argo/kustomization.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# Component to scale down the deployments for the workflow controller and argo server,
# for use with profiles where argo is run locally to avoid conflicts.
apiVersion: kustomize.config.k8s.io/v1alpha1
kind: Component
replicas:
- name: argo-server
count: 0
- name: workflow-controller
count: 0
16 changes: 4 additions & 12 deletions test/e2e/manifests/events/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,16 +6,8 @@ resources:
- https://raw.githubusercontent.com/argoproj/argo-events/v1.2.0/manifests/install.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/v1.2.0/examples/eventbus/native.yaml

patches:
- path: ../mixins/argo-server-deployment.yaml
- path: ../mixins/argo-server-cluster-workflow-template-rbac.yaml
- path: ../mixins/workflow-controller-configmap.yaml
- path: ../mixins/workflow-controller-deployment.yaml
- path: ../mixins/workflow-controller-cluster-workflow-template-rbac.yaml
- path: ../mixins/minio-deployment.yaml
components:
- ../components/base
- ../components/local-argo

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
namespace: argo
namespace: argo
File renamed without changes.
23 changes: 7 additions & 16 deletions test/e2e/manifests/minimal/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,21 +6,12 @@ resources:
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_eventbus.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_eventsources.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_sensors.yaml
- ../mixins/argo-workflows-agent-ca-certificates.yaml
- ../mixins/argo-server.service-account-token-secret.yaml
- ../mixins/argo.service-account-token-secret.yaml
- ../mixins/get-cm-rbac.yaml
- argo-workflows-agent-ca-certificates.yaml
- argo.service-account-token-secret.yaml
- get-cm-rbac.yaml

patches:
- path: ../mixins/argo-server-deployment.yaml
- path: ../mixins/argo-server-cluster-workflow-template-rbac.yaml
- path: ../mixins/workflow-controller-configmap.yaml
- path: ../mixins/workflow-controller-deployment.yaml
- path: ../mixins/workflow-controller-cluster-workflow-template-rbac.yaml
- path: ../mixins/minio-deployment.yaml
components:
- ../components/base
- ../components/local-argo

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
namespace: argo
namespace: argo

This file was deleted.

15 changes: 3 additions & 12 deletions test/e2e/manifests/mysql/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,9 @@ resources:
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_eventbus.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_eventsources.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_sensors.yaml
- ../mixins/argo-server.service-account-token-secret.yaml

patches:
- path: ../mixins/argo-server-deployment.yaml
- path: ../mixins/argo-server-cluster-workflow-template-rbac.yaml
- path: ../mixins/workflow-controller-configmap.yaml
- path: ../mixins/workflow-controller-deployment.yaml
- path: ../mixins/workflow-controller-cluster-workflow-template-rbac.yaml
- path: ../mixins/minio-deployment.yaml
components:
- ../components/base
- ../components/local-argo

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
namespace: argo
17 changes: 4 additions & 13 deletions test/e2e/manifests/postgres/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,9 @@ resources:
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_eventbus.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_eventsources.yaml
- https://raw.githubusercontent.com/argoproj/argo-events/stable/manifests/base/crds/argoproj.io_sensors.yaml
- ../mixins/argo-server.service-account-token-secret.yaml

patches:
- path: ../mixins/argo-server-deployment.yaml
- path: ../mixins/argo-server-cluster-workflow-template-rbac.yaml
- path: ../mixins/workflow-controller-configmap.yaml
- path: ../mixins/workflow-controller-deployment.yaml
- path: ../mixins/workflow-controller-cluster-workflow-template-rbac.yaml
- path: ../mixins/minio-deployment.yaml
components:
- ../components/base
- ../components/local-argo

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
namespace: argo
namespace: argo
14 changes: 2 additions & 12 deletions test/e2e/manifests/prometheus/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,5 @@ resources:
- ../../../../manifests/quick-start/minimal
- ../../../../manifests/quick-start/base/prometheus

patches:
- path: ../mixins/argo-server-deployment.yaml
- path: ../mixins/argo-server-cluster-workflow-template-rbac.yaml
- path: ../mixins/workflow-controller-configmap.yaml
- path: ../mixins/workflow-controller-deployment.yaml
- path: ../mixins/workflow-controller-cluster-workflow-template-rbac.yaml
- path: ../mixins/minio-deployment.yaml

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
components:
- ../components/base
17 changes: 4 additions & 13 deletions test/e2e/manifests/sso/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,18 +3,9 @@ kind: Kustomization

resources:
- ../../../../manifests/quick-start/sso
- ../mixins/argo-server.service-account-token-secret.yaml

patches:
- path: ../mixins/argo-server-deployment.yaml
- path: ../mixins/argo-server-cluster-workflow-template-rbac.yaml
- path: ../mixins/workflow-controller-configmap.yaml
- path: ../mixins/workflow-controller-deployment.yaml
- path: ../mixins/workflow-controller-cluster-workflow-template-rbac.yaml
- path: ../mixins/minio-deployment.yaml
components:
- ../components/base
- ../components/local-argo

labels:
- includeSelectors: true
pairs:
app.kubernetes.io/part-of: argo
namespace: argo
namespace: argo
1 change: 0 additions & 1 deletion test/e2e/manifests/stress/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ resources:
- ../../../../manifests/quick-start/minimal
- ../../../../manifests/quick-start/base/prometheus
- workflow-controller-pprof-service.yaml
- workflow-controller-podpriorityclass.yaml
- argo-server-podpriorityclass.yaml
- minio-podpriorityclass.yaml

Expand Down

This file was deleted.

0 comments on commit 81993b5

Please sign in to comment.