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] bugfix: when hitting the "start" button on a job page, if it has no submission data, fallback to raw json definition into release/1.6.x #18625

Conversation

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

Backport

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

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


In Nomad 1.6, we introduced HCL-in-the-UI using a new /submissions endpoint among other things.

Shortly after the release, we realized we missed a spot: stop/starting via the UI would not maintain the submission data. My initial fix for this solved the issue of maintaining HCL, but caused a new bug: jobs that lacked submission data (perhaps they were submitted via curl with just the json spec, and without HCL) would 404 on /submission and fail to progress through to /parse and submit.

This PR provides a fallback to the initial method in cases where the /submission request fails.

Resolves #18547
Resolves #18536


Overview of commits

@hc-github-team-nomad-core hc-github-team-nomad-core force-pushed the backport/18547-job-cannot-be-restarted-from-ui-in-162/factually-moved-coyote branch 2 times, most recently from 3319c68 to 7418daf Compare September 29, 2023 18:52
@vercel vercel bot temporarily deployed to Preview – nomad September 29, 2023 18:58 Inactive
@vercel vercel bot temporarily deployed to Preview – nomad-storybook-and-ui September 29, 2023 18:59 Inactive
@github-actions
Copy link

Ember Test Audit comparison

release/1.6.x 7418daf change
passes 1532 1532 0
failures 1 1 0
flaky 0 0 0
duration 000ms 000ms -000ms

@philrenaud philrenaud merged commit bd4996c into release/1.6.x Sep 29, 2023
11 checks passed
@philrenaud philrenaud deleted the backport/18547-job-cannot-be-restarted-from-ui-in-162/factually-moved-coyote branch September 29, 2023 19:21
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