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

MEN-4813: avoid race condition when aborting deployments #603

Merged
merged 1 commit into from
Jun 30, 2021

Conversation

tranchitella
Copy link
Contributor

If the user aborts a deployment while devices are updating their status,
it can happen a stats recalculation is performed in parallel with the
abort operation. In this case, deployment's end time is set but the
status is not "finished", making it impossible to abort the deployment.

Changelog: none

Signed-off-by: Fabio Tranchitella [email protected]

@tranchitella
Copy link
Contributor Author

@mender-test-bot start pipeline

@mender-test-bot
Copy link

@tranchitella, Let me know if you want to start the integration pipeline by mentioning me and the command "start pipeline".

@mender-test-bot
Copy link

Hello 😸 I created a pipeline for you here: Pipeline-329028279

Build Configuration Matrix

Key Value
AUDITLOGS_REV origin/master
BUILD_CLIENT false
CREATE_ARTIFACT_WORKER_REV origin/master
DEPLOYMENTS_ENTERPRISE_REV origin/master
DEPLOYMENTS_REV pull/603/head
DEVICEAUTH_REV origin/master
DEVICECONFIG_REV origin/master
DEVICECONNECT_REV origin/master
GUI_REV origin/master
INTEGRATION_REV master
INVENTORY_ENTERPRISE_REV origin/master
INVENTORY_REV origin/master
MENDER_ARTIFACT_REV origin/master
MENDER_CLI_REV origin/master
MENDER_CONNECT_REV origin/master
MENDER_REV origin/master
MTLS_AMBASSADOR_REV origin/master
RUN_INTEGRATION_TESTS true
TENANTADM_REV origin/master
USERADM_ENTERPRISE_REV origin/master
USERADM_REV origin/master
WORKFLOWS_ENTERPRISE_REV origin/master
WORKFLOWS_REV origin/master

If the user aborts a deployment while devices are updating their status,
it can happen a stats recalculation is performed in parallel with the
abort operation. In this case, deployment's end time is set but the
status is not "finished", making it impossible to abort the deployment.

Changelog: none

Signed-off-by: Fabio Tranchitella <[email protected]>
@tranchitella
Copy link
Contributor Author

@mender-test-bot start pipeline

@mender-test-bot
Copy link

Hello 😸 I created a pipeline for you here: Pipeline-329068268

Build Configuration Matrix

Key Value
AUDITLOGS_REV origin/master
BUILD_CLIENT false
CREATE_ARTIFACT_WORKER_REV origin/master
DEPLOYMENTS_ENTERPRISE_REV origin/master
DEPLOYMENTS_REV pull/603/head
DEVICEAUTH_REV origin/master
DEVICECONFIG_REV origin/master
DEVICECONNECT_REV origin/master
GUI_REV origin/master
INTEGRATION_REV master
INVENTORY_ENTERPRISE_REV origin/master
INVENTORY_REV origin/master
MENDER_ARTIFACT_REV origin/master
MENDER_CLI_REV origin/master
MENDER_CONNECT_REV origin/master
MENDER_REV origin/master
MTLS_AMBASSADOR_REV origin/master
RUN_INTEGRATION_TESTS true
TENANTADM_REV origin/master
USERADM_ENTERPRISE_REV origin/master
USERADM_REV origin/master
WORKFLOWS_ENTERPRISE_REV origin/master
WORKFLOWS_REV origin/master

@tranchitella tranchitella merged commit 4d94818 into mendersoftware:master Jun 30, 2021
@tranchitella tranchitella deleted the men-4813 branch June 30, 2021 13:25
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.

4 participants