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.

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

[test] Do not commit #16129

Closed
wants to merge 1 commit into from
Closed

[test] Do not commit #16129

wants to merge 1 commit into from

Conversation

perryjrandall
Copy link
Contributor

Description

How Has This Been Tested?

Key Areas to Review

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
  • Move Compiler
  • Other (specify)

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

Copy link

trunk-io bot commented Mar 16, 2025

⏱️ 41m total CI duration on this PR
Slowest 15 Jobs Cumulative Duration Recent Runs
execution-performance / test-target-determinator 9m 🟩🟩
semgrep/ci 8m 🟩🟩🟥
check-dynamic-deps 8m 🟩🟩🟩
rust-doc-tests 6m 🟩
test-target-determinator 5m 🟩
rust-cargo-deny 2m 🟩
fetch-last-released-docker-image-tag 1m 🟩
general-lints 53s 🟩
file_change_determinator 22s 🟩🟩
execution-performance / single-node-performance 17s 🟩🟩
file_change_determinator 10s 🟩
permission-check 10s 🟩🟩🟩
permission-check 6s 🟩🟩🟩
determine-docker-build-metadata 3s 🟩
permission-check 2s 🟩

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

Job Duration vs 7d avg Delta
semgrep/ci 7m 24s +1691%
check-dynamic-deps 5m 2m +121%
execution-performance / single-node-performance 8s 20m -99%

settingsfeedbackdocs ⋅ learn more about trunk.io

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@perryjrandall perryjrandall added CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR CICD:run-framework-upgrade-test labels Mar 16, 2025

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Copy link
Contributor

❌ Forge suite compat failure on 82240c9c7137087c575bf5d670abfa0dddc3ae9f ==> c72f1271e9c1afda8b4d14fa29aff104f140ddf5

Forge test runner terminated:
Trailing Log Lines:
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:552:40
  34: std::panicking::try
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:516:19
  35: std::panic::catch_unwind
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panic.rs:146:14
  36: std::rt::lang_start_internal
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/rt.rs:148:20
  37: main
  38: __libc_start_main
  39: _start
Debugging output:
NAME                           READY   STATUS   RESTARTS   AGE
forge-testnet-deployer-lgngv   0/1     Error    0          36s

Copy link
Contributor

❌ Forge suite realistic_env_max_load failure on c72f1271e9c1afda8b4d14fa29aff104f140ddf5

Forge test runner terminated:
Trailing Log Lines:
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:552:40
  34: std::panicking::try
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:516:19
  35: std::panic::catch_unwind
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panic.rs:146:14
  36: std::rt::lang_start_internal
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/rt.rs:148:20
  37: main
  38: __libc_start_main
  39: _start
Debugging output:
NAME                           READY   STATUS   RESTARTS   AGE
forge-testnet-deployer-5576q   0/1     Error    0          37s

Copy link
Contributor

❌ Forge suite framework_upgrade failure on 82240c9c7137087c575bf5d670abfa0dddc3ae9f ==> c72f1271e9c1afda8b4d14fa29aff104f140ddf5

Forge test runner terminated:
Trailing Log Lines:
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:552:40
  34: std::panicking::try
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panicking.rs:516:19
  35: std::panic::catch_unwind
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/panic.rs:146:14
  36: std::rt::lang_start_internal
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/std/src/rt.rs:148:20
  37: main
  38: __libc_start_main
  39: _start
Debugging output:
NAME                           READY   STATUS   RESTARTS   AGE
forge-testnet-deployer-4vckg   0/1     Error    0          43s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CICD:run-e2e-tests when this label is present github actions will run all land-blocking e2e tests from the PR CICD:run-framework-upgrade-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant