Skip to content

Conversation

@sjagoe
Copy link
Contributor

@sjagoe sjagoe commented Oct 1, 2025

This fixes #325 by building upon the fix (#327) from @VolodiaPG for build targets, adding a fix for extra build args.

With this, deploying to multiple hosts with build args works again, e.g.

../deploy-rs/target/debug/deploy -d --targets .#host1 .#host2 .#host3 --keep-result --skip-checks -- --max-jobs 0

Copy link

@Sntx626 Sntx626 left a comment

Choose a reason for hiding this comment

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

Tested, fixes the problem, looks good.
LGTM

@neunenak
Copy link

I ended up pushing a PR to fix this #351 without checking to see if that had been done already. @Sntx626 what needs to happen for this fix to be mergeable?

@anna-oake
Copy link

Hey @rvem could you take a look at this long awaited fix please? 🙏
There's also #333 which is slightly different

@rvem
Copy link
Member

rvem commented Dec 18, 2025

LGTM, thanks for the contribution.
I apologize for not responding quickly.

@rvem rvem merged commit d5eff7f into serokell:master Dec 18, 2025
magic0whi added a commit to magic0whi/nixos_configs_flake that referenced this pull request Jan 22, 2026
- Update deployment instructions to use shell expansion for targets (e.g., `Proteus-NixOS-{1..6}`) and single `--targets` flag, leveraging fixes in `deploy-rs` (PR #341).
- Add `-- --show-trace --verbose` flags to the deployment example.

ref: serokell/deploy-rs#341

Signed-off-by: Proteus Qian <sudaku233@outlook.com>
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.

a6b0cb2c5e1c1efea2d48fdf7bb5b36b6675e40e broke --targets

6 participants