Skip to content

Commit

Permalink
ci: disable put results to cache on PR builds, also disable dump ya m…
Browse files Browse the repository at this point in the history
…ake graph
  • Loading branch information
nikitka committed Nov 30, 2023
1 parent 2e9a8ee commit 84f2d6a
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 5 deletions.
1 change: 0 additions & 1 deletion .github/actions/build_ya/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ runs:
./ya make -k --build "${build_type}" --force-build-depends -D'BUILD_LANGUAGES=CPP PY3 PY2 GO' -T --stat \
--log-file "$TMP_DIR/ya_log.txt" --evlog-file "$TMP_DIR/ya_evlog.jsonl" \
--dump-graph --dump-graph-to-file "$TMP_DIR/ya_graph.json" \
--cache-size 512G --link-threads "${{ inputs.link_threads }}" \
"${extra_params[@]}"
Expand Down
9 changes: 6 additions & 3 deletions .github/workflows/build_and_test_ya.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,9 @@ on:
type: boolean
default: false
description: "Use cache for tests"
put_build_results_to_cache:
type: boolean
default: true

jobs:
main:
Expand Down Expand Up @@ -73,9 +76,9 @@ jobs:
with:
build_target: ${{ inputs.build_target }}
build_preset: ${{ inputs.build_preset }}
bazel_remote_uri: ${{ vars.REMOTE_CACHE_URL_YA || '' }}
bazel_remote_username: ${{ secrets.REMOTE_CACHE_USERNAME }}
bazel_remote_password: ${{ secrets.REMOTE_CACHE_PASSWORD }}
bazel_remote_uri: ${{ inputs.put_build_results_to_cache && vars.REMOTE_CACHE_URL_YA || '' }}
bazel_remote_username: ${{ inputs.put_build_results_to_cache && secrets.REMOTE_CACHE_USERNAME || '' }}
bazel_remote_password: ${{ inputs.put_build_results_to_cache && secrets.REMOTE_CACHE_PASSWORD || '' }}
link_threads: ${{ inputs.link_threads }}

- name: Run tests
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/build_and_test_ya_ondemand.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,6 +83,9 @@ on:
type: string
default: 8
description: "link threads count"
put_build_results_to_cache:
type: boolean
default: true

jobs:
provide-runner:
Expand Down Expand Up @@ -141,6 +144,7 @@ jobs:
test_type: ${{ inputs.test_type }}
link_threads: ${{ inputs.link_threads }}
test_threads: ${{ inputs.test_threads }}
put_build_results_to_cache: ${{ inputs.put_build_results_to_cache }}
secrets: inherit

release-runner:
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/build_and_test_ya_provisioned.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,9 @@ on:
type: boolean
default: false
description: "Use cache for tests"
put_build_results_to_cache:
type: boolean
default: true
jobs:
main:
uses: ./.github/workflows/build_and_test_ya.yml
Expand All @@ -105,4 +108,5 @@ jobs:
link_threads: ${{ inputs.link_threads }}
test_threads: ${{ inputs.test_threads }}
cache_tests: ${{ inputs.cache_tests }}
put_build_results_to_cache: ${{ inputs.put_build_results_to_cache }}
secrets: inherit
2 changes: 1 addition & 1 deletion .github/workflows/pr_check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -111,5 +111,5 @@ jobs:
test_type: "unittest,py3test,py2test,pytest"
test_threads: 52
runner_label: auto-provisioned
cache_tests: true
put_build_results_to_cache: false
secrets: inherit

0 comments on commit 84f2d6a

Please sign in to comment.