simpler data model, not sure it's desirable but at least it's consistent

This commit is contained in:
Kiara Grouwstra 2025-08-26 17:31:53 +02:00
parent cce6dc9a75
commit ca51b2377a
Signed by: kiara
SSH key fingerprint: SHA256:COspvLoLJ5WC5rFb9ZDe5urVCkK4LJZOsjfF4duRJFU
3 changed files with 10 additions and 11 deletions

View file

@ -66,9 +66,7 @@ let
requests: requests:
let let
# Filter out requests that need wheel if policy doesn't allow it # Filter out requests that need wheel if policy doesn't allow it
validRequests = lib.filterAttrs ( validRequests = lib.filterAttrs (_name: req: !req.login-shell.wheel || config.wheel) requests;
_name: req: !req.login-shell.wheel || config.wheel
) requests.resources;
in in
lib.optionalAttrs (validRequests != { }) { lib.optionalAttrs (validRequests != { }) {
${config.username} = { ${config.username} = {
@ -94,7 +92,7 @@ let
implementation = cfg: { implementation = cfg: {
input = cfg; input = cfg;
output = lib.optionalAttrs cfg.enable { output = lib.optionalAttrs cfg.enable {
resources.hello.login-shell.packages.hello = pkgs.hello; "my".login-shell.packages.hello = pkgs.hello;
}; };
}; };
}; };
@ -125,9 +123,11 @@ let
else else
null; null;
users.users = environment.config.resources."operator-environment".login-shell.apply { users.users = environment.config.resources."operator-environment".login-shell.apply (
resources = lib.filterAttrs (_name: value: value ? login-shell) requests; lib.filterAttrs (_name: value: value ? login-shell) (
}; lib.concatMapAttrs (k': lib.mapAttrs' (k: v: lib.nameValuePair "${k'}.${k}" v)) requests
)
);
}; };
}; };
}; };

View file

@ -73,9 +73,7 @@ in
requests: requests:
let let
# Filter out requests that need wheel if policy doesn't allow it # Filter out requests that need wheel if policy doesn't allow it
validRequests = lib.filterAttrs ( validRequests = lib.filterAttrs (_name: req: !req.login-shell.wheel || config.wheel) requests;
_name: req: !req.login-shell.wheel || config.wheel
) requests.resources;
in in
lib.optionalAttrs (validRequests != { }) { lib.optionalAttrs (validRequests != { }) {
${config.username} = { ${config.username} = {
@ -158,7 +156,7 @@ in
resources = resources =
fediversity.applications.hello.resources fediversity.applications.hello.resources
fediversity."example-configuration".applications.hello; fediversity."example-configuration".applications.hello;
hello-shell = resources.resources.hello.login-shell; hello-shell = resources."my".login-shell;
environment = fediversity.environments.single-nixos-vm.resources."operator-environment".login-shell; environment = fediversity.environments.single-nixos-vm.resources."operator-environment".login-shell;
result = mkDeployment { result = mkDeployment {
modules = [ modules = [

View file

@ -190,6 +190,7 @@ in
type = types.optionType; type = types.optionType;
}; };
# TODO(@fricklerhandwerk): we may want to make the function type explicit here: `application-resources -> resource-type` # TODO(@fricklerhandwerk): we may want to make the function type explicit here: `application-resources -> resource-type`
# and then also rename this to be consistent with the application's resource mapping
options.apply = mkOption { options.apply = mkOption {
description = "Apply the policy to a request"; description = "Apply the policy to a request";
type = functionTo policy.config.resource-type; type = functionTo policy.config.resource-type;