feat: replace TryCP with Nomad for running scenarios #136
Conversation
cdunster
force-pushed
the
132-replace-trycp-with-nomad-for-running-scenarios
branch
from
6207ecb to
412fb29
Compare
| RUST_LOG = "info" | ||
| } | ||
| config { | ||
| command = abspath("result/bin/app_install") |
There was a problem hiding this comment.
Will there be a way to have this reference a common path locally and remotely so that we can use the same scripts in dev as on a real environment?
There was a problem hiding this comment.
I'm hoping to allow this but I'm not yet sure what is the best way. I think at the very least it should be possible to provide either a path or a URL and if it's a URL then the binary gets downloaded as an artefact before being run.
| "--agents", "2", | ||
| "--behaviour", "minimal:1", | ||
| "--behaviour", "large:1", | ||
| "--duration", "5", |
There was a problem hiding this comment.
Should this default to a longer time when you're ready to merge this?
There was a problem hiding this comment.
My idea (already in the latest version) is that if no duration is provided then it is not actually set and so it will use the default provided by the scenario. This could be an issue for neverending scenarios but I think it makes the most sense and we can always kill a running scenario at set the duration if we miss it by accident.
This allows us to run all scenarios as Nomad jobs using the same job spec.
Closes #132