Skip to content

Conversation

Conni2461
Copy link
Member

@Conni2461 Conni2461 commented Jul 31, 2025

related pr in ofborg: ofborg/ofborg-infrastructure#14

StateDirectoryMode = "0700";
ReadWritePaths = [
"/nix/var/nix/gcroots/"
"/run/postgresql/.s.PGSQL.${toString config.services.postgresql.port}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this break when postgres restarts and recreates the socket?

@Conni2461 Conni2461 force-pushed the feat/init-queue-runner-v2 branch 4 times, most recently from 5b593d9 to 4121e0e Compare July 31, 2025 16:24
@Conni2461 Conni2461 force-pushed the feat/init-queue-runner-v2 branch 2 times, most recently from b779eb8 to 222c51c Compare August 5, 2025 07:31
@Mic92
Copy link
Member

Mic92 commented Aug 5, 2025

Please run nix fmt once locally.

@Conni2461 Conni2461 force-pushed the feat/init-queue-runner-v2 branch from 222c51c to e21d6f5 Compare August 7, 2025 08:50
zlib
protobuf

nixVersions.nix_2_29
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also builds and works with 2_30, but i think you are currently still using 2_29 ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this use config.nix.package?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for hydra probably.

@Conni2461
Copy link
Member Author

Conni2461 commented Aug 7, 2025

found a small issue, with the ffi changes, will update asap.

Edit: resolved

@Conni2461 Conni2461 force-pushed the feat/init-queue-runner-v2 branch from e21d6f5 to 285a237 Compare August 7, 2025 11:04
@Mic92 Mic92 added this pull request to the merge queue Aug 9, 2025
Merged via the queue into NixOS:main with commit 5d02095 Aug 9, 2025
15 checks passed
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.

4 participants