Skip to content

Commit

Permalink
Merge branch 'master' into andyzhangx/upgrade-sidecar-images2
Browse files Browse the repository at this point in the history
  • Loading branch information
andyzhangx authored Jun 26, 2024
2 parents 5ca279b + 410ad9f commit 0f8d045
Show file tree
Hide file tree
Showing 177 changed files with 45,408 additions and 1,458 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/check-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
if: success()
uses: actions/setup-go@v3
with:
go-version: 1.20.2
go-version: '1.22'
- name: Checkout code
uses: actions/checkout@v4
- name: Run unit tests
Expand Down Expand Up @@ -52,4 +52,4 @@ jobs:
uses: coverallsapp/github-action@v2
with:
parallel-finished: true
carryforward: "run-1"
carryforward: "run-1"
2 changes: 1 addition & 1 deletion .github/workflows/check-generated.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-go@v3
with:
go-version: '^1.16'
go-version: '1.22'
- run: |
make
git diff --exit-code
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/check-shell.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-go@v3
with:
go-version: '^1.16'
go-version: '1.22'
- run: |
make validate-shell
name: Verify Shell scripts
2 changes: 1 addition & 1 deletion .github/workflows/check-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-go@v3
with:
go-version: '^1.16'
go-version: '1.22'
- run: |
E2E_TEST=true
export E2E_TEST
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/golangci-lint-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,13 @@ jobs:
steps:
- uses: actions/setup-go@v3
with:
go-version: '1.19'
go-version: '1.22'
- uses: actions/checkout@v4
- name: golangci-lint
uses: golangci/golangci-lint-action@v4
with:
# Optional: version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version
version: v1.51.2
version: v1.59.1

# Optional: working directory, useful for monorepos
# working-directory: somedir
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/golangci-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,13 @@ jobs:
steps:
- uses: actions/setup-go@v3
with:
go-version: '1.19'
go-version: '1.22'
- uses: actions/checkout@v4
- name: golangci-lint
uses: golangci/golangci-lint-action@v4
with:
# Optional: version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version
version: v1.51.2
version: v1.59.1

# Optional: working directory, useful for monorepos
# working-directory: somedir
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/validate-components.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-go@v3
with:
go-version: '1.21.4'
go-version: '1.22'
- name: 'install cue'
run: |
go version
Expand Down
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,5 @@ hack/tools/bin

# to make sure we don't accidently add this back after the recent
# migration off of go-bindata
pkg/templates/templates_generated.go
pkg/templates/templates_generated.go
e2e/scenario-logs/
14 changes: 11 additions & 3 deletions .pipelines/e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ pr:
- pkg/agent/testdata/AKSWindows* # Windows test data

variables:
- group: "AKS Dev Assistant (KV)"
- group: ab-e2e
- name: timeout
value: 90
Expand All @@ -41,15 +40,24 @@ jobs:
- bash: |
export PATH="/usr/local/go/bin:$PATH"
go version
echo "VHD_BUILD_ID=$VHD_BUILD_ID"
export BUILD_ID=$(Build.BuildId)
if [ -n "${VHD_BUILD_ID}" ]; then
export SIG_VERSION_TAG_NAME=buildId
export SIG_VERSION_TAG_VALUE=$VHD_BUILD_ID
export IGNORE_SCENARIOS_WITH_MISSING_VHD=true
# You can add other commands here that you want to execute
else
echo "Build.BuildId is not specified. Running default e2e tests."
fi
cd e2e
go test -parallel ${{ variables.parallel}} -timeout ${{ variables.timeout }}m -v -run Test_All ./
displayName: Run AgentBaker E2E
env:
VHD_BUILD_ID: $(VHD_BUILD_ID)
ADO_PAT: $(PAT-aksdevassistant)
- publish: $(System.DefaultWorkingDirectory)/e2e/scenario-logs
artifact: scenario-logs
condition: always()
Expand Down
2 changes: 1 addition & 1 deletion .pipelines/scripts/setup_go.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

set -euo pipefail

GOLANG_VERSION="go1.19.7"
GOLANG_VERSION="go1.22.4"
echo "Downloading ${GOLANG_VERSION}"
curl -O "https://dl.google.com/go/${GOLANG_VERSION}.linux-amd64.tar.gz"

Expand Down
10 changes: 0 additions & 10 deletions e2e/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,19 +141,9 @@ specified VHD build, the selector will select the corresponding default catalog
VHD_BUILD_ID=123456789 SCENARIOS_TO_RUN=ubuntu2204,ubuntu2204-arm64,ubuntu2204-gpu-ncv3 ./e2e-local.sh
```

***NOTE: To utilize this feature, you'll also need to provide the suite with an ADO PAT (personal access token) with
which it can access the ADO resources to download the appropriate build artifacts.***

To specify your PAT, simply set the `ADO_PAT` environment variable accordingly:

```bash
ADO_PAT=<secret> VHD_BUILD_ID=123456789 SCENARIOS_TO_RUN=ubuntu2204,ubuntu2204-arm64,ubuntu2204-gpu-ncv3 ./e2e-local.sh
```

or:

```bash
export ADO_PAT=<secret>
VHD_BUILD_ID=123456789 SCENARIOS_TO_RUN=ubuntu2204,ubuntu2204-arm64,ubuntu2204-gpu-ncv3 ./e2e-local.sh
VHD_BUILD_ID=234567891 SCENARIOS_TO_RUN=ubuntu2204,ubuntu2204-arm64,ubuntu2204-gpu-ncv3 ./e2e-local.sh
...
Expand Down
8 changes: 0 additions & 8 deletions e2e/artifact/const.go

This file was deleted.

201 changes: 0 additions & 201 deletions e2e/artifact/download.go

This file was deleted.

30 changes: 0 additions & 30 deletions e2e/artifact/types.go

This file was deleted.

Loading

0 comments on commit 0f8d045

Please sign in to comment.