Commit 139dabe5 authored by Matthew Slipper's avatar Matthew Slipper Committed by GitHub

ci: Remove Kurtosis-specific go job (#13249)

* ci: Remove Kurtosis-specific go job

Kurtosis landed [my fix](https://github.com/kurtosis-tech/kurtosis/pull/2567) for the race condition that prevented us from running a shared Kurtosis instance between jobs. This PR brings that back.

* remove dep
parent d3fbc576
......@@ -807,63 +807,6 @@ jobs:
make lint-go
working_directory: .
go-test-kurtosis:
parameters:
module:
description: Go Module Name
type: string
uses_artifacts:
description: Uses contract artifacts
type: boolean
default: false
test_directory:
description: Test directory
type: string
default: "./..."
machine:
image: <<pipeline.parameters.base_image>>
resource_class: xlarge
steps:
- run:
name: Install components
command: |
go version
go install gotest.tools/gotestsum@v1.11.0
- run:
name: Install Kurtosis
command: |
echo "deb [trusted=yes] https://apt.fury.io/kurtosis-tech/ /" | sudo tee /etc/apt/sources.list.d/kurtosis.list
sudo apt update
sudo apt install kurtosis-cli=1.4.0
kurtosis engine start
- checkout
- when:
condition: <<parameters.uses_artifacts>>
steps:
- attach_workspace: { at: "." }
- run:
name: prep results dir
command: |
# Make sure the workspace is properly owned
mkdir -p ./tmp/test-results
mkdir -p ./tmp/testlogs
- run:
name: run tests
command: |
ENABLE_KURTOSIS=true gotestsum \
--format=testname \
--junitfile=../tmp/test-results/<<parameters.module>>.xml \
--jsonfile=../tmp/testlogs/log.json \
-- -parallel=$(nproc) \
-coverpkg=github.com/ethereum-optimism/optimism/... \
-coverprofile=coverage.out <<parameters.test_directory>>
working_directory: <<parameters.module>>
- store_test_results:
path: tmp/test-results
- store_artifacts:
path: tmp/testlogs
when: always
go-tests:
parameters:
notify:
......@@ -919,6 +862,7 @@ jobs:
formatted_packages="$formatted_packages ./$package/..."
done
export ENABLE_KURTOSIS=true
export OP_E2E_CANNON_ENABLED="false"
export OP_E2E_SKIP_SLOW_TEST=true
export OP_E2E_USE_HTTP=true
......@@ -1351,12 +1295,6 @@ workflows:
on_changes: op-e2e,packages/contracts-bedrock/src
uses_artifacts: true
requires: ["contracts-bedrock-build"]
- go-test-kurtosis:
name: op-deployer-integration
module: op-deployer
test_directory: ./pkg/deployer/integration_test
uses_artifacts: true
requires: ["contracts-bedrock-build"]
- go-tests:
packages: |
op-batcher
......@@ -1391,7 +1329,6 @@ workflows:
- check-generated-mocks-op-node
- check-generated-mocks-op-service
- go-mod-download
- op-deployer-integration
- op-program-compat
# Not needed for the devnet but we want to make sure they build successfully
- cannon-docker-build
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment