Skip to content

[28.x backport] Dockerfile.windows: remove deprecated 7Zip4Powershell #50938

Merged
austinvazquez merged 1 commit intomoby:28.xfrom
vvoland:50936-28.x
Sep 9, 2025
Merged

[28.x backport] Dockerfile.windows: remove deprecated 7Zip4Powershell #50938
austinvazquez merged 1 commit intomoby:28.xfrom
vvoland:50936-28.x

Conversation

@vvoland
Copy link
Copy Markdown
Contributor

@vvoland vvoland commented Sep 9, 2025

tar utility is included in Windows 10 (17063+) and Windows Server 2019+ so we can use it directly.

This fixes recent Windows CI failures: https://github.com/moby/moby/actions/runs/17578675113/job/49940905820?pr=50932

 WARNING: The specified PackageManagement provider 'NuGet' is not available.
Install-Package : No match was found for the specified search criteria and 
package name '7Zip4PowerShell'. Try Get-PackageSource to see all available 
registered package sources.
At line:1 char:3086
+ ... ownloading containerd; Install-Package -Force 7Zip4PowerShell; $locat ...
+                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Microsoft.Power....InstallPacka 
   ge:InstallPackage) [Install-Package], Exception
    + FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.Pac

@vvoland
Dockerfile.windows: remove deprecated 7Zip4Powershell
81da1cf 
`tar` utility is included in Windows 10 (17063+) and Windows Server
2019+ so we can use it directly.

Signed-off-by: Paweł Gronowski <[email protected]>
(cherry picked from commit 8c8324b)
Signed-off-by: Paweł Gronowski <[email protected]>
@vvoland vvoland added this to the 28.4.1 milestone Sep 9, 2025
@vvoland vvoland self-assigned this Sep 9, 2025
@vvoland vvoland modified the milestones: 28.4.1, 28.5.0 Sep 9, 2025
thaJeztah
thaJeztah approved these changes Sep 9, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@austinvazquez
Copy link
Copy Markdown
Contributor

austinvazquez commented Sep 9, 2025

Integration test failure is unrelated known flaky test. Merging.

=== Failed
=== FAIL: amd64.integration.networking TestBridgeICC/IPv6_link-local_address_on_internal_network (4.36s)
    firewall.go:122: Firewalld reload completed at 2025-09-09T16:02:02Z
    bridge_linux_test.go:219: assertion failed: 1 (res.ExitCode int) != 0 (int)
    bridge_linux_test.go:221: assertion failed: string "PING fe80::2%eth0 (fe80::2): 56 data bytes\n\n--- fe80::2%eth0 ping statistics ---\n1 packets transmitted, 0 packets received, 100% packet loss\n" does not contain "1 packets transmitted, 1 packets received"
    --- FAIL: TestBridgeICC/IPv6_link-local_address_on_internal_network (4.36s)

=== FAIL: amd64.integration.networking TestBridgeICC (26.25s)

@austinvazquez austinvazquez merged commit 46dcd7f into moby:28.x Sep 9, 2025
169 of 170 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thaJeztah thaJeztah thaJeztah approved these changes

@austinvazquez austinvazquez austinvazquez approved these changes

Assignees

@vvoland vvoland

Labels

area/testing kind/bugfix PR's that fix bugs platform/windows status/2-code-review

Projects

None yet

Milestone

28.5.0

Development

Successfully merging this pull request may close these issues.

3 participants

@vvoland @austinvazquez @thaJeztah