-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
nixos/cloudflare-warp: init #168092
nixos/cloudflare-warp: init #168092
Conversation
6ca7448
to
6f81fe2
Compare
a039c07
to
fd58cf1
Compare
Does enabling this service change your iptables rules and would it impact tailscale exit node configuration? |
@CMCDragonkai This is a good question. I can check this later and provide a response, at least regarding iptables. |
Did you find out? |
Sorry for the late response, I was able to check the iptables and there were no changes. For the other one I wasn't able to check out. |
I just tried running warp on NixOS (using the package, not the module) and I see this a lot in the
Does this indicate that it's trying to modify iptables rules and failing? And by extension why you saw no iptables changes? EDIT: Using strace I've found that warp-svc tries to execute |
I would believe so, or I might be running the wrong command to see the rules. Let me try to fix that. I haven't seen that error on the logs though, so I'm going to try to dedicate some time further to check this. Will set this as a draft. |
Note that I believe NixOS uses a compatibility of iptables commands that actually interact with the underlying nftables system. |
fd58cf1
to
6a67339
Compare
Since it does automatic iptables/nftables manipulation, it would be important to check if it create rules that are compatible with the existing nixos firewall. Tailscale does it well and no conflicts. |
I still need to figure out the issue pointed by @poita66. I have added UPDATE: Effectively, |
And yeah, by using
|
6a67339
to
1c914dd
Compare
db6291e
to
9c05f0a
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
9c05f0a
to
ba0e1d3
Compare
Fair enough. Can you please throw a review @psibi? |
@WolfangAukang I tested it, it doesn't seem to work. Sorry for leading you to wrong path. Looking into cloudflare forum, it seems to be the similar case for other Distros: https://community.cloudflare.com/t/disable-warp-svc-debug-logs/382246 I think we should just undo this commit and get this PR merged as this is usable in the current state and is on par with what's available in other distributions. |
3be944c
to
6af0893
Compare
No worries at all @psibi. Somehow the complaint by @crinklywrappr is pretty much common between WARP users, and Cloudflare hasn't provided any response (at least from what I have searched so far) on how to control the log level, leading to bloated logs (as indicated here). |
I was tagged. Hey @WolfangAukang any thoughts on my PR? WolfangAukang#1 Ooo... it has conflicts now. I don't really have time to deal with them atm. |
@crinklywrappr I didn't see that PR was pointing to my fork, my bad. Going to check it. |
6af0893
to
7b458b1
Compare
4e9a269
to
56ebe23
Compare
56ebe23
to
31056db
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/cant-start-cloudflare-warp-cli/23267/2 |
As there hasn't been any checks on this PR, I moved it to my personal repo and will close this. |
@WolfangAukang can you share instructions on how we can use it from your personal repo? Thanks! |
Add my repo to your flake inputs (remember it must be in the |
I've just tried the service module from @WolfangAukang's personal repo and everything worked well. Just had to manually download the cloudflare root certificate and specify the path in the config. Not sure why this PR wasn't merged. |
It mostly works well, but I get an error about Warp being unable to overwrite the DNS servers:
|
I had the same error as @tshakah. Could there be a missing write permission? |
I run |
Thanks, @tshakah! It worked to me. |
This has stopped working for me (I think as of the 23.05 release) |
it's certainly working for me on 23.05 (I'm just using |
Description of changes
Creating module based on #148395
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes