Skip to content

Commit

Permalink
Fix nixops check
Browse files Browse the repository at this point in the history
  • Loading branch information
roberth committed Aug 26, 2021
1 parent c2d3e6b commit 00eb6e3
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion nixops/script_defs.py
Original file line number Diff line number Diff line change
Expand Up @@ -493,7 +493,11 @@ def check(depl: nixops.deployment.Deployment) -> None:
else:
resources.append(m)

with one_or_all(args, writable=False, activityDescription="nixops check") as depls:
# TODO: writable=False?
# Historically, nixops check was allowed to write to the state file.
# With remote state however, this requires an exclusive lock, which may
# not be the best choice.
with one_or_all(args, writable=True, activityDescription="nixops check") as depls:
for depl in depls:
check(depl)

Expand Down

0 comments on commit 00eb6e3

Please sign in to comment.