kanboard: init at 1.2.42

[yzx9]

KanBoard is project management software that focuses on the Kanban methodology.

Previously, Kanboard was added but later removed due to its limited utility. This PR reinstates the package and introduces a NixOS module to facilitate hosting the web application.

close #288788

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[yzx9]

Thanks to @getchoo for the detailed and thoughtful review. I've updated the code according to your suggestions. Please review it again, and feel free to share any further suggestions

[getchoo]

If I have the time I'll try to actually test this module out locally -- but for now, this looks great!

The only other recommendation I would have is creating a VM test for this module, even if it's as simple as pinging the service once it's up. It's a great way to make sure things are working at a basic level, and helps a ton with updating the module and the package itself. Ping me again if you need any help with it :)

[yzx9]

The test has been added, and some default values have been changed to make the service easier to set up.

[yzx9]

@getchoo I have updated this PR. Please review it again. I’ve pushed a commit to facilitate the review, and I will squash it after approval. I hope I haven't misunderstood you this time.

[getchoo]

Everything else here looks good! 👍

[yzx9]

Thank for you detailed review.

1. I have changed lib.optionalAttrs to lib.mkIf.
2. I’ve made the root option force, similar to anuko-time-tracker how handles it,

nixpkgs/nixos/modules/services/web-apps/anuko-time-tracker.nix

Lines 322 to 337 in 7422541

	virtualHosts."${cfg.hostname}" = lib.mkMerge [
	cfg.nginx
	{
	root = lib.mkForce "${package}";
	locations = {
	"/".index = "index.php";
	"~ [^/]\\.php(/|$)" = {
	extraConfig = ''
	fastcgi_split_path_info ^(.+?\.php)(/.*)$;
	fastcgi_pass unix:${config.services.phpfpm.pools.anuko-time-tracker.socket};
	'';
	};
	};
	}
	];
	};

which resulted in an error:

The option `nodes.machine.services.nginx.virtualHosts.kanboard.root` is defined both null and not null,

[getchoo]

Module and package LGTM, OfBorg is happy, and the root option of the virtualHost appears to be working as expected with

{
  services.kanboard = {
    enable = true;
    domain = "example.org";
  };
  
  services.nginx.virtualHosts."example.org".root = "/foo/bar";
}

Thanks for all your work here! Feel free to ping me again in a couple days if there are no other reviews :)

[yzx9]

I have squashed the commits

[yzx9]

Hello @getchoo, could you take another look at this PR? I haven’t made any changes in the latest squashed commit.

[getchoo]

Thanks again for your work here!

[getchoo]

@ofborg test kanboard

[getchoo]

@ofborg build kanboard kanboard.passthru.tests

Hopefully last build 🤞

[getchoo]

Ok, now we can merge :p