search

anza-xyz / validator-lab

Deploy and test your new Agave validator features in a kubernetes-based cluster
https://www.anza.xyz/
7 stars 4 forks source link

remove --full-rpc-api as default in startup_scripts.rs #56

Closed gregcusack closed 3 months ago

gregcusack commented 3 months ago

Removed the --full-rpc-api default for all validators in startup_script.rs.

PR https://github.com/anza-xyz/validator-lab/pull/53 set --full-rpc-api as default for only RPC node.

Added support for --full-rpc-api in startup_scripts.rs

gregcusack commented 3 months ago

so actually this will not work. we have a load balancer in front of the rpc nodes and the bootstrap. so if --full-rpc-api is not used, then the load balancer may send an rpc request to the bootstrap which doesn't have rpc enabled. so i'm going to add a fix to that as a separate PR so these can be merged back to back

gregcusack commented 3 months ago

updated so bootstrap is not behind load balancer when we run without --full-rpc-api. exit immediately if no --full-rpc-api and no rpc nodes update load balancer to serve websocket port 8900. need for client