Skip to content

Commit

Permalink
ci: remove test splitting for compatibility tests (#17166)
Browse files Browse the repository at this point in the history
* remove test splitting from compatibility-integration-tests

* enable on push

* remove ipv6 loopback fix

* re-add ipv6 loopback fix

* remove test splitting from upgrade-integration-tests

* remove test splitting from upgrade-integration-tests

* put test splitting back in for upgrade tests

* upgrade-integration tests-o
ne runner no retries
  • Loading branch information
jmurret committed May 8, 2023
1 parent d62d175 commit fb49a40
Showing 1 changed file with 5 additions and 55 deletions.
60 changes: 5 additions & 55 deletions .github/workflows/test-integrations.yml
Original file line number Diff line number Diff line change
Expand Up @@ -237,42 +237,11 @@ jobs:
name: ${{ env.TEST_RESULTS_ARTIFACT_NAME }}
path: ${{ env.TEST_RESULTS_DIR }}

# not using test cases on 1.13 branch because the directory
# structure is different and we can just test the whole directory.
# generate-compatibility-job-matrices:
# needs: [setup]
# runs-on: ${{ fromJSON(needs.setup.outputs.compute-small) }}
# name: Generate Compatibility Job Matrices
# outputs:
# compatibility-matrix: ${{ steps.set-matrix.outputs.compatibility-matrix }}
# steps:
# - uses: actions/checkout@24cb9080177205b6e8c946b17badbe402adc938f # v3.4.0
# - name: Generate Compatibility Job Matrix
# id: set-matrix
# env:
# TOTAL_RUNNERS: 1
# JQ_SLICER: '[ inputs ] | [_nwise(length / $runnercount | floor)]'
# run: |
# cd ./test/integration/consul-container
# {
# echo -n "compatibility-matrix="
# find . -maxdepth 2 -type d -print0 | xargs -0 -n 1 \
# | grep -v lib \
# | jq --raw-input --argjson runnercount "$TOTAL_RUNNERS" "$JQ_SLICER" \
# | jq --compact-output 'map(join(" "))'
# } >> "$GITHUB_OUTPUT"
# cat "$GITHUB_OUTPUT"

compatibility-integration-test:
runs-on: ${{ fromJSON(needs.setup.outputs.compute-xl) }}
needs:
- setup
- dev-build
# - generate-compatibility-job-matrices
# strategy:
# fail-fast: false
# matrix:
# test-cases: ${{ fromJSON(needs.generate-compatibility-job-matrices.outputs.compatibility-matrix) }}
steps:
- uses: actions/checkout@24cb9080177205b6e8c946b17badbe402adc938f # v3.4.0
- uses: actions/setup-go@6edd4406fa81c3da01a34fa6f6343087c207a568 # v3.5.0
Expand Down Expand Up @@ -302,36 +271,18 @@ jobs:
mkdir -p "/tmp/test-results"
cd ./test/integration/consul-container
docker run --rm ${{ env.CONSUL_LATEST_IMAGE_NAME }}:local consul version
# not using test cases on 1.13 branch because the directory
# structure is different and we can just test the whole directory.
# echo "Running $(sed 's,|, ,g' <<< "${{ matrix.test-cases }}" |wc -w) subtests"
# # shellcheck disable=SC2001
# sed 's,|,\n,g' <<< "${{ matrix.test-cases }}"
# go run gotest.tools/gotestsum@v${{env.GOTESTSUM_VERSION}} \
# --raw-command \
# --format=short-verbose \
# --debug \
# --rerun-fails=3 \
# -- \
# go test \
# -p=4 \
# -tags "${{ env.GOTAGS }}" \
# -timeout=30m \
# -json \
# "${{ matrix.test-cases }}" \
# --target-image ${{ env.CONSUL_LATEST_IMAGE_NAME }} \
# --target-version local \
# --latest-image ${{ env.CONSUL_LATEST_IMAGE_NAME }} \
# --latest-version latest
go run gotest.tools/gotestsum@v${{env.GOTESTSUM_VERSION}} \
--format=short-verbose \
--raw-command \
--format=standard-verbose \
--debug \
--rerun-fails=3 \
--packages="./..." \
-- \
--tags "${GOTAGS}" \
-timeout=30m \
./... \
-json \
`go list ./... | grep -v upgrade` \
--target-image ${{ env.CONSUL_LATEST_IMAGE_NAME }} \
--target-version local \
--latest-version latest
ls -lrt
Expand All @@ -356,7 +307,6 @@ jobs:
- vault-integration-test
- generate-envoy-job-matrices
- envoy-integration-test
# - generate-compatibility-job-matrices
- compatibility-integration-test
runs-on: ${{ fromJSON(needs.setup.outputs.compute-small) }}
if: ${{ always() }}
Expand Down

0 comments on commit fb49a40

Please sign in to comment.