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

feat: deploy Openzeppelin accounts script #1396

Merged
merged 6 commits into from
Sep 6, 2024

Conversation

eugypalu
Copy link
Contributor

@eugypalu eugypalu commented Sep 6, 2024

Time spent on this PR: 0.5

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Resolves #

What is the new behavior?

Script that allows the deployment of OpenZeppelin accounts, saving the private key on AWS Secrets and exporting a JSON containing the addresses of the deployed accounts and the secret ID to retrieve the private key (file stored in kakarot_scripts/data).

Parameters to configure in order to run the script:

  • num_accounts - The number of OZ accounts.
  • amount - The amount to send to the address to deploy the accounts.
  • private_key - The private key to use, which will also be saved on AWS Secrets. Therefore, it only needs to be provided the first time; in subsequent executions, you can use an empty string "" to retrieve the previous key. Otherwise, the secret will be updated.
  • class_hash - The class hash of OpenZeppelinAccount.

To interact with AWS Secrets, boto3 requires the AWS configuration variables to be set up. Here is the documentation


This change is Reviewable

@ClementWalter ClementWalter merged commit 35b4db5 into kkrt-labs:main Sep 6, 2024
6 checks passed
matthieuauger pushed a commit to matthieuauger/kakarot that referenced this pull request Nov 9, 2024
* fix: type constants

* fix: move DEFAULT_BLOCK_GAS_LIMIT to constants.ts

* fix: move IGNORED_KEYS to constants.ts

* fix: move config const to constants.ts

* fix: import

* reorganize constants

* fmt

* update location

---------

Co-authored-by: Thomas Coratger <[email protected]>
Co-authored-by: Thomas Coratger <[email protected]>
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