Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 4 commits into from
Jan 26, 2024

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
2 checks passed
@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