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.

Sign up for GitHub

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

Jump to bottom

ExportGenesisState renamed to ExportGenesisHead in Polkadot-SDK 1.6.0 #1712

Closed
rphmeier opened this issue Jan 31, 2024 · 4 comments · Fixed by #1715
Closed

ExportGenesisState renamed to ExportGenesisHead in Polkadot-SDK 1.6.0 #1712

rphmeier opened this issue Jan 31, 2024 · 4 comments · Fixed by #1715
Assignees
@pepoviola
Labels
bug Something isn't working

Comments

@rphmeier
Copy link

Issue Description

ZombieNet still uses ExportGenesisState and should be able to handle nodes which only have an ExportGenesisHead command.

At the moment, chains build using Polkadot-SDK 1.6.0 cannot use ZombieNet without continuing to expose an ExportGenesisState command.

Steps to reproduce the issue

run ZombieNet with a chain built using Polkadot-SDK 1.6.0 which does have ExportGenesisHead and does not have ExportGenesisState.

Describe the results you received

n/a

Describe the results you expected

n/a

Zombienet version

1.3.91

Provider

Native

Provider version

N/A

Upstream Latest Release

Yes

Additional environment details

Additional environment details

Additional information

Additional information like issue happens only occasionally or issue happens with a particular architecture or on a particular setting

Screenshots

No response
@pepoviola pepoviola self-assigned this Feb 1, 2024
@pepoviola pepoviola added the bug Something isn't working label Feb 1, 2024
@pepoviola
Copy link
Collaborator

Hi @rphmeier, thanks for reporting! I will fix this asap and draft a new release.
Thx!!

@rphmeier
Copy link
Author

rphmeier commented Feb 1, 2024

Thanks @pepoviola !

@pepoviola pepoviola mentioned this issue Feb 2, 2024
Merged
pepoviola added a commit that referenced this issue Feb 2, 2024
@pepoviola
Fix #1712, add new cliArgsVersion and clean the logic (#1715)
28a665d 
* fix(orchestrator): Add new cliArgs version to support 'export-genesis-head' subcommand and clean the logic to set the version in the diff. providers

* typo

* add debug
@pepoviola
Copy link
Collaborator

Hi @rphmeier, this should be fixed by #1715. I will draft a new release today including this and other fixes. Thanks again for reporting!

@pepoviola
Copy link
Collaborator

Hi @rphmeier, I just release a new version https://github.com/paritytech/zombienet/releases/tag/v1.3.92 that includes this fix.

Thanks!

gabriele-0201 added a commit to thrumdev/blobs that referenced this issue Feb 9, 2024
@gabriele-0201
node: remove export-genesis-state alias
5ca5a9c 
The alias is no longer needed by zombienet after the following issue was solved in version 1.3.92
paritytech/zombienet#1712 (comment)
@gabriele-0201 gabriele-0201 mentioned this issue Feb 9, 2024
Merged
rphmeier pushed a commit to thrumdev/blobs that referenced this issue Feb 9, 2024
@gabriele-0201 @rphmeier
node: remove export-genesis-state alias
ce38387 
The alias is no longer needed by zombienet after the following issue was solved in version 1.3.92
paritytech/zombienet#1712 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pepoviola pepoviola

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #1712, add new cliArgsVersion and clean the logic paritytech/zombienet
2 participants
@pepoviola @rphmeier