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

WIP: Update Cloud VPN Tunnel #2

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

WIP: Update Cloud VPN Tunnel #2

wants to merge 4 commits into from

Conversation

jeffmccune
Copy link
Member

This patch addes a scarab update command to update the Cloud VPN tunnel with a newly obtained IP address.

Without this patch the configuration file doesn't work well with an
EdgeRouter.  This patch reads the configuration values from the
/config/scarab directory which persists across reboots.
 * /config is used because it persists across firmware upgrades.
 * /config/scarab is used to contain all scarab files in one directory.

A wrapper script at /config/scarab/bin/scarab configures only the
application default credentials.  All other configuration should be
handled by the configuration file at /config/scarab/scarab.yaml to keep
things clear and in one place.
@jeffmccune jeffmccune force-pushed the update branch 3 times, most recently from 86232e3 to 7bfd8d9 Compare January 1, 2020 01:22
Create the ExternalVPNGateway
==

This patch creates the ExternalVPNGateway resource which is required to
create the VPN Tunnel.  The ExternalVPNGateway references the dynamic IP
address of the peer.

Creation of the ExternalVPNGateway is an asynchronous Operation.  This
patch implements waitDone() using go routines and channels to poll for
the completion of the operation using [globalOperations.get][get].
[globalOperations.wait][wait] is a future optimization.

[get]: https://cloud.google.com/compute/docs/reference/rest/v1/globalOperations/get
[wait]: https://cloud.google.com/compute/docs/reference/rest/v1/globalOperations/wait
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