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

Restart AppNet Relay on failure #3469

Merged
merged 1 commit into from Nov 7, 2022
Merged

Conversation

yinyic
Copy link
Contributor

@yinyic yinyic commented Nov 4, 2022

Summary

Restart AppNet Relay on failure

Implementation details

Configure Docker restart policy for AppNet Relay container so that it will be restarted by docker if it exits due to an error.
MaximumRetryCount is set to 0 to indicate that we are not enforcing a maximum count (reference)

Testing

New tests cover the changes: no

Tested with new ECS Agent and verified that AppNet Relay has correct restart policy configured

$ docker inspect 805d395608a5 | jq .[0].HostConfig.RestartPolicy
{
  "Name": "on-failure",
  "MaximumRetryCount": 0
}

Description for the changelog

Restart AppNet Relay on failure

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@yinyic yinyic requested a review from a team as a code owner November 4, 2022 21:35
@yinyic yinyic added the bot/test label Nov 4, 2022
@yinyic yinyic merged commit 097bf88 into aws:dev Nov 7, 2022
@yinyic yinyic deleted the appnet-relay-restart branch November 7, 2022 18:02
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.

None yet

4 participants