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

Build: log when a build is reset #9015

Merged
merged 1 commit into from
Mar 16, 2022
Merged

Build: log when a build is reset #9015

merged 1 commit into from
Mar 16, 2022

Conversation

humitos
Copy link
Member

@humitos humitos commented Mar 15, 2022

If the build worker is killed for any reason, as we are using ACKS_LATE, the
build task will be picked by another worker and re-executed.

When that happens, we detect that the Build object already has some commands
saved on it and we delete those commands to start again from scratch.

This commit just adds a log line to be aware how often this situation happens in
production and be able to make other decisions.

If the build worker is killed for any reason, as we are using `ACKS_LATE`, the
build task will be picked by another worker and re-executed.

When that happens, we detect that the `Build` object already has some commands
saved on it and we delete those commands to start again from scratch.

This commit just adds a log line to be aware how often this situation happens in
production and be able to make other decisions.
Copy link
Member

@ericholscher ericholscher left a comment

Choose a reason for hiding this comment

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

This makes sense, and will make it easier to understand how often it happens.

@humitos humitos merged commit 902590d into master Mar 16, 2022
@humitos humitos deleted the humitos/build-reset-log branch March 16, 2022 08:55
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