Skip to content

Docker client: reimplement the events API #12486

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

simonrw
Copy link
Contributor

@simonrw simonrw commented Apr 4, 2025

Motivation

In #9932 we implemented the events API which will be useful for general container management, however it introduced some flaky tests. We reverted it in #10025. This test reinstates that implementation with some minor consistency tweaks and bug fixes.

Changes

TBD

@simonrw simonrw added the semver: patch Non-breaking changes which can be included in patch releases label Apr 4, 2025
@simonrw simonrw self-assigned this Apr 4, 2025
Copy link

github-actions bot commented Apr 5, 2025

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 51m 51s ⏱️ - 3m 30s
4 325 tests ±0  3 981 ✅ ±0  344 💤 ±0  0 ❌ ±0 
4 327 runs  ±0  3 981 ✅ ±0  346 💤 ±0  0 ❌ ±0 

Results for commit f76edd2. ± Comparison against base commit 30a0d92.

@simonrw simonrw added this to the Playground milestone Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant