Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[ignore] test PR for changes in new workflow for compat test #14701

Open
wants to merge 37 commits into
base: yzaccc/auto-bump-forge-compat-base-image-to-latest-release
Choose a base branch
from
Open

[ignore] test PR for changes in new workflow for compat test #14701

wants to merge 37 commits into from
+201 −14

Conversation

yzaccc
Copy link

@yzaccc yzaccc commented Sep 20, 2024

This is a PR to test workflow against changes in yzaccc/auto-bump-forge-compat-base-image-to-latest-release
#e2e

Description

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Performance improvement
  • Refactoring
  • Dependency update
  • Documentation update
  • Tests

Which Components or Systems Does This Change Impact?

  • Validator Node
  • Full Node (API, Indexer, etc.)
  • Move/Aptos Virtual Machine
  • Aptos Framework
  • Aptos CLI/SDK
  • Developer Infrastructure
  • Other (specify)

How Has This Been Tested?

Key Areas to Review

Checklist

  • I have read and followed the CONTRIBUTING doc
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I identified and added all stakeholders and component owners affected by this change as reviewers
  • I tested both happy and unhappy path of the functionality
  • I have made corresponding changes to the documentation

Zac Yang added 18 commits September 19, 2024 02:16
Automatically bump Forge compat base image to latest release
df93f9b
refined
29f276f
temporarilly changing the trigger from pull_request_target to pull_re…
03bbc8f 
…quest
temporarilly changing the trigger from pull_request_target to pull_re…
8d4bf67 
…quest
temporarilly changing the trigger from pull_request_target to pull_re…
e9b4bbc 
…quest
refined the algorithm to find the last release branch
7410679
refined workflow
5f91ed3
refined workflow
85e0751
refined workflow
a244c00
fix rust-images-consensus-only-perf-test errors
dc4d810
fix rust-images-consensus-only-perf-test errors
bbecf69
refined
e6a5e43
refined
2197361
changed put logic in py script in stead of bash
4ed32c0
changed put logic in py script in stead of bash
4f72522
changed put logic in py script in stead of bash
08f4ad1
changed put logic in py script in stead of bash
5641bbf
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
ab9f125 
…test-release
@yzaccc yzaccc added CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR CICD:build-failpoints-images Build failpoints docker image CICD:build-performance-images build performance docker image variants labels Sep 20, 2024
@trunk-io Trunk.io
Copy link

trunk-io bot commented Sep 20, 2024 •
edited
Loading

⏱️ 8h 58m total CI duration on this PR
Job Cumulative Duration Recent Runs
execution-performance / single-node-performance 3h 24m 🟥 🟩 🟩 🟩 🟥 (+5 more)
test-target-determinator 1h 4m 🟩 🟩 🟩 🟩 🟩 (+15 more)
forge-e2e-test / forge 46m 🟩 🟩 🟩 ⬜ 
fetch-last-released-docker-image-tag 40m 🟥 🟥 🟥 🟥 🟥 (+15 more)
execution-performance / test-target-determinator 33m 🟩 🟩 🟩 🟩 🟩 (+5 more)
check 22m 🟩 🟩 🟩 ⬜ 🟩 (+2 more)
check-dynamic-deps 11m 🟩 🟩 🟩 🟩 🟩 (+5 more)
rust-move-tests 10m 🟩 
rust-move-tests 10m 🟩 
rust-move-tests 9m 🟩 
rust-move-tests 9m 🟩 
indexer-grpc-e2e-tests / test-indexer-grpc-docker-compose 9m 🟩 🟩 🟩 🟩 🟥 (+1 more)
rust-move-tests 8m ⬜ 
rust-move-tests 7m ⬜ 
rust-doc-tests 5m 🟩 
rust-doc-tests 5m 🟩 
rust-doc-tests 4m 🟩 
rust-doc-tests 4m 🟩 
rust-doc-tests 4m 🟩 
rust-doc-tests 4m 🟩 
rust-cargo-deny 4m 🟥 🟥 🟥 ⬜ 🟩 (+2 more)
file_change_determinator 4m 🟩 🟩 🟩 🟩 🟩 (+15 more)
file_change_determinator 4m 🟩 🟩 🟩 🟩 🟩 (+15 more)
semgrep/ci 4m 🟩 🟩 🟩 🟩 🟩 (+5 more)
general-lints 3m 🟩 🟩 🟩 ⬜ 🟥 (+2 more)
permission-check 1m 🟩 🟩 🟩 🟩 🟩 (+18 more)
permission-check 1m 🟩 🟩 🟩 🟩 🟩 (+18 more)
permission-check 1m 🟩 🟩 🟩 🟩 🟩 (+16 more)
file_change_determinator 1m 🟩 🟩 🟩 🟩 🟩 (+2 more)
permission-check 1m 🟩 🟩 🟩 🟩 🟩 (+15 more)
permission-check 56s 🟩 🟩 🟩 🟩 🟩 (+15 more)
determine-docker-build-metadata 54s 🟩 🟩 🟩 🟩 🟩 (+15 more)
rust-doc-tests 20s ⬜ 
permission-check 13s 🟩 🟩 🟩 
Backport PR 10s 🟥 🟥 🟥 
rust-move-tests 9s ⬜ 

🚨 3 jobs on the last run were significantly faster/slower than expected

Job Duration vs 7d avg Delta
execution-performance / single-node-performance 24m 18m +34%
test-target-determinator 3m 5m -31%
execution-performance / test-target-determinator 3m 5m -33%

settings ⋅ feedback ⋅ docs ⋅ learn more about trunk.io

@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@yzaccc yzaccc requested a review from a team as a code owner September 20, 2024 04:17
@yzaccc yzaccc force-pushed the yzaccc/auto-bump-forge-compat-base-image-to-latest-release branch from fdaaaef to 31a388a Compare September 20, 2024 04:19
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions

This comment has been minimized.

Sign in to view
@github-actions GitHub Actions
Copy link
Contributor

Forge is running suite realistic_env_max_load on 5d48852f9c662ace89dcbcbe64037e4445a8c305

Zac Yang added 10 commits September 20, 2024 01:52
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
7bc9c31 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
5000d1b 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
c612656 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
e7f1b5b 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
97ad2d6 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
1c544c3 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
203b6b7 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
688beba 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
81b8cd2 
…test-release
branch to test against yzaccc/auto-bump-forge-compat-base-image-to-la…
eea1722 
…test-release
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bchocho bchocho Awaiting requested review from bchocho bchocho is a code owner

Assignees
No one assigned
Labels
CICD:build-failpoints-images Build failpoints docker image CICD:build-performance-images build performance docker image variants CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yzaccc