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

manual backport 889c5aa0f7 #17208

Closed
wants to merge 17 commits into from
Closed

Conversation

shoenig
Copy link
Member

@shoenig shoenig commented May 16, 2023

No description provided.

hc-github-team-nomad-core and others added 17 commits May 5, 2023 09:46
…g type. into release/1.5.x (#17096)

This pull request was automerged via backport-assistant
…7071)

Fix for #17070. Corrected the status description and modify time of evals which are canceled due to another eval having completed in the meantime.
Nomad 1.5.4 shipped with a logmon bug that we rolled out a fix for in Nomad
1.5.5. Unfortunately we can't yank the release but we should leave a note in the
upgrade guide telling users to avoid it.
This pull request was automerged via backport-assistant
This pull request was automerged via backport-assistant
This pull request was automerged via backport-assistant
This pull request was automerged via backport-assistant
This pull request was automerged via backport-assistant
This pull request was automerged via backport-assistant
… (#17174)

Tools like `nomad-nodesim` are unable to implement a minimal implementation of
an allocrunner so that we can test the client communication without having to
lug around the entire allocrunner/taskrunner code base. The allocrunner was
implemented with an interface specifically for this purpose, but there were
circular imports that made it challenging to use in practice.

Move the AllocRunner interface into an inner package and provide a factory
function type. Provide a minimal test that exercises the new function so that
consumers have some idea of what the minimum implementation required is.

Co-authored-by: Tim Gross <[email protected]>
This pull request was automerged via backport-assistant
This pull request was automerged via backport-assistant
…release/1.5.x (#17207)

This pull request was automerged via backport-assistant
…#16905)

* services: un-mark group services as deregistered if restart hook runs

This PR may fix a bug where group services will never be deregistered if the
group undergoes a task restart.

* e2e: add test case for restart and deregister group service

* cl: add cl

* e2e: add wait for service list call
@shoenig
Copy link
Member Author

shoenig commented May 16, 2023

Ehh lemme try this again with the correct base

@shoenig shoenig closed this May 16, 2023
@shoenig shoenig deleted the manual-backport-889c5aa0f7 branch May 16, 2023 17:35
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.

NOMAD_ADDR_* variables incorrect with ipv6
5 participants