abracadaniel / cardano-pool-docker

Docker container for setting up and running a Cardano Stake Pool
164 stars 62 forks source link

Register multiple relays by hostname #67

Closed robinboening closed 11 months ago

robinboening commented 1 year ago

This PR adds the ability to register multiple relays when passing PUBLIC_RELAY_HOSTS with a comma separated list to the container, e.g. -e PUBLIC_RELAY_HOSTS=relay1.mypool.net:3001,relay2.mypool.net:3001

Be aware PUBLIC_RELAY_HOST changed to PUBLIC_RELAY_HOSTS.

Please note this PR does not change the behaviour of registering a relay using it's IP address. Hence, still only one single relay can be registered using PUBLIC_RELAY_IP.

robinboening commented 1 year ago

@abracadaniel Sorry for the confusion, but I noticed you committed the commit of this PR to master. I just added another commit on top fixing an issue as I used a wrong test operator.

Could you also cherry-pick commit 9101a5db2869647195b6d0c7e1e58da1e8ee9bfa as otherwise it'll cause annoying issues.

Thank you!

abracadaniel commented 11 months ago

I think it is as it should be. Could you please check? 9101a5d is merged as well.