add service file for pigeond #36
Conversation
|
Do not merge yet please, working on testing it. Will update here once done |
There was a problem hiding this comment.
@aditya-manit Will you update without the sudo on the commands? We are tending to lean against sudo in the system commands themselves.
|
Okay sure, however by default permissions I believe sudo is required to restart systemctl services |
|
I think I was successfully able to run it as service However my node is still syncing, so as per my node, my address doesn't have any balance yet, so I am getting the below error It might take couple of more hours for my node to sync, so in case we wanna merge it asap, I'll recommend someone else to try setup it to verify, however I believe it's fine and will work when my node is synced. I'll update when its synced |
README.md
Outdated
|
|
||
| #### Using systemd service | ||
|
|
||
| ```bash |
There was a problem hiding this comment.
you mean shell script ? I don't readme highlight just with shell
|
@aditya-manit You don't need to make those changes, I can do a followup PR with my particulars. Overall readme changes look probably gtg. |
|
Done with the changes |
| After=network-online.target | ||
|
|
||
| [Service] | ||
| Environment="PALOMA_KEYRING_PASS=$PALOMA_KEYRING_PASS" |
There was a problem hiding this comment.
Does this assume that the person running this command has $PALOMA_KEYRING_PASS and other env variables set when running this command? (@aditya-manit )
There was a problem hiding this comment.
Yes, we are setting it here before running that command. Line 103 to line 110
| Create service file | ||
|
|
||
| ```shell script | ||
| tee <<EOF >/dev/null /etc/systemd/system/pigeond.service |
There was a problem hiding this comment.
@aditya-manit this should be pigeon, not pigeond
There was a problem hiding this comment.
Usually I have seen a d suffix in binaries and services too, d for daemon I guess. We can remove it if we want to - nothing major.
There was a problem hiding this comment.
the prefix d is a cosmos thing where they add the d at the end of a chain executable
| Environment="PALOMA_KEYRING_PASS=$PALOMA_KEYRING_PASS" | ||
| Environment="ETH_RPC_URL=$ETH_RPC_URL" | ||
| Environment="ETH_PASSWORD=$ETH_PASSWORD" | ||
| Environment="ETH_SIGNING_KEY=$ETH_SIGNING_KEY" |
There was a problem hiding this comment.
Given the default env variables that we are exporting and based on the config file in this readme, you need to add VALIDATOR environment variable as well.
palomachain/paloma#311
Background
Add service file for pigeond
Testing completed