Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

bump zombienet version, support new weights #5965

Closed
wants to merge 2 commits into from
Closed

Conversation

pepoviola
Copy link
Contributor

Add support for weights changes (paritytech/substrate#12138).

@pepoviola pepoviola requested review from ordian and sandreim September 2, 2022 18:03
@pepoviola pepoviola requested a review from a team as a code owner September 2, 2022 18:03
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Sep 2, 2022
@paritytech-ci paritytech-ci requested a review from a team September 2, 2022 18:03
@pepoviola pepoviola added I5-tests Tests need fixing, improving or augmenting. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Sep 2, 2022
@ordian
Copy link
Member

ordian commented Sep 2, 2022

The new table output looks very nice :)

The gitlab-zombienet-test-parachains-upgrade-smoke-test is failing though.

@ordian
Copy link
Member

ordian commented Sep 2, 2022

Probably has to do with paritytech/substrate#12157 and #5964.

@pepoviola
Copy link
Contributor Author

Probably has to do with paritytech/substrate#12157 and #5964.

Thanks @ordian! Yes, make sense. I will revert to adjust the js script to fix that.

@pepoviola
Copy link
Contributor Author

Closing here, since the weight change was removed in paritytech/substrate#12157.

@pepoviola pepoviola closed this Sep 5, 2022
@ordian ordian deleted the bump-zombienet branch September 5, 2022 12:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. I5-tests Tests need fixing, improving or augmenting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants