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

Move task event message logic into the API #3399

Closed
DingoEatingFuzz opened this issue Oct 16, 2017 · 1 comment · Fixed by #3486
Closed

Move task event message logic into the API #3399

DingoEatingFuzz opened this issue Oct 16, 2017 · 1 comment · Fixed by #3486
Labels
theme/api HTTP API and SDK issues theme/cleanup

Comments

@DingoEatingFuzz
Copy link
Contributor

Currently the alloc-status command has a 100+ line decision tree for determining what message to show given the shape and state of a task event.

The UI duplicates this logic for the allocation detail page.

alloc-status source
Nomad UI source

This should be moved up into the API so the two implementations won't drift and so other API consumers can benefit from simple message property.

@github-actions
Copy link

github-actions bot commented Dec 6, 2022

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.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
theme/api HTTP API and SDK issues theme/cleanup
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant