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

Backport of [ui] Pack metadata booleanified and added to the statuses endpoint into release/1.8.x #23644

Conversation

hc-github-team-nomad-core
Copy link
Contributor

Backport

This PR is auto-generated from #23404 to be assessed for backporting due to the inclusion of the label backport/1.8.x.

The below text is copied from the body of the original PR.


Context: Way back in Two-ought-twenty-two, we added a visual indicator for whether a job was run via Nomad Pack in the UI. We could do this because the /jobs endpoint returns job meta information, and pack consistently puts meta.pack.name etc. such that we can detect it.

With the purpose-built /statuses endpoint, we can be choosy about what we want to include and by default weren't delving into job metadata.

This PR checks to see if any of the tell-tale signs of pack are present, and if so, presents an isPack boolean to the endpoint that the UI turns into a little badge:

image


Overview of commits

@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/f-ui/pack-meta-on-statuses/formerly-brave-beetle branch from 24c512b to 8446885 Compare July 19, 2024 15:17
@vercel vercel bot temporarily deployed to Preview – nomad-ui July 19, 2024 15:20 Inactive
Copy link

Ember Test Audit comparison

release/1.8.x 8446885 change
passes 1569 1569 0
failures 2 2 0
flaky 0 0 0
duration 000ms 000ms -000ms

@philrenaud philrenaud merged commit ff13e20 into release/1.8.x Jul 19, 2024
28 of 30 checks passed
@philrenaud philrenaud deleted the backport/f-ui/pack-meta-on-statuses/formerly-brave-beetle branch July 19, 2024 17:16
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