-
Notifications
You must be signed in to change notification settings - Fork 837
chore: task reexecution #4443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
+350
−171
Merged
chore: task reexecution #4443
Changes from 25 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
5964641
chore: externalize c-chain-reexecution json based workflow into tasks
Elvis339 70fbc9f
ci(c-chain-reexecution): remove relative imports to make this action …
Elvis339 d5bc227
ci(c-chain-reexecution): use correct shells
Elvis339 ff86c39
ci(c-chain-reexecution)!:
Elvis339 b88c5ea
copilot PR comments
Elvis339 c753cf9
ci(c-chain-reexecution): set absolute path to benchmark output file r…
Elvis339 7b3a231
address pr review
Elvis339 c0281ed
address PR
Elvis339 10c5ef7
ci(c-chain-reexecution-benchmark): revert change on `Push Post-State …
Elvis339 7d7956c
Merge branch 'master' into es/task-reexecution
Elvis339 a42e26b
Merge branch 'master' into es/task-reexecution
Elvis339 785278f
Merge branch 'master' into es/task-reexecution
Elvis339 c5b650a
chore(benchamrk_cchain_range): avoid duplicating the default of `RUNN…
Elvis339 20c4ad0
ci(c-chain-reexecution):
Elvis339 f9b3ce0
Merge branch 'master' into es/task-reexecution
Elvis339 519e92a
chore: rm doc
Elvis339 f3d56fb
Merge remote-tracking branch 'origin/es/task-reexecution' into es/tas…
Elvis339 69c38b5
Merge branch 'master' into es/task-reexecution
Elvis339 a800ff1
ci(c-chain-reexecution): support alternative approach to task based o…
Elvis339 f7f0ccf
ci(c-chain-reexecution): support both container and native altenrativ…
Elvis339 ef5cae7
Merge remote-tracking branch 'origin/es/task-reexecution' into es/tas…
Elvis339 7921a03
lint
Elvis339 e703fe0
refactor(scripts): extract S3 bucket config from copy_dir.sh to Taskf…
Elvis339 d359c7a
fix
Elvis339 30e7b0c
Merge branch 'master' into es/task-reexecution
Elvis339 0cb04dc
ci(c-chain-reexecution): use dynamic env variable for benchmark outpu…
Elvis339 8082a9e
Merge remote-tracking branch 'origin/es/task-reexecution' into es/tas…
Elvis339 3b2a82f
ci(c-chain-reexecution): remove unused `TASK_NAME` environment variable
Elvis339 3483737
ci(c-chain-reexecution): enforce required `runner` parameter by remov…
Elvis339 c14c088
Merge branch 'master' into es/task-reexecution
Elvis339 5aab3dc
docs(reexecute)
Elvis339 7182799
Merge branch 'master' into es/task-reexecution
Elvis339 4615490
ci(c-chain-reexecution): add runner presets with custom override
Elvis339 6090a32
Merge branch 'es/task-reexecution' of github.com:ava-labs/avalanchego…
Elvis339 c8b4a5a
Merge branch 'master' into es/task-reexecution
Elvis339 1dc34a4
Merge branch 'master' into es/task-reexecution
Elvis339 fa2ebb5
ci(c-chain-reexecution):
Elvis339 1079097
Merge remote-tracking branch 'origin/es/task-reexecution' into es/tas…
Elvis339 2d97f32
Merge branch 'master' into es/task-reexecution
Elvis339 1186fb2
Merge branch 'master' into es/task-reexecution
Elvis339 171634f
ci(c-chain-reexecution): extend workflow inputs with additional param…
Elvis339 b4e92af
ci(c-chain-reexecution): set runner in container json
Elvis339 9e90f3f
ci(c-chain-reexecution): fix s3 path
Elvis339 72ed001
lint
Elvis339 8dc0563
ci(c-chain-reexecution): remove fallback to workflow inputs for matri…
Elvis339 875b12f
ci(c-chain-reexecution): comment out pull_request trigger and fix mat…
Elvis339 2b3a0e0
ci(c-chain-reexecution)
Elvis339 41ddb71
ci(c-chain-reexecution): install xz-utils regardless
Elvis339 a3cb90a
ci: lint
Elvis339 bdcfb84
test
Elvis339 b83c631
test
Elvis339 872d035
ci(c-chain-reexecution): add default value to task fixing weird bug t…
Elvis339 013834c
lint
Elvis339 8d3fd52
ci(c-chain-reexecution): add fallback to workflow inputs for matrix p…
Elvis339 0ff68b7
ci(c-chain-reexecution): reorder matrix input, adjust defaults, and u…
Elvis339 12afd9d
lint
Elvis339 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
23 changes: 23 additions & 0 deletions
23
.github/actions/c-chain-reexecution-benchmark/output-metrics-url.sh
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| # WARNING: This file is a duplication of: | ||
| # - .github/actions/run-monitored-tmpnet-cmd/output-metrics-url.sh (source of truth) | ||
| # Changes must be made to BOTH files. | ||
|
|
||
| set -euo pipefail | ||
|
|
||
| # Timestamps are in seconds | ||
| from_timestamp="$(date '+%s')" | ||
| monitoring_period=900 # 15 minutes | ||
| to_timestamp="$((from_timestamp + monitoring_period))" | ||
|
|
||
| # Grafana expects microseconds, so pad timestamps with 3 zeros | ||
| metrics_url="${GRAFANA_URL}&var-filter=gh_job_id%7C%3D%7C${GH_JOB_ID}&from=${from_timestamp}000&to=${to_timestamp}000" | ||
|
|
||
| # Optionally ensure that the link displays metrics only for the shared | ||
| # network rather than mixing it with the results for private networks. | ||
| if [[ -n "${FILTER_BY_OWNER:-}" ]]; then | ||
| metrics_url="${metrics_url}&var-filter=network_owner%7C%3D%7C${FILTER_BY_OWNER}" | ||
| fi | ||
|
|
||
| echo "${metrics_url}" |
4 changes: 4 additions & 0 deletions
4
.github/actions/run-monitored-tmpnet-cmd/output-metrics-url.sh
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.