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.

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

[network-definition-spec] improve keystore keys network spec #1370

Merged

Conversation

klbrvik
Copy link
Contributor

@klbrvik klbrvik commented Sep 25, 2023

Current implementation doesn't allow to create desired keys in node keystore.

Suggested changes are backward compatible and will allow to define custom key types together with their schemas.

@klbrvik klbrvik changed the title [network-definition-spec] improve keystore keys network spec by allowing to specify custom key types and their schemas [network-definition-spec] improve keystore keys network spec Sep 25, 2023
Copy link
Collaborator

@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.

Look great @klbrvik 🚀 , just one nit. Why the default scheme is ed since most of the keys are sr?

@klbrvik
Copy link
Contributor Author

klbrvik commented Sep 26, 2023

Look great @klbrvik 🚀 , just one nit. Why the default scheme is ed since most of the keys are sr?

No specific reason, I'll change it to sr since it's most prevalent.

@klbrvik klbrvik force-pushed the improve-keystore-keys-network-spec branch 2 times, most recently from f2083ff to af99462 Compare September 26, 2023 07:40
@klbrvik klbrvik force-pushed the improve-keystore-keys-network-spec branch from 9af244c to 7982570 Compare September 26, 2023 07:41
@klbrvik
Copy link
Contributor Author

klbrvik commented Sep 26, 2023

continuous-integration/gitlab-db-snapshot is failing but I cannot access logs at GitLab with following:

Your account is pending approval from your GitLab administrator and hence blocked. Please contact your GitLab administrator if you think this is an error.

So currently blocked from fixing.

@klbrvik klbrvik requested a review from pepoviola September 26, 2023 11:58
@pepoviola
Copy link
Collaborator

continuous-integration/gitlab-db-snapshot is failing but I cannot access logs at GitLab with following:

Your account is pending approval from your GitLab administrator and hence blocked. Please contact your GitLab administrator if you think this is an error.

So currently blocked from fixing.

This is a known issue, merging now. Thanks for your contribution!!

@pepoviola pepoviola merged commit 3b87796 into paritytech:main Sep 26, 2023
8 checks passed
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.

2 participants