Skip to content
This repository was archived by the owner on Jun 27, 2025. It is now read-only.

Add Levant dispatch command to dispatch against Nomad jobs.#128

Merged
jrasell merged 1 commit into
masterfrom
f_gh_113
Mar 26, 2018
Merged

Add Levant dispatch command to dispatch against Nomad jobs.#128
jrasell merged 1 commit into
masterfrom
f_gh_113

Conversation

@jrasell

@jrasell jrasell commented Mar 25, 2018

Copy link
Copy Markdown
Member

Levant now has the functionality to dispatch against Nomad jobs
using the dispatch command. Jobs that are dispatched go through
the job status checker and so dispatched jobs can be confirm to
reach the running state.

Closes #113

Levant now has the functionality to dispatch against Nomad jobs
using the dispatch command. Jobs that are dispatched go through
the job status checker and so dispatched jobs can be confirm to
reach the running state.

Closes #113
@jrasell

jrasell commented Mar 25, 2018

Copy link
Copy Markdown
Member Author

@sevagh I had some time to complete my started work today so raised this PR and would love any feedback you have.

@sevagh

sevagh commented Mar 26, 2018

Copy link
Copy Markdown

I'll compare this directly to my code later today, but it looks good. Mine is definitely more hacky (e.g. not supporting payloads, and passing -meta as a single comma-separated list instead of multiple flags).

I suspect the dispatch behavior is the same as nomad job dispatch? Return a successful allocation.

I'll compile your branch and run it with my arguments today as a sanity check.

@jrasell

jrasell commented Mar 26, 2018

Copy link
Copy Markdown
Member Author

The run behaviour is identical; but it will go through the additional checking that Levant provides and so will ensure that the dispatched job reaches the running state as well as all the allocations that are part of it.

@stevenscg

Copy link
Copy Markdown

I use the heck out of dispatch jobs. Nice work!

@sevagh

sevagh commented Mar 26, 2018

Copy link
Copy Markdown

Works nice for me.

I still rely on code from here (-monitor and -timeout) to poll Nomad and check for the end result of the dispatched job.

@sevagh

sevagh commented Mar 26, 2018

Copy link
Copy Markdown

Once that other PR is merged independently, extending both deploy and dispatch to support the blocking behavior should be entirely possible. I didn't mean to imply that this PR is unusable without the other one.

@jrasell jrasell merged commit 40ac877 into master Mar 26, 2018
@jrasell jrasell deleted the f_gh_113 branch March 26, 2018 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants