Skip to content

Conversation

@Hook25
Copy link
Collaborator

@Hook25 Hook25 commented Jun 14, 2023

Description

This is done because if a test run is interrupted midway through, reusing the container will actually break it (because the old run is still in the container files, leading checkbox to prompt for a restart/drop)

Resolved issues

N/A

Documentation

N/A

Tests

This was tested by running metabox with a container to update, one to create and one that should be updated but it is in an invalid state (missing provisioned)

@Hook25 Hook25 requested a review from kissiel June 14, 2023 09:10
This is done because if a test run is interrupted midway through,
reusing the container will actually break it (because the old run
is still in the container files, leading checkbox to prompt for a
restart/drop)
@Hook25 Hook25 force-pushed the fix_mb_reuse_provisioning_rollback branch from d66d5b0 to e992c80 Compare June 14, 2023 09:24
Copy link
Contributor

@kissiel kissiel left a comment

Choose a reason for hiding this comment

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

LGTM!
I like how this also fixes a problem of half-baked containers.

@Hook25 Hook25 merged commit e48c8e0 into main Jun 14, 2023
@Hook25 Hook25 deleted the fix_mb_reuse_provisioning_rollback branch June 14, 2023 09:33
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