Skip to content

Carthage Deployment Scripts#4417

Merged
mnaamani merged 31 commits intoJoystream:carthagefrom
mnaamani:carthage-deployment-scripts
Nov 5, 2022
Merged

Carthage Deployment Scripts#4417
mnaamani merged 31 commits intoJoystream:carthagefrom
mnaamani:carthage-deployment-scripts

Conversation

@mnaamani
Copy link
Copy Markdown
Member

@mnaamani mnaamani commented Oct 31, 2022

  • Include git commit hash and runtime code shasum to the node version string when doing docker build. We do this manually by setting env variable as we no longer copy the entire .git repo into the docker build context and the rust custom build fails to get git commit.

  • Fixed bug in chainspec builder "missing" // prefix on seeds when generating new random seeds. (generate sub command)

  • Added support for the chainspec builder new subcommand to pass either seeds or full validator accounts in --authorities argument. More secure than providing seeds on deployment host.

  • Added a utility session-keys to create validator session keys in a keystore. This allows keys to be generated on individual validators, and deployment scripts do not need to fetch from single host and push to validators. More secure setup.

  • Updated ansible deployment scripts to support adding separate boot nodes, instead of using validators as boot nodes, and using session-keys to generate keys on validators. Other minor refactoring how node keys and keystore are stored on target hosts.

┆Issue is synchronized with this Asana task by Unito
┆Link To Task: https://app.asana.com/0/1202132419573087/1203291205609117

@vercel
Copy link
Copy Markdown

vercel bot commented Oct 31, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
pioneer-testnet ⬜️ Ignored (Inspect) Nov 4, 2022 at 2:39PM (UTC)

@mnaamani mnaamani marked this pull request as draft November 2, 2022 21:06
@dobertRowneySr dobertRowneySr marked this pull request as ready for review November 3, 2022 08:31
Copy link
Copy Markdown
Collaborator

@dobertRowneySr dobertRowneySr left a comment

Choose a reason for hiding this comment

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

Rust part LGTM

@mnaamani mnaamani merged commit 3b85588 into Joystream:carthage Nov 5, 2022
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