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

[gas] Bump gas version on main after release from 30 -> 31 #15759

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

perryjrandall
Copy link
Contributor

@perryjrandall perryjrandall commented Jan 16, 2025

Release 1.26 has version 30

Bump gas to ensure we are working on the next version in main

Test Plan: main + debug framework / compat tests

@perryjrandall perryjrandall requested a review from a team as a code owner January 16, 2025 19:45
Copy link

trunk-io bot commented Jan 16, 2025

⏱️ 27m total CI duration on this PR
Job Cumulative Duration Recent Runs
check-dynamic-deps 8m 🟩🟩🟩
test-target-determinator 7m 🟩
rust-cargo-deny 5m 🟩🟩🟩
fetch-last-released-docker-image-tag 3m 🟩🟩
general-lints 1m 🟩🟩🟩
semgrep/ci 1m 🟩🟩🟩
file_change_determinator 29s 🟩🟩🟩
cut-release-branch 26s 🟥
file_change_determinator 23s 🟩🟩
permission-check 8s 🟩🟩🟩
permission-check 7s 🟩🟩🟩
permission-check 6s 🟩🟩
determine-docker-build-metadata 4s 🟩🟩
check-branch-prefix 1s 🟩

settingsfeedbackdocs ⋅ learn more about trunk.io

Copy link
Contributor Author

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

Release 1.26 has version 30

Bump gas to ensure we are working on the next version in main

Test Plan: main + debug framework / compat tests
@perryjrandall perryjrandall force-pushed the 01-16-_gas_bump_gas_version_on_main_after_release_from_30_-_31 branch from e3771e6 to eb7cefa Compare January 16, 2025 19:52

This comment has been minimized.

Copy link
Contributor

❌ Forge suite framework_upgrade failure on f69dc31b0f674662b6bfa941dc7eee9ce9035e29 ==> eb7cefa58df456bcece6ee7496bae55928e5ff04

Forge test runner terminated:
Trailing Log Lines:
             at ./testsuite/forge/src/runner.rs:332:50
  31: forge::run_forge_with_changelog
             at ./testsuite/forge-cli/src/main.rs:428:24
  32: forge::main
             at ./testsuite/forge-cli/src/main.rs:331:21
  33: core::ops::function::FnOnce::call_once
             at /rustc/9b00956e56009bab2aa15d7bff10916599e3d6d6/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
{"level":"INFO","source":{"package":"aptos_forge","file":"testsuite/forge/src/backend/k8s/cluster_helper.rs:381"},"thread_name":"main","hostname":"forge-framework-upgrade-pr-15759-1737058053-f69dc31b0f674662b6b","timestamp":"2025-01-16T20:15:40.339166Z","message":"Deleting namespace forge-framework-upgrade-pr-15759: Some(NamespaceStatus { conditions: None, phase: Some(\"Terminating\") })"}
{"level":"INFO","source":{"package":"aptos_forge","file":"testsuite/forge/src/backend/k8s/cluster_helper.rs:398"},"thread_name":"main","hostname":"forge-framework-upgrade-pr-15759-1737058053-f69dc31b0f674662b6b","timestamp":"2025-01-16T20:15:40.339212Z","message":"aptos-node resources for Forge removed in namespace: forge-framework-upgrade-pr-15759"}
Debugging output:
NAME                                   READY   STATUS      RESTARTS      AGE
aptos-node-0-validator-0               1/1     Running     0             3m16s
aptos-node-1-validator-0               1/1     Running     0             2m36s
aptos-node-2-validator-0               1/1     Running     0             7m39s
aptos-node-3-validator-0               1/1     Running     2 (80s ago)   7m39s
forge-testnet-deployer-vptlf           0/1     Completed   0             7m47s
genesis-aptos-genesis-eforge21-xtl2q   0/1     Completed   0             7m40s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants