Skip to content
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

Sync monitor timer #25

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Sync monitor timer #25

wants to merge 11 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Jun 11, 2019

No description provided.

Jared Null added 10 commits March 10, 2019 16:55
Current example scp command won't work without :~ at the end.
changed `linux-image-extra-` to `linux-modules-extra-`
`iproute` to `iproute2`
--force is the same as: --allow-downgrades --allow-remove-essential --allow-change-held-packages
 iproute package installed twice and deprecated for iproute2
easy_install not included as an install package
easy_install has been removed from python-setuptools
@bennlich
Copy link
Collaborator

Woot!!! Awesome!

@bennlich
Copy link
Collaborator

This seems good to me. Have you tried creating a new exitnode with the script from this branch and tested to see if it worked?

I think it would be sufficient to test that your new script is getting called every two minutes like you expect.

P.s. One thing you could do to be a little fancy would be to curl the bash script from the monitor github repo instead of hardcoding it into this repo. E.g. https://raw.githubusercontent.com/sudomesh/monitor/master/post-routing-table.sh.

@ghost
Copy link
Author

ghost commented Jun 12, 2019

This seems good to me. Have you tried creating a new exitnode with the script from this branch and tested to see if it worked?

I think it would be sufficient to test that your new script is getting called every two minutes like you expect.

P.s. One thing you could do to be a little fancy would be to curl the bash script from the monitor github repo instead of hardcoding it into this repo. E.g. https://raw.githubusercontent.com/sudomesh/monitor/master/post-routing-table.sh.

This is the exact same script, timer, and service files on the HE exitnode, the only manual commands that needed to be run were:
systemctl enable exitnode-sync.timer and systemctl start exitnode-sync.timer which i dropped in the create_exitnode.sh script next to the start and enable babeld timer & service.

I would like to try this on a pre-production environment, but that would have to create an entire exitnode and also the monitor separately. The quick way to test this is add a test exitnode ip to exitnodeIPs.js on peoplesopen.net can i have access to that? also one of the HE routes is malformed and needs to be removed manually.

I would not rely on external services for the same reason that retrieve_ip might fail from certificate or resource availability problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant