Skip to content

chore: trim down fisherman deployment#18062

Merged
alexghr merged 1 commit intonextfrom
ag/slim-fisherman-deployments
Oct 31, 2025
Merged

chore: trim down fisherman deployment#18062
alexghr merged 1 commit intonextfrom
ag/slim-fisherman-deployments

Conversation

@alexghr
Copy link
Contributor

@alexghr alexghr commented Oct 29, 2025

This PR builds on top of #18060 and removes some components that are not going to be necessary for fisherman networks (like web3signer)

custom_settings = {
"nodeType" = "rpc"
"replicaCount" = var.RPC_REPLICAS
"node.env.NETWORK" = var.NETWORK
Copy link
Member

Choose a reason for hiding this comment

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

did you mean to remove this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes! The network env var is already handled in the underlying helm chart. This was duplicating the env definition

@alexghr alexghr force-pushed the ag/slim-fisherman-deployments branch 2 times, most recently from 994acfb to 826d4c4 Compare October 30, 2025 18:18
@alexghr alexghr force-pushed the ag/slim-fisherman-deployments branch from 826d4c4 to 7227f1c Compare October 30, 2025 18:43
@alexghr alexghr added this pull request to the merge queue Oct 31, 2025
@AztecBot
Copy link
Collaborator

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033[38;2;188;109;208mFLAKED\033[0m (\033[38;2;250;217;121m8;;http://ci.aztec-labs.com/37cfe678444216f1�37cfe678444216f18;;�\033[0m): BOX=vite BROWSER=firefox run_compose_test vite-firefox box boxes (431s) (code: 1) (\033[38;2;188;109;208mAlex Gherghisan\033[0m: chore: trim down fisherman deployment (#18062))

Merged via the queue into next with commit d04540d Oct 31, 2025
14 checks passed
@alexghr alexghr deleted the ag/slim-fisherman-deployments branch October 31, 2025 14:01
@AztecBot
Copy link
Collaborator

❌ Failed to cherry-pick to v2 due to conflicts. Please backport manually.

ludamad pushed a commit that referenced this pull request Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants