-
-
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: tailscale: option to disable path warning #194768
Conversation
Crap, I fixed this on the wrong branch. Thanks @LunNova |
9186edd
to
0785116
Compare
I guess I was fixing it in the right place, cherry-picking it to my usual nixpkgs and just forgot to push. Should be good now. |
@ofborg eval |
0785116
to
7da34c5
Compare
7da34c5
to
0011198
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Do we have any other options to silence warnings? |
Not that I'm aware of. It would be nice if there were some way to force them to have unique identifiers and then allow the user to opt-"in" to ignoring it the same way they can opt-"in" to ignore unfree-restrictions and use unfree packages. But, I don't think that exists. |
I was thinking the same thats why I am not that keen about adding this. |
Let me just mention that there are discussions upstream about fixing the underlying problem so that we eventually wouldn't need a warning. |
Fine with this not getting in if it's too hacky. Would a more thought out system for silencing warnings need an RFC? To solve the warning being annoying locally I could just patch it out anyway. |
Existing RFC which discusses adding a way to disable warnings, RFC 0127: |
Fixed by #201119 |
Description of changes
fixed #194623 by adding an option
services.tailscale.ignoreReversePathFilter
to quiet a warning that is relevant to only some tailscale clients/users.cc: @LunNova @danderson
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