Skip to content

Commit

Permalink
Merge branch 'move-cargo-jobs-off-legacy-image' into 'master'
Browse files Browse the repository at this point in the history
Refactor: remove cargo-ubuntu-k8s [benchmark]

This should have no functional impact. Just cleaning up CI jobs to make them simpler. Let me know if I missed something, but it didn't look like the RUSTFLAGS were being used for anything in the benchmark tests. 

See merge request dfinity-lab/public/ic!15594
  • Loading branch information
cgundy committed Oct 26, 2023
2 parents e89564a + d133687 commit c5b72c8
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 118 deletions.
8 changes: 0 additions & 8 deletions gitlab-ci/config/common.yml
Original file line number Diff line number Diff line change
Expand Up @@ -110,14 +110,6 @@ after_script:
image:
name: "registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513"

.ubuntu-cargo-k8s:
extends:
- .build-k8s-legacy
variables:
# This MUST match the directory used in our `build-ic.sh` script
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"


# Container images used by jobs.

.ic-build-image:
Expand Down
2 changes: 1 addition & 1 deletion gitlab-ci/config/container-image-autobuild.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
container-autobuild-protected:
extends:
- .ubuntu-cargo-k8s
- .build-k8s-legacy
rules:
# Run immediately on Dockerfile changes pushed to a protected branch - such as master.
- if: '$CI_COMMIT_REF_PROTECTED == "true" && $CI_PIPELINE_SOURCE == "push"'
Expand Down
5 changes: 4 additions & 1 deletion gitlab-ci/config/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,12 @@ no-interrupt:

cargo-audit:
extends:
- .ubuntu-cargo-k8s
- .build-k8s-legacy
rules:
- if: '$CI_PIPELINE_SOURCE == "schedule" && $SCHEDULE_NAME == "cargo-audit"'
variables:
# This MUST match the directory used in our `build-ic.sh` script
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=--strip-all"
script:
- cd rs; cargo audit

Expand Down
27 changes: 2 additions & 25 deletions gitlab-ci/config/spawn-pipeline--benchmark.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
.benchmark-test:
extends:
- .ubuntu-cargo-k8s
- .build-k8s-legacy
rules:
- if: '$CI_PIPELINE_SOURCE == "merge_request_event" && $CI_MERGE_REQUEST_TITLE =~ /\[benchmark\]/'
when: manual
Expand Down Expand Up @@ -423,7 +423,7 @@ statesync-experiment:
.benchmark-spot-test:
extends:
- .ubuntu-cargo-k8s
- .build-k8s-legacy
rules:
- if: '$CI_PIPELINE_SOURCE == "merge_request_event" && $CI_MERGE_REQUEST_TITLE =~ /\[benchmark\]/'
when: manual
Expand Down Expand Up @@ -604,29 +604,6 @@ xnet-nightly:
$SHELL_WRAPPER rclone --config="${CI_PROJECT_DIR}/.rclone.conf" copyto "scalability/results/$GIT_REVISION" "performance-testing:performance-testing-results/$GIT_REVISION"
.sns-tests:
extends:
- .ubuntu-cargo-k8s
rules:
- if: '$CI_PIPELINE_SOURCE == "merge_request_event" && $CI_MERGE_REQUEST_TITLE =~ /\[sns\]/'
when: manual
allow_failure: true
- if: '$CI_PIPELINE_SOURCE == "schedule" && $SCHEDULE_NAME == "SNS_CD"'
when: always
resource_group: $TESTNET
dependencies: [] # don't copy artifacts from other jobs by default
variables:
SHELL_WRAPPER: "/usr/bin/time"
DISKIMG_BRANCH: "${CI_COMMIT_SHA}"
CURRENT_BRANCH: "$CI_COMMIT_REF_NAME"
cd_target_env: "BENCHMARK_NIGHTLY"
CD_ENV: "SNS_CD"
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
artifacts:
when: always
paths:
- $CI_JOB_STAGE/$CI_JOB_NAME
timeout: 1 hour
# BENCHMARK SUITE tecdsa
tecdsa:
Expand Down
88 changes: 5 additions & 83 deletions gitlab-ci/config/zz-generated-gitlab-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,7 @@
when: always
dependencies: []
extends:
- ".ubuntu-cargo-k8s"
- ".build-k8s-legacy"
image:
name: registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513
resource_group: "$TESTNET"
Expand All @@ -224,7 +224,6 @@
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdrc02
WG_TESTNET: cdrc02
Expand All @@ -237,7 +236,7 @@
when: always
dependencies: []
extends:
- ".ubuntu-cargo-k8s"
- ".build-k8s-legacy"
image:
name: registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513
resource_group: "$TESTNET"
Expand All @@ -258,7 +257,6 @@
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
".build-base-image": |
Expand Down Expand Up @@ -552,60 +550,6 @@
- allow_failure: true
if: $CI_PIPELINE_SOURCE == "trigger"
when: manual
".sns-tests":
artifacts:
expire_in: 3 days
paths:
- "$CI_JOB_STAGE/$CI_JOB_NAME"
when: always
dependencies: []
extends:
- ".ubuntu-cargo-k8s"
image:
name: registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513
resource_group: "$TESTNET"
rules:
- allow_failure: true
if: $CI_PIPELINE_SOURCE == "merge_request_event" && $CI_MERGE_REQUEST_TITLE =~ /\[sns\]/
when: manual
- if: $CI_PIPELINE_SOURCE == "schedule" && $SCHEDULE_NAME == "SNS_CD"
when: always
tags:
- dfinity-ic
timeout: 1 hour
variables:
CD_ENV: SNS_CD
CURRENT_BRANCH: "$CI_COMMIT_REF_NAME"
DISKIMG_BRANCH: "${CI_COMMIT_SHA}"
GET_SOURCES_ATTEMPTS: 5
GIT_CLONE_PATH: "$CI_BUILDS_DIR/clean/$CI_PROJECT_NAME"
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
cd_target_env: BENCHMARK_NIGHTLY
".ubuntu-cargo-k8s":
artifacts:
expire_in: 3 days
paths:
- junit_data/*
- coredumps/*.txt
- coredumps/*.gz
when: always
extends:
- ".build-k8s-legacy"
image:
name: registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513
tags:
- dfinity-ic
variables:
GET_SOURCES_ATTEMPTS: 5
GIT_CLONE_PATH: "$CI_BUILDS_DIR/clean/$CI_PROJECT_NAME"
GIT_DEPTH: 0
GIT_STRATEGY: fetch
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
after-script-test-ic-build-image:
artifacts:
expire_in: 3 days
Expand Down Expand Up @@ -1799,7 +1743,6 @@ benchmark-response-payload:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
benchmark-xnet:
Expand Down Expand Up @@ -1850,7 +1793,6 @@ benchmark-xnet:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
benchmarks:
Expand Down Expand Up @@ -3157,7 +3099,7 @@ cargo-audit:
- coredumps/*.gz
when: always
extends:
- ".ubuntu-cargo-k8s"
- ".build-k8s-legacy"
image:
name: registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513
rules:
Expand All @@ -3171,7 +3113,7 @@ cargo-audit:
GIT_CLONE_PATH: "$CI_BUILDS_DIR/clean/$CI_PROJECT_NAME"
GIT_DEPTH: 0
GIT_STRATEGY: fetch
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=--strip-all"
SHELL_WRAPPER: "/usr/bin/time"
cargo-build-release-linux:
artifacts:
Expand Down Expand Up @@ -3424,7 +3366,7 @@ container-autobuild-protected:
- coredumps/*.gz
when: always
extends:
- ".ubuntu-cargo-k8s"
- ".build-k8s-legacy"
image:
name: registry.gitlab.com/dfinity-lab/core/docker/ic-build-legacy:8de36394739a3905305fd1f2161b60a788975d147e9efaa46553d068d2b17513
needs: []
Expand Down Expand Up @@ -3466,7 +3408,6 @@ container-autobuild-protected:
GIT_CLONE_PATH: "$CI_BUILDS_DIR/clean/$CI_PROJECT_NAME"
GIT_DEPTH: 0
GIT_STRATEGY: fetch
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
cut-release-candidate:
artifacts:
Expand Down Expand Up @@ -3556,7 +3497,6 @@ delegated-identity-bench:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
dependencies-check:
Expand Down Expand Up @@ -3716,7 +3656,6 @@ heavy-memory-update-performance-test-nightly:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdrc02
WG_TESTNET: cdrc02
Expand Down Expand Up @@ -3829,7 +3768,6 @@ large-response-downloading-nightly:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdrc02
WG_TESTNET: cdrc02
Expand Down Expand Up @@ -4138,7 +4076,6 @@ maximum-capacity-boundary-nodes-queries:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-canister-http:
Expand Down Expand Up @@ -4206,7 +4143,6 @@ maximum-capacity-canister-http:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-large-memory-query:
Expand Down Expand Up @@ -4257,7 +4193,6 @@ maximum-capacity-large-memory-query:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-large-memory-update:
Expand Down Expand Up @@ -4308,7 +4243,6 @@ maximum-capacity-large-memory-update:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-mixed-counter-memory:
Expand Down Expand Up @@ -4362,7 +4296,6 @@ maximum-capacity-mixed-counter-memory:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-mixed-workloads:
Expand Down Expand Up @@ -4413,7 +4346,6 @@ maximum-capacity-mixed-workloads:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-multiple-large-memory-update:
Expand Down Expand Up @@ -4464,7 +4396,6 @@ maximum-capacity-multiple-large-memory-update:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-qr:
Expand Down Expand Up @@ -4515,7 +4446,6 @@ maximum-capacity-qr:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-sha256:
Expand Down Expand Up @@ -4566,7 +4496,6 @@ maximum-capacity-sha256:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-system-baseline-query:
Expand Down Expand Up @@ -4632,7 +4561,6 @@ maximum-capacity-system-baseline-query:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
maximum-capacity-system-baseline-update:
Expand Down Expand Up @@ -4683,7 +4611,6 @@ maximum-capacity-system-baseline-update:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
no-interrupt:
Expand Down Expand Up @@ -5099,7 +5026,6 @@ statesync-experiment:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
system-baseline-query-performance-test-nightly:
Expand Down Expand Up @@ -5155,7 +5081,6 @@ system-baseline-query-performance-test-nightly:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdrc02
WG_TESTNET: cdrc02
Expand Down Expand Up @@ -5213,7 +5138,6 @@ system-baseline-update-performance-test-nightly:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdrc02
WG_TESTNET: cdrc02
Expand Down Expand Up @@ -5266,7 +5190,6 @@ tecdsa:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdslo
test-push-branch:
Expand Down Expand Up @@ -5486,7 +5409,6 @@ xnet-nightly:
GIT_DEPTH: 0
GIT_STRATEGY: fetch
PARENT_PIPELINE_ID: "${CI_PIPELINE_ID}"
RUSTFLAGS: "--remap-path-prefix=${CI_PROJECT_DIR}=/ic -C link-arg=-s"
SHELL_WRAPPER: "/usr/bin/time"
TESTNET: cdrc02
WG_TESTNET: cdrc02
Expand Down

0 comments on commit c5b72c8

Please sign in to comment.