Skip to content

integration/internal/container: IsInState: touch up error-logs#49220

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:touch_up_IsInState
Jan 7, 2025
Merged

integration/internal/container: IsInState: touch up error-logs#49220
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:touch_up_IsInState

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Before this patch:

remove_test.go:62: timeout hit after 10s: waiting for container to be one of (exited), currently running

After this patch:

remove_test.go:62: waiting for container State.Status to be 'exited', currently 'running'

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah thaJeztah added status/2-code-review kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. area/testing labels Jan 6, 2025
@thaJeztah thaJeztah added this to the 28.0.0 milestone Jan 6, 2025
@thaJeztah thaJeztah self-assigned this Jan 6, 2025
Before this patch:

    remove_test.go:62: timeout hit after 10s: waiting for container to be one of (exited), currently running

After this patch:

    remove_test.go:62: waiting for container State.Status to be 'exited', currently 'running'

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah merged commit b2450ff into moby:master Jan 7, 2025
@thaJeztah thaJeztah deleted the touch_up_IsInState branch January 7, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/testing kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants