Skip to content

Comments

Allow building content-addressed derivations with hydra, minimally#1349

Merged
Ericson2314 merged 4 commits intomasterfrom
ca-no-new-col
Jan 26, 2024
Merged

Allow building content-addressed derivations with hydra, minimally#1349
Ericson2314 merged 4 commits intomasterfrom
ca-no-new-col

Conversation

@Ericson2314
Copy link
Member

This verison has a worse UI, but also chnages the schema less: One non-null constraint is removed, but no new columns are added.

This verison has a worse UI, but also chnages the schema less: One
non-null constraint is removed, but no new columns are added.

Co-Authored-By: Andrea Ciceri <[email protected]>
Co-Authored-By: regnat <[email protected]>
Ericson2314 added a commit to ngi-nix/ngi0-infra that referenced this pull request Jan 26, 2024
Flake lock file updates:

• Updated input 'hydra':
    'github:thufschmitt/hydra/002a77a954259ddb49fd262607ba44fb2c3b34a9' (2024-01-24)
  → 'github:nixos/hydra/323b556dc8c348f1f9d5bf5b8a35f2617bfe600c' (2024-01-26)
• Updated input 'hydra/nix':
    'github:NixOS/nix/b38e5a665e9d0aa7975beb0ed12e42d13a392e74' (2023-12-13)
  → 'github:NixOS/nix/2c4bb93ba5a97e7078896ebc36385ce172960e4e' (2024-01-25)
• Added input 'hydra/nixpkgs-for-fileset':
    'github:NixOS/nixpkgs/a77ab169a83a4175169d78684ddd2e54486ac651' (2024-01-24)
@Mindavi
Copy link
Contributor

Mindavi commented Jan 26, 2024

What exactly is meant by worse UI?

@Ericson2314
Copy link
Member Author

@Mindavi The other one used the new contentAddressed column to show more info. This one, you cannot tell the difference after an output is built and the path column is filled in.

@Ericson2314 Ericson2314 changed the title Minimal CA support Allow building content-addressed derivations with hydra, minimally Jan 26, 2024
@Ericson2314
Copy link
Member Author

This PR is now basically just the Schema change. I added a migration and tested it with https://hydra.ngi0.nixos.org/ and the migration worked just fine. I think we are all set!

@Ericson2314 Ericson2314 merged commit 838648c into master Jan 26, 2024
@Ericson2314 Ericson2314 deleted the ca-no-new-col branch January 26, 2024 22:54
@Ericson2314 Ericson2314 mentioned this pull request Jan 26, 2024
20 tasks
lorenzleutgeb pushed a commit to ngi-nix/ngipkgs that referenced this pull request Apr 26, 2024
Flake lock file updates:

• Updated input 'hydra':
    'github:thufschmitt/hydra/002a77a954259ddb49fd262607ba44fb2c3b34a9' (2024-01-24)
  → 'github:nixos/hydra/323b556dc8c348f1f9d5bf5b8a35f2617bfe600c' (2024-01-26)
• Updated input 'hydra/nix':
    'github:NixOS/nix/b38e5a665e9d0aa7975beb0ed12e42d13a392e74' (2023-12-13)
  → 'github:NixOS/nix/2c4bb93ba5a97e7078896ebc36385ce172960e4e' (2024-01-25)
• Added input 'hydra/nixpkgs-for-fileset':
    'github:NixOS/nixpkgs/a77ab169a83a4175169d78684ddd2e54486ac651' (2024-01-24)
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