Skip to content

Conversation

@muharem
Copy link
Contributor

@muharem muharem commented Sep 17, 2025

fix scheduler and checking balance tests failures reported from dry runs

fixes https://github.com/paritytech/ahm-dryrun/actions/runs/17782824014/job/50544999858#step:25:1153

Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Next failure after I run the test against this branch:

2025-09-18T01:01:36.801638Z  INFO runtime::frame-support: 🩺 Running "RcMigrator" try-state checks    
2025-09-18T01:01:37.763118Z ERROR runtime::defensive: a defensive failure has been triggered; please report the block number at https://github.com/paritytech/polkadot-sdk/issues: "ah_checking_balance_before ==\npallet_balances::Pallet::<AhRuntime>::minimum_balance()"    

thread 'tests::post_migration_checks_only' panicked at /Users/kianenigma/Desktop/Parity/ahm-dryrun/runtimes/pallets/ah-migrator/src/society.rs:217:4:
NextIntakeAt should be empty on the relay chain after migration

Makes sense, probably the on-init of pallet-society is still running post migration and populating this?

@kianenigma
Copy link
Contributor

@muharem pushed a few of the tests that still fail ahead of you. Afterwards, I still get:

Total balance mismatch for account "5DyvUvG3oG6858tdKvvAKguED85RMHgc2KUWXgT562uYcMYh" between RC pre-migration and AH post-migration
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
FAILED

failures:

failures:
    tests::post_migration_checks_only

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 21 filtered out; finished in 28.50s

@ggwpez ggwpez merged commit 4111ecf into oty-dev-asset-hub-migration-2506 Sep 22, 2025
51 of 63 checks passed
@ggwpez ggwpez deleted the muharem-ahm-scheduler-tests-fix branch September 22, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants