-
Notifications
You must be signed in to change notification settings - Fork 598
chore: Setup helm config for alpha testnet #12997
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,86 @@ | ||
| telemetry: | ||
| enabled: true | ||
|
|
||
| aztec: | ||
| realProofs: true | ||
| numberOfDefaultAccounts: 0 | ||
| testAccounts: true | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
| sponsoredFPC: true | ||
| bootstrapENRs: "enr:-LO4QLbJddVpePYjaiCftOBY-L7O6Mfj_43TAn5Q1Y-5qQ_OWmSFc7bTKWHzw5xmdVIqXUiizum_kIRniXdPnWHHcwEEhWF6dGVjqDAwLTExMTU1MTExLTAwMDAwMDAwLTAtMTgwNmEwMjgtMWE1MzBmM2KCaWSCdjSCaXCEI8nh9YlzZWNwMjU2azGhA-_dX6aFcXP1DLk91negbXL2a0mNYGXH4hrMvb2i92I0g3VkcIKd0A,enr:-LO4QN4WF8kFyV3sQVX0C_y_03Eepxk5Wac70l9QJcIDRYwKS6aRst1YcfbTDdvovXdRfKf-WSXNVWViGLhDA-dUz2MEhWF6dGVjqDAwLTExMTU1MTExLTAwMDAwMDAwLTAtMTgwNmEwMjgtMWE1MzBmM2KCaWSCdjSCaXCEIicTHolzZWNwMjU2azGhAsz7aFFYRnP5xjTux5UW-HyEQcW_EJrZMT1CNm79N4g-g3VkcIKd0A,enr:-LO4QFrGfkRaCk_iFTeUjR5ESwo45Eov9hx_T1-BLdoT-iHzFgCiHMT4V1KBtdFp8D0ajLSe5HcNYrhalmdJXgv6NTUEhWF6dGVjqDAwLTExMTU1MTExLTAwMDAwMDAwLTAtMTgwNmEwMjgtMWE1MzBmM2KCaWSCdjSCaXCEIlICt4lzZWNwMjU2azGhAlC6nKB3iDtRFqWKWqxf_t-P9hc-SZ6VFBJV4y3bTZBQg3VkcIKd0A" | ||
| contracts: | ||
| registryAddress: "0x290667b79dd73ce03cd2fc901b905e7f1b001d18" | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Alpha testnet contracts and bootnodes |
||
| slashFactoryAddress: "0xc5aceadbb626630e854ffd6cb39105e9c541fa34" | ||
|
|
||
| network: | ||
| public: true | ||
| setupL2Contracts: false | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Did the fee juice contract get initialized out of band?
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't think we want the helm deployment setting up the L2 contracts. The genesist state has been setup to fund the FPC, we just need to deploy the contract. But I was going to that once I see the network is up and runing producing empty blocks. |
||
| p2pBootstrapNodesAsFullPeers: false | ||
|
|
||
| blobSink: | ||
| enabled: true | ||
| dataStoreConfig: | ||
| dataDir: "/data" | ||
| storageSize: "128Gi" | ||
| dataStoreMapSize: "134217728" # 128 GB | ||
|
|
||
| bot: | ||
| enabled: false | ||
|
|
||
| pxe: | ||
| enabled: false | ||
|
|
||
| faucet: | ||
| enabled: false | ||
|
|
||
| bootNode: | ||
| enabled: false | ||
| # unused. | ||
| externalHost: "http://localhost:8080" | ||
|
|
||
| proverNode: | ||
| l1FixedPriorityFeePerGas: 3 | ||
| l1GasLimitBufferPercentage: 15 | ||
| l1GasPriceMax: 1000 | ||
| maxOldSpaceSize: "8192" | ||
| storageSize: "512Gi" | ||
| resources: | ||
| requests: | ||
| cpu: "3" | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 3 cores so we use the new 4 core pool. Leaves a core for K8s/Google usage
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also 10Gi memory. These have 16GB available but K8s bloat takes some.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Nope, good call. Will add
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nope, will add |
||
| memory: "10Gi" | ||
|
|
||
| validator: | ||
| replicas: 3 | ||
| l1FixedPriorityFeePerGas: 3 | ||
| l1GasLimitBufferPercentage: 15 | ||
| l1GasPriceMax: 1000 | ||
| storageSize: "512Gi" | ||
| sequencer: | ||
| minTxsPerBlock: 0 | ||
| maxTxsPerBlock: 4 | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 4 txs per block (0.1 TPS) |
||
| validator: | ||
| disabled: false | ||
| maxOldSpaceSize: "8192" | ||
| resources: | ||
| requests: | ||
| cpu: "3" | ||
| memory: "10Gi" | ||
|
|
||
| proverAgent: | ||
| replicas: 32 | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 32 x 32 core spot provers (-1 core for K8s) |
||
| bb: | ||
| hardwareConcurrency: 31 | ||
| gke: | ||
| spotEnabled: true | ||
| resources: | ||
| requests: | ||
| memory: "116Gi" | ||
| cpu: "31" | ||
|
|
||
| ethereum: | ||
| chainId: "11155111" | ||
| l1GasPriceMax: 1000 | ||
| l1FixedPriorityFeePerGas: 3 | ||
|
|
||
| jobs: | ||
| deployL1Verifier: | ||
| enable: false | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -50,7 +50,7 @@ export async function startNode( | |
|
|
||
| const testAccounts = nodeConfig.testAccounts ? (await getInitialTestAccounts()).map(a => a.address) : []; | ||
| const sponsoredFPCAccounts = nodeConfig.sponsoredFPC ? [await getSponsoredFPCAddress()] : []; | ||
| const initialFundedAccounts = [...testAccounts, ...sponsoredFPCAccounts]; | ||
| const initialFundedAccounts = testAccounts.concat(sponsoredFPCAccounts); | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I was paranoid that I hadn't used the same concat method used when creating the genesis state in the cli deployment of L1 contracts. |
||
|
|
||
| const { genesisBlockHash, genesisArchiveRoot, prefilledPublicData } = await getGenesisValues(initialFundedAccounts); | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it ok for this value to be false here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, well it depends on what the default behaviour of this is in the cli. I haven't updated our L1 contract deployment process to specify
--sponsored-fpcso I would hope this would mean it would default to false.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same default as
testAccounts, I specified them the same:aztec-packages/yarn-project/cli/src/cmds/l1/index.ts
Line 44 in 73820e4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume that's false. It's not exactly clear
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it defaults to false (checked commander.js docs)