-
-
Notifications
You must be signed in to change notification settings - Fork 14.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/glpi-agent: init #361759
base: master
Are you sure you want to change the base?
nixos/glpi-agent: init #361759
Conversation
22c6bd6
to
3965250
Compare
How to work :
|
Also this need a backport to 24.11 if is possible is tested and used on 24.11 |
9c7cfdf
to
9a03b8d
Compare
8dda0d2
to
43b340c
Compare
bfb0179
to
5528c1e
Compare
f947cce
to
b82c538
Compare
37e765f
to
fb92f51
Compare
c16f6c0
to
c5b2f65
Compare
c5b2f65
to
16f3fb1
Compare
a876968
to
ee36818
Compare
ee36818
to
40757a5
Compare
40757a5
to
4549bbd
Compare
settings = lib.mkOption { | ||
type = settingsType; |
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.
Shouldn't we go RFC42-style freeformType
here ?
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.
Looks like it doesn't support arbitrary nesting?
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.
I do wonder if there's a way to reuse one of the existing writers for this though.
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.
settingsType seems more relevant to me compared to GLPI but I could be wrong.
https://glpi-agent.readthedocs.io/en/latest/configuration.html#parameter-syntax
services.glpiAgent = {
enable = true;
settings = {
server = [ "https://server.com" ];
delaytime = 3600;
lazy = false;
"scan-homedirs" = false;
"scan-profiles" = false;
html = false;
"backend-collect-timeout" = 30;
force = true;
"no-p2p" = false;
tasks = [ "inventory" ];
user = "USER";
password = "PWD";
timeout = 180;
"no-httpd" = true;
logger = [ "stderr" ];
logfacility = "LOG_USER";
color = false;
debug = 0;
};
};
Fix : #361741
This PR is module for : #361746
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.