-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Flag to wait till deployment actually succeeded #4606
Comments
a work-around for now is to use jrasell/levant |
@jippi thanks! seen it before, never had a chance to use it yet. In my opinion it's not something that should be external to nomad, since it is already aware of the state of deployed services. Anyway, gonna try levant for now, leaving issue open in case there would be some update whenever feature may land on roadmap for some future release. |
Yes, I'm pretty sure Nomad will eventually support this - should be pretty trivial to add - just wanted to unblock you until Nomad got native support :) |
@jippi in that you've succeeded =) Thank again |
Well, levant seems to be completely broken, deploying nomad job with broken on purpose healthcheck results in success, so hope this issue will see some attention someday. |
@yellowmegaman feel free to raise an issue against Levant and I can take a look into it. We use it extensively and successfully in my current job so I would be curious what the issue is. |
@yellowmegaman I do think it is a good feature and would want to add it. @jrasell PRs welcome 😄 |
@dadgar that is super-awesome =) |
@yellowmegaman : FYI, i'm adding this feature to the Nomad TF provider: |
Looks like levant is abandoned by owner. Thus the flag in Nomad would be super useful for us too. |
Is there any news here? :) |
No hcl2 support is a blocker to use levant. A Not seeing allocation and deployments details as well as process stdout/err on a |
we want to see this feature in nomad cli. So that we can make ci-cd via nomad cli to get realtime deployment in logs |
This issue is a duplicate of #6818, which is on the roadmap. I'm going to close this issue in lieu of #6818 only because it has more of the implementation details on how we want to proceed on this. Keep an eye there for future progress (also: our Product folks do look at emoji reactions to issues, so feel free to add them there!) |
I'm going to lock this issue because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active issues. |
Hello there! Question here.
It would be really great if i could run
nomad run
and it won't release terminal till deployment is not only scheduled, but also completed successfully.Use case:
This approach is sub-optimal since it relies on some scripting which can be faulty and may have some corner cases.
Can we add flag to
nomad run
so it won't throw0
exit code till deployment is finished?Same would be awesome for nomad provider for terraform.
Thanks for attention in advance!
The text was updated successfully, but these errors were encountered: