disable flaky tests temporarily & update version #20531
Conversation
product-security-plaid-production
bot
requested review from
kdihalas and
skudasov
There was a problem hiding this comment.
Pull request overview
This PR temporarily disables two flaky CCIP upgrade tests related to EVM to Sui integration by commenting them out in the integration test configuration file.
Key changes:
- Commented out
Test_CCIP_Upgrade_EVM2Suitest configuration - Commented out
Test_CCIP_Upgrade_CommonPkg_EVM2Suitest configuration
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # - id: smoke/ccip/ccip_sui_upgrade_test.go:Test_CCIP_Upgrade_EVM2Sui | ||
| # path: integration-tests/smoke/ccip/ccip_sui_upgrade_test.go | ||
| # test_env_type: in-memory | ||
| # runs_on_self_hosted: runs-on/cpu=16/ram=64/family=m6i+m5.*/spot=false/image=ubuntu24-full-x64/extras=s3-cache+tmpfs | ||
| # triggers: | ||
| # - PR Integration CCIP Tests | ||
| # - Nightly Integration CCIP Tests | ||
| # test_cmd: | | ||
| # go test ./smoke/ccip -run "Test_CCIP_Upgrade_EVM2Sui" -timeout 25m -test.parallel=1 -count=1 -json | ||
| # test_go_project_path: integration-tests | ||
| # sui_cli_version: mainnet-1.60.1 | ||
| # install_plugins_public: true | ||
| # free_disk_space: true |
There was a problem hiding this comment.
Add a comment explaining why these tests are disabled and reference a tracking issue (e.g., 'Temporarily disabled due to flakiness - see LINK-777'). This helps maintainers understand when and why these tests should be re-enabled.
| # - id: smoke/ccip/ccip_sui_upgrade_test.go:Test_CCIP_Upgrade_CommonPkg_EVM2Sui | ||
| # path: integration-tests/smoke/ccip/ccip_sui_upgrade_test.go | ||
| # test_env_type: in-memory | ||
| # runs_on: ubuntu-latest | ||
| # triggers: | ||
| # - PR Integration CCIP Tests | ||
| # - Nightly Integration CCIP Tests | ||
| # test_cmd: | | ||
| # go test ./smoke/ccip -run "Test_CCIP_Upgrade_CommonPkg_EVM2Sui" -timeout 25m -test.parallel=1 -count=1 -json | ||
| # test_go_project_path: integration-tests | ||
| # sui_cli_version: mainnet-1.60.1 | ||
| # install_plugins_public: true | ||
| # free_disk_space: true |
There was a problem hiding this comment.
Add a comment explaining why these tests are disabled and reference a tracking issue (e.g., 'Temporarily disabled due to flakiness - see LINK-777'). This helps maintainers understand when and why these tests should be re-enabled.
|
added a ticket to track fixing these: https://smartcontract-it.atlassian.net/browse/NONEVM-2558 |
FelixFan1992
dismissed stale reviews from ro-tex and anirudhwarrier
via
6eeed22
FelixFan1992
changed the title
|
5 participants
Requires
Supports