Skip to content

refactor container_list_windows_test.go to use Tigron#4780

Merged
AkihiroSuda merged 1 commit into
containerd:mainfrom
must108:containers-list-windows-test
Jul 25, 2026
Merged

refactor container_list_windows_test.go to use Tigron#4780
AkihiroSuda merged 1 commit into
containerd:mainfrom
must108:containers-list-windows-test

Conversation

@must108

@must108 must108 commented Mar 7, 2026

Copy link
Copy Markdown
Contributor

Tests now use nerdtest.Setup, and Tigron

ResolvesUpdates (EDIT: @AkihiroSuda) #4613

@must108

must108 commented Mar 15, 2026

Copy link
Copy Markdown
Contributor Author

@AkihiroSuda

@haytok haytok left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Comment thread cmd/nerdctl/container/container_list_windows_test.go Outdated
Comment thread cmd/nerdctl/container/container_list_windows_test.go Outdated
Comment thread cmd/nerdctl/container/container_list_windows_test.go Outdated
@must108
must108 requested review from haytok and sathiraumesh April 19, 2026 21:42
@must108
must108 force-pushed the containers-list-windows-test branch from 189b030 to 374029c Compare April 20, 2026 18:10
Comment thread cmd/nerdctl/container/container_list_windows_test.go Outdated
Comment thread cmd/nerdctl/container/container_list_windows_test.go Outdated
@AkihiroSuda

Copy link
Copy Markdown
Member

ping @must108

Could you take a look at the review comments from haytok

@must108

must108 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

@haytok apologies for the delay

@AkihiroSuda AkihiroSuda added this to the v2.3.2 milestone Jun 9, 2026

@haytok haytok left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@haytok
haytok requested review from AkihiroSuda and removed request for sathiraumesh June 9, 2026 16:16
@AkihiroSuda

Copy link
Copy Markdown
Member

windows (canary) CI seems constantly failing

@AkihiroSuda

Copy link
Copy Markdown
Member

Also please squash the commits

@AkihiroSuda AkihiroSuda removed this from the v2.3.2 milestone Jun 11, 2026
@AkihiroSuda

Copy link
Copy Markdown
Member

ping @must108

@AkihiroSuda

Copy link
Copy Markdown
Member

Marking the PR as a draft as the CI is failing. Please click Ready for review when ready.

@AkihiroSuda
AkihiroSuda marked this pull request as draft July 18, 2026 21:16
@AkihiroSuda AkihiroSuda reopened this Jul 21, 2026
@AkihiroSuda
AkihiroSuda marked this pull request as ready for review July 22, 2026 07:54
@AkihiroSuda

Copy link
Copy Markdown
Member

The PR is passing CI after restarting, but please make sure to squash the commits @must108

Tests now use nerdtest.Setup and Tigron.

Updates containerd#4613

Signed-off-by: Mustaeen Ahmed <contact@mustaeen.dev>
@must108
must108 force-pushed the containers-list-windows-test branch from 8eeb32b to c44c9c1 Compare July 25, 2026 07:39
@must108

must108 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor Author

@AkihiroSuda should be good for final review

@AkihiroSuda AkihiroSuda added this to the v2.4.0 milestone Jul 25, 2026

@AkihiroSuda AkihiroSuda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda
AkihiroSuda merged commit cf40fc8 into containerd:main Jul 25, 2026
64 of 68 checks passed
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