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

Move ipsec.secrets to /config so that it can be easily preserved between rebuilds. #26

Merged
merged 1 commit into from
Sep 27, 2016
Merged

Move ipsec.secrets to /config so that it can be easily preserved between rebuilds. #26

merged 1 commit into from
Sep 27, 2016

Conversation

jimsynz
Copy link
Contributor

@jimsynz jimsynz commented Sep 21, 2016

So I pulled the latest version of this image and rebuilt my container and poof my PSK was gone and I had to reconfigure all my clients, so to save myself the trouble I symlinked it into /config and made it a volume so that it can be easily re-used.

@gaomd
Copy link
Owner

gaomd commented Sep 25, 2016

That's a very good idea, I'd like to merge it, but before that, I need to completely understand the usage so I could update the documentation as well.

So to be sure, is mounting local dir with -v local/path/to/config:/config the correct usage in order to preserve the config between rebuilds?

@jimsynz
Copy link
Contributor Author

jimsynz commented Sep 25, 2016

That's right. All it will preserve is the preshared key file. Is there any other config that also needs to be stored?
— James Harton Professional Tinkerer, Rabid 0226803869 | http://rabid.co.nz/
On Mon, Sep 26, 2016 at 06:27, Mengdi Gao notifications@github.com wrote:
That's a very good idea, I'd like to merge it, but before that, I need to completely understand the usage so I could update the documentation as well.

So to be sure, is mounting local dir with -v local/path/to/config:/config the correct usage in order to preserve the config between rebuilds?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com//pull/26#issuecomment-249434318] , or mute the thread [https://github.com/notifications/unsubscribe-auth/AADoOX4yOo08Qw672wqvAdeyUgH9ClrKks5qtq8bgaJpZM4KCUBR] .

@gaomd gaomd merged commit 39c95c2 into gaomd:master Sep 27, 2016
@gaomd
Copy link
Owner

gaomd commented Sep 27, 2016

Thanks!

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.

2 participants