Skip to content
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

Signal and fail deployment when no eligible nodes are found #381

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ianic
Copy link

@ianic ianic commented Nov 30, 2020

I run into this problem by writing wrong datacenter name in the nomadfile.
There where no nodes in that misspelled datacenter but the deployment
run forever waiting for the node to appear without indication of the
problem.
Nomad UI is showing the warning that there are no nodes eligible for
evaluation. But there is no such warning in levant output. So I added
the same warning as in UI.
Instead of waiting for some node to appear we are now failing deployment.

I run into this problem by writing wrong datacenter name in the nomadfile.
There where no nodes in that misspelled datacenter but the deployment
run forever waiting for the node to appear without indication of the
problem.
Nomad UI is showing the warning that there are no nodes eligible for
evaluation. But there is no such warning in levant output. So I added
the same warning as in UI.
Instead of waiting for some node to appear we are now failing deployment.
@hashicorp-cla
Copy link

hashicorp-cla commented Nov 30, 2020

CLA assistant check
All committers have signed the CLA.

Base automatically changed from master to main March 3, 2021 19:50
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.

2 participants