Skip to content

Fix zombienet tests: Use compressed runtimes & log errors.#11641

Merged
eskimor merged 15 commits intomasterfrom
rk-fix-code-upgrade-too-large-in-testing
Apr 7, 2026
Merged

Fix zombienet tests: Use compressed runtimes & log errors.#11641
eskimor merged 15 commits intomasterfrom
rk-fix-code-upgrade-too-large-in-testing

Conversation

@eskimor
Copy link
Copy Markdown
Member

@eskimor eskimor commented Apr 4, 2026

Test runtimes grew too large, we need compression now to fit the limits. This fixes the runtime upgrading zombienet tests. In addition I added logging, such that errors like this will be easier to find next time.

Fixes: #11568

@eskimor eskimor added T0-node This PR/Issue is related to the topic “node”. T10-tests This PR/Issue is related to tests. T18-zombienet_tests Trigger zombienet CI tests. labels Apr 4, 2026
@eskimor eskimor changed the title Use compressed runtimes & log errors. Fix zombienet tests: Use compressed runtimes & log errors. Apr 4, 2026
@eskimor
Copy link
Copy Markdown
Member Author

eskimor commented Apr 4, 2026

/cmd fmt

@eskimor eskimor force-pushed the rk-fix-code-upgrade-too-large-in-testing branch from c1bad62 to 6c5ba25 Compare April 4, 2026 20:11
@eskimor eskimor requested a review from koute as a code owner April 4, 2026 21:11
@eskimor eskimor force-pushed the rk-fix-code-upgrade-too-large-in-testing branch from b6cb9fd to da55577 Compare April 4, 2026 22:19
Copy link
Copy Markdown
Contributor

@pepoviola pepoviola left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!!

@eskimor eskimor added this pull request to the merge queue Apr 7, 2026
Merged via the queue into master with commit 1b71ca6 Apr 7, 2026
257 of 259 checks passed
@eskimor eskimor deleted the rk-fix-code-upgrade-too-large-in-testing branch April 7, 2026 16:01
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

None of these changes should be required. I don't get why you are touching this file, when you are trying to use compressed files in zombienet. The zombienet test are compiled any way in release mode? Please explain why this is required.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Correct fix here: #11765

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

Labels

T0-node This PR/Issue is related to the topic “node”. T10-tests This PR/Issue is related to tests. T18-zombienet_tests Trigger zombienet CI tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

zombienet test fails after #7035

5 participants