Commit e9010614 authored by Haylin Moore's avatar Haylin Moore
Browse files

nixos/chromadb: deprecate logPath option 

The module and nixos test are currently broken because the logPath option is always set by default and it passes a parameter to the CLI that no longer exists. Lets just remove logPath all together as the parameter it relied on got removed.
parent eaca18bd

nixos/modules/services/databases/chromadb.nix

+7 −9
Original line number Diff line number Diff line
@@ -19,6 +19,12 @@ in 


  meta.maintainers = with lib.maintainers; [ ];



  imports = [

    (lib.mkRemovedOptionModule [ "services" "chromadb" "logFile" ] ''

      ChromaDB has removed the --log-path parameter that logFile relied on.

    '')

  ];



  options = {

    services.chromadb = {

      enable = mkEnableOption "ChromaDB, an open-source AI application database.";
@@ -47,14 +53,6 @@ in 
        '';

      };



      logFile = mkOption {

        type = types.path;

        default = "/var/log/chromadb/chromadb.log";

        description = ''

          Specifies the location of file for logging output.

        '';

      };



      dbpath = mkOption {

        type = types.str;

        default = "/var/lib/chromadb";
@@ -81,7 +79,7 @@ in 
        StateDirectory = "chromadb";

        WorkingDirectory = "/var/lib/chromadb";

        LogsDirectory = "chromadb";

        ExecStart = "${lib.getExe cfg.package} run --path ${cfg.dbpath} --host ${cfg.host} --port ${toString cfg.port} --log-path ${cfg.logFile}";

        ExecStart = "${lib.getExe cfg.package} run --path ${cfg.dbpath} --host ${cfg.host} --port ${toString cfg.port}";

        Restart = "on-failure";

        ProtectHome = true;

        ProtectSystem = "strict";