Skip to content

add typesense-server service#1015

Draft
ufundo wants to merge 1 commit intocivicrm:masterfrom
ufundo:add-typesense
Draft

add typesense-server service#1015
ufundo wants to merge 1 commit intocivicrm:masterfrom
ufundo:add-typesense

Conversation

@ufundo
Copy link
Contributor

@ufundo ufundo commented Feb 3, 2026

Enables adding Typesense server service for use with https://lab.civicrm.org/extensions/typesense/

I wasn't sure if there is a good way to do this in an "optional" way? E.g. add a new separate profile that extends the existing profile? Or add a variable flag somehow?

Likewise wasn't sure how best to handle TYPESENSE_API_KEY variable. I suppose a dummy value is no less secure than the default mysql root password, so maybe it is fine as is?

@github-actions
Copy link

github-actions bot commented Mar 6, 2026

This pull request has reached 30 days with no activity, it has been marked as stale.

@ufundo
Copy link
Contributor Author

ufundo commented Mar 14, 2026

Ping @totten

@ufundo ufundo reopened this Mar 14, 2026
@totten
Copy link
Member

totten commented Mar 14, 2026

@ufundo I'm OK with this; it is opt-in. I haven't thought critically about TYPESENSE_API_KEY, but I think we can defer that question initially. It's fine for experimental/validation period.

One thing - when pulling in the package from dists.default (aka nixpkgs 25.05), I get a problem on macOS:

these 2 derivations will be built:
  /nix/store/wly8ks10sbm757901r7vb22lzp7r4wgm-typesense-server-28.0-darwin-arm64.tar.gz.drv
  /nix/store/567m3m1x5q0y9my6qil6fhn5jbs99ps4-typesense-28.0.drv
building '/nix/store/wly8ks10sbm757901r7vb22lzp7r4wgm-typesense-server-28.0-darwin-arm64.tar.gz.drv'...

trying https://dl.typesense.org/releases/28.0/typesense-server-28.0-darwin-arm64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 36.0M  100 36.0M    0     0  45.2M      0 --:--:-- --:--:-- --:--:-- 45.1M
error: hash mismatch in fixed-output derivation '/nix/store/wly8ks10sbm757901r7vb22lzp7r4wgm-typesense-server-28.0-darwin-arm64.tar.gz.drv':
         specified: sha256-q+uOfNsyao8r2wea49vX8hJWWQYZdOfk7pB0e531FDA=
            got:    sha256-kOjO3kokmvaXr55wcQKUboYzzTSaF9Ye+tY/A4hNvsY=
error: 1 dependencies of derivation '/nix/store/567m3m1x5q0y9my6qil6fhn5jbs99ps4-typesense-28.0.drv' failed to build

However, pulling it from v25.11 is OK. At some point, we will need to update dists.default, so it'll work eventually. Here are a couple quick/interim options:

  • Only load typesense on Linux. (Look two lines below -- chromium is loaded conditionally. You could put typesense there too. Leave a comment about why it's conditional.)
  • Add v25.11 and use it for typesense. (https://gist.github.com/totten/7df90a5a64063100451b62da420e05ea) (The download will be a bit heavier, but it'll get lighter when we upgrade other packages.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants