Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Commit

Permalink
ci: remove aks-engine soak clusters from pr and nightly (#1346)
Browse files Browse the repository at this point in the history
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
  • Loading branch information
aramase authored Nov 29, 2022
1 parent b32183e commit 6cf6c88
Show file tree
Hide file tree
Showing 10 changed files with 22 additions and 145 deletions.
5 changes: 0 additions & 5 deletions .pipelines/nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,9 @@ jobs:
parameters:
clusterConfigs:
- "aks"
# File names in test/e2e/cluster_configs without file extension
- "vmss"
- "vmas"
- template: templates/soak-test.yml
parameters:
clusterConfigs:
- "pi-vmas-e2e-daily"
- "pi-vmss-e2e-daily"
- "pi-aks-e2e-daily"
- template: templates/load-test.yml
parameters:
Expand Down
3 changes: 0 additions & 3 deletions .pipelines/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,3 @@ jobs:
parameters:
clusterConfigs:
- "aks"
# File names in test/e2e/cluster_configs without file extension
- "vmss"
- "vmas"
31 changes: 0 additions & 31 deletions .pipelines/templates/deploy-aks-engine-cluster.yml

This file was deleted.

8 changes: 4 additions & 4 deletions .pipelines/templates/e2e-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,10 @@ jobs:
- ${{ each clusterConfig in parameters.clusterConfigs }}:
- job:
displayName: ${{ format('{0}', clusterConfig) }}
dependsOn: unit_test
dependsOn:
- scan_images
- lint
- unit_test
timeoutInMinutes: 120
cancelTimeoutInMinutes: 5
workspace:
Expand All @@ -35,9 +38,6 @@ jobs:
- ${{ if eq(clusterConfig, 'aks') }}:
- template: deploy-aks-cluster.yml

- ${{ if not(eq(clusterConfig, 'aks')) }}:
- template: deploy-aks-engine-cluster.yml

- template: role-assignment.yml
parameters:
identity_resource_group: $(IDENTITY_RESOURCE_GROUP)
Expand Down
20 changes: 15 additions & 5 deletions .pipelines/templates/unit-test.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,19 @@
jobs:
- job: scan_images
timeoutInMinutes: 15
workspace:
clean: all
steps:
- template: scan-images.yml
- job: lint
timeoutInMinutes: 15
workspace:
clean: all
steps:
- script: make lint
displayName: golangci-lint
- script: make helm-lint
displayName: helm lint
- job: unit_test
timeoutInMinutes: 20
cancelTimeoutInMinutes: 5
Expand All @@ -7,8 +22,3 @@ jobs:
steps:
- script: make unit-test
displayName: Run unit tests
- script: make lint
displayName: Run lint
- script: make helm-lint
displayName: Run helm lint
- template: scan-images.yml
5 changes: 1 addition & 4 deletions codecov.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,7 @@ parsers:
method: no
macro: no

comment:
layout: "header, diff"
behavior: default
require_changes: no
comment: false

ignore:
- "test/**/*.go"
2 changes: 1 addition & 1 deletion manifest_staging/charts/aad-pod-identity/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ The following steps will help you create a new Azure identity ([Managed Service

### Prerequisites
* [Azure Subscription](https://azure.microsoft.com/)
* [Azure Kubernetes Service (AKS)](https://azure.microsoft.com/services/kubernetes-service/) or [AKS Engine](https://github.com/Azure/aks-engine) deployment
* [Azure Kubernetes Service (AKS)](https://azure.microsoft.com/services/kubernetes-service/) deployment
* [kubectl](https://kubernetes.io/docs/tasks/tools/install-kubectl/) (authenticated to your Kubernetes cluster)
* [Helm 3](https://v3.helm.sh/)
* [Azure CLI 2.0](https://docs.microsoft.com/en-us/cli/azure/install-azure-cli?view=azure-cli-latest)
Expand Down
46 changes: 0 additions & 46 deletions test/e2e/cluster_configs/vmas.json

This file was deleted.

45 changes: 0 additions & 45 deletions test/e2e/cluster_configs/vmss.json

This file was deleted.

2 changes: 1 addition & 1 deletion website/content/en/docs/Contributing/_index.md
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ When you submit a pull request, a CLA-bot will automatically determine whether y

- Go 1.18
- Docker
- an AKS cluster / AKS-Engine cluster
- an AKS cluster
- Helm 3
- A container registry. One of the following would work:
- Dockerhub
Expand Down

0 comments on commit 6cf6c88

Please sign in to comment.