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

Force remove container if updateMetadata fails #2226

Merged
merged 1 commit into from
Nov 22, 2023
Merged

Conversation

pipex
Copy link
Contributor

@pipex pipex commented Nov 15, 2023

The updateMetadata step renames the container to match the target release when the service doesn't change between releases. We have seen this step fail because of an engine bug that seems to relate to the engine keeping stale references after container restarts. The only way around this issue is to remove the old container and create it again. This implements that workaround during the updateMetadata step to deal with that issue.

Change-type: minor
Relates-to: balena-os/balena-engine#261

Copy link
Contributor

@cywang117 cywang117 left a comment

Choose a reason for hiding this comment

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

LGTM, merge when ready!

@pipex pipex requested a review from klutchell November 20, 2023 17:20
The `updateMetadata` step renames the container to match the target
release when the service doesn't change between releases. We have seen
this step fail because of an engine bug that seems to relate to the
engine keeping stale references after container restarts. The only way
around this issue is to remove the old container and create it again.
This implements that workaround during the updateMetadata step to deal
with that issue.

Change-type: minor
Relates-to: balena-os/balena-engine#261
@pipex
Copy link
Contributor Author

pipex commented Nov 22, 2023

I self-certify!

@flowzone-app flowzone-app bot merged commit 698b815 into master Nov 22, 2023
56 checks passed
@flowzone-app flowzone-app bot deleted the sandbox-error branch November 22, 2023 20:10
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.

3 participants