-
Notifications
You must be signed in to change notification settings - Fork 135
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
Add support for DSM 7.0 #71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really good work! Thank you a lot for contributing this 🎉. I don't have a device that I want to test this on, so I'll take your word that it works on DSM 7.
Before merging I'd like to recompile all packages and try the DSM 6 variant with the new scripts on my own NAS. It'll likely be a week or two before I get the opportunity to do so. If you haven't heard anything from me after that, feel free to bug me here 😅.
Hi there, I tried building the package for my 918+ (which is running DSM 7.0 Beta), using |
@quexten After examining the logs, I find that the package was built using the master branch from pkgscripts-ng. Most likely you cloned the synology-wireguard repository incorrectly and continue to use the version for DSM 6. $ git clone https://github.com/runfalk/synology-wireguard
$ cd synology-wireguard
$ git fetch origin pull/71/head:DSM7.0
$ git checkout DSM7.0
$ sudo docker build -t synobuild .
$ sudo docker run --rm --privileged --env PACKAGE_ARCH=apollolake --env DSM_VER=7.0 -v $(pwd)/artifacts:/result_spk synobuild Parse argument result for DSM 7.0:
Your parse argument result:
|
@matige Okay so the problem seems to have been that while I did start the build in the DSM7.0 branch, I didn't build the "synobuild" docker image in the DSM7.0 branch, instead I built it in the master. After re-building the synobuild docker image on the DSM7.0 branch and then running the build, the package now installs and runs correctly. |
I am getting the same error as @quexten "Invalid file format, please contact the package developer". I have switched to the DSM7.0 branch and built the synobuild container... I am compiling for a DS1019+ (apollolake)
here is my parse argument result:
|
It seems that you're using the 6.2 SDK. This should only happen if you haven't rebuilt the Docker image. Try deleting it or build using a different name than |
I tried rebuilding synobuild image with different name and it still doesn't work. Get the same error message: "Invalid file format, please contact the package developer" My parse arguement result is the same as @matige's above... it also has sdk_ver set to 6.2. |
@jaromirrivera Yeah sdk_ver was also 6.2 on the build that works on my DSM 7 Beta 918+. Try cleaning your docker images, and the build directory you have mounted ( |
@jaromirrivera Will you share the resulting SPK file with us? Maybe I can figure out what is causing the problems. |
Sure @matige, attached is the SPK I built compressed in a zip file. Also of note my DSM 7 version is DSM 7.0-41222 |
@jaromirrivera The package appears to be built correctly. The directory structure and metadata files are correct. Unfortunately, but I can't tell you why you can't install this package. I also don't have the Apollokale platform, so I can't check anything else. |
Ha, I found out what the issue was... I previously had the Wireguard spk installed for DSM 6.2, then upgraded to DSM 7.0 beta. The wireguard package was still "installed" but would not run. I needed to uninstall the DSM 6.2 version of the WireGuard package before installing the DSM 7.0 one I compiled. @runfalk That might be something to note for upgraders... Backup wireguard configs, uninstall any previous versions of wireguard and then install DSM 7 version. |
Yeah, definitely something that we should list along the release. |
@nc88keyz The problem is most likely due to an mistake in the comment with the package build instructions. You should run |
https://www.dropbox.com/s/z5ae2sjej32rxfl/WireGuard-bromolow-1.0.20210219.zip Edit: 03/21/2021 Works with binhex/arch-qbittorrentvpn:4.3.3-1-02 ( Last version to support the Kernel for the DS3615XS) I was having issues because of the kernel support ending in these containers from the devs. DS3615XS |
I tried this package but it fails to runa dn wants to "repair". Unsure why. Also did have the 6.2 package and upgraded but removed it |
You need to follow the instructions you can access through this link. Basically, don't autostart initially and do it manually the first time using |
@jobhax25 The solution can be found in #63. Follow this instruction, pay attention to step 4. |
The implementation of Is64BitPlatform in pkgscripts-ng has been improved and the workaround is no longer needed.
Sorry to clutter this thread, but I can't find a way to contact @nc88keyz. I see you have binhex/arch-qbittorrentvpn working with this WireGuard implementation. Can you share how? You may e-mail me, it's the last e-mail in this Base64 encoded string. I've had no luck with |
Just a FYI, PR #71 compiles and runs fine on DSM 7 on a DS216j (armada38x). Works very well with excellent performance. Thank you Runfalk and Matige. |
I compiled the SPK for geminilake (DS220+) but I am getting an error when starting the tunnel:
I am trying to use the NAS as a client. My wg0.conf is
Any idea how to fix this? |
Hey all! What's the status of this PR? |
I try to compile (on a DS720 DSM 7.0 machine) a DSM7.0 package for a DS216+II braswell architecture and get the following error: sudo docker run --rm --privileged --env PACKAGE_ARCH=braswell --env DSM_VER=7.0 -v WireGuard version: 1.0.20210606 Cloning into 'pkgscripts-ng'...
|
I just built a spk for a DS920+ on geminilake. |
Hi. Any updates on the status of this PR and when we might see it release? |
I've been putting it off since I haven't touched my Synology since this PR was created. It does seem a lot of people are having success with so I'm just going to merge it. I apologize for the huge delay, and a big thanks to @matige for implementing it. |
Things are merged, but I haven't built any packages yet. |
@runfalk Hi, |
Thank you for building this! Disappointing that there is no official support for wireguard on synology. |
Thanks, installation worked for me! (DS 720+) You have to run "sudo /var/packages/WireGuard/scripts/start" to start the package |
@matige @runfalk
|
docker: Error response from daemon: Bind mount failed: '/volume1/downloads/synology-wireguard/artifacts' does not exists. |
@oschmidteu Yes I know, but why? |
sudo docker run ... -v $(pwd)/artifacts:/result_spk synobuild You are trying to mount a folder which does not exist... |
Nevermind i figured it out... I was attempting to do this on my synology NAS... |
well that didn't get me anywhere either, the built .spk fails to run on my DS713+. |
Well you should read and follow the installation guide. Your problem was already discussed a few times, you could try to use the search function. |
Compile for cedarview - https://cloud.mail.ru/public/13QD/cstYBiMby |
@nohnaimer This build had the exact same effect as my build, I can install and attempt to run it but it fails to start according to synology logs. |
@cchhat01 compilation of the package should be done outside the DSM environment. Cross compilation is used, so the fact that the compilation is done on an amd64 platform is not a problem. |
@matige holy crap that worked (and my apologies for overlooking that step)... |
Maybe... I don't know... ping the other endpoint? :D And RTFM ;) |
Looks like I have a DNS entry in my
Is there anything that I can do to resolve this (pun intended)? I searched for solutions but most of the solutions are for non-synology users. |
How about remove the DNS entry from wg0.conf? |
Would that not leak DNS ? |
Explain like I’m dumb. Does this just mean you need to compile it on a different PC than the synology itself? |
Yes. You run follow the steps in the compilation section of the README on your normal computer (not your NAS). That generates a package that can then be installed on your Synology NAS through the web UI. Don't forget to read all the instructions because it's a bit more involved on DSM 7. |
This pull request adds support for DSM 7.0. Additionally, a
wg-init
script has been developed, which allows you to easily add the WireGuard interface to autostart.This package has been tested on DS220+ (geminilake) with DSM 7.0, and Virtual DSM (kvmx64) with DSM 6.2 and 7.0.