Skip to content

cli: Add shutdown_delay to full job init output to fix warning.#27900

Open
jrasell wants to merge 1 commit intomainfrom
b-add-shutdown-delay-job-init
Open

cli: Add shutdown_delay to full job init output to fix warning.#27900
jrasell wants to merge 1 commit intomainfrom
b-add-shutdown-delay-job-init

Conversation

@jrasell
Copy link
Copy Markdown
Member

@jrasell jrasell commented May 5, 2026

Nomad recently added a warning to the CLI output when a job includes a service in a group/task that doesn't have a shutdown delay configured.

This change brings the Nomad example job spec created by the init command inline with our recommendations.

Links

Related: #27782

Contributor Checklist

  • Changelog Entry If this PR changes user-facing behavior, please generate and add a
    changelog entry using the make cl command.
  • Testing Please add tests to cover any new functionality or to demonstrate bug fixes and
    ensure regressions will be caught.
  • Documentation If the change impacts user-facing functionality such as the CLI, API, UI,
    and job configuration, please update the Nomad product documentation, which is stored in the
    web-unified-docs repo. Refer to the web-unified-docs contributor guide for docs guidelines.
    Please also consider whether the change requires notes within the upgrade
    guide
    . If you would like help with the docs, tag the nomad-docs team in this PR.

Reviewer Checklist

  • Backport Labels Please add the correct backport labels as described by the internal
    backporting document.
  • Commit Type Ensure the correct merge method is selected which should be "squash and merge"
    in the majority of situations. The main exceptions are long-lived feature branches or merges where
    history should be preserved.
  • Enterprise PRs If this is an enterprise only PR, please add any required changelog entry
    within the public repository.

@jrasell jrasell self-assigned this May 5, 2026
@jrasell jrasell requested review from gulducat and pkazmierczak May 5, 2026 12:09
@jrasell jrasell added backport/ent/1.10.x+ent backport to 1.10.x+ent release line backport/ent/1.11.x+ent backport to 1.11.x+ent release line backport/2.0.x backport to 2.0.x release line labels May 5, 2026
@jrasell jrasell marked this pull request as ready for review May 5, 2026 12:31
@jrasell jrasell requested review from a team as code owners May 5, 2026 12:31
@gulducat
Copy link
Copy Markdown
Member

gulducat commented May 5, 2026

Hey nice. There are other examples too, namely -connect and -connect -short have service{} blocks without any shutdown_delay.

Nomad recently added a warning to the CLI output when a job
includes a service in a group/task that doesn't have a shutdown
delay configured.

This change brings the Nomad example job specs created by the init
command inline with our reccommendations.
Copy link
Copy Markdown
Member

@gulducat gulducat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Just noting that there's no change to example-short.nomad.hcl because it has no service. Although, we could add a provider=nomad service in there... 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/ent/1.10.x+ent backport to 1.10.x+ent release line backport/ent/1.11.x+ent backport to 1.11.x+ent release line backport/2.0.x backport to 2.0.x release line

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants