Skip to content

Commit

Permalink
fix(lib/ops): tag defaulting while forwarding mkStandardOCI to mkOCI
Browse files Browse the repository at this point in the history
  • Loading branch information
blaggacao committed Jul 13, 2023
1 parent 74e13ed commit 301a649
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 12 deletions.
13 changes: 7 additions & 6 deletions src/lib/ops/mkStandardOCI.nix
Original file line number Diff line number Diff line change
Expand Up @@ -106,14 +106,15 @@ in
'';
in
with dmerge;
l.throwIf (args ? tag && meta ? tags)
"mkStandardOCI: use of `tag` and `meta.tags` arguments are not supported together. Remove the former."
cell.ops.mkOCI (
merge
{
inherit name tag uid gid labels options perms config meta setup runtimeInputs;
entrypoint = operable';
}
({
inherit name uid gid labels options perms config meta setup runtimeInputs;
entrypoint = operable';
}
# keep this optional so that mkOCI can likewise
# match undefined on args ? tag
// l.optionalAttrs (args ? tag) {inherit tag;})
{
# mkStandardOCI differentiators over mkOCI
# - live & readiness probes
Expand Down
6 changes: 3 additions & 3 deletions src/local/flake.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions src/tests/flake.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 301a649

Please sign in to comment.