Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

[17.10] Backport version fix for Windows manifest lists #266

Merged
merged 2 commits into from Oct 11, 2017

Conversation

johnstep
Copy link
Contributor

Backport fix:

with cherry pick of moby/moby@38aef56 and moby/moby@8ed8f4a:

git cherry-pick -s -x -Xsubtree=components/engine 38aef5
git cherry-pick -s -x -Xsubtree=components/engine 8ed8f4

Conflict with pull_v2.go in moby/moby@8ed8f4a resolved by reverting back to detecting the OS the previous way (before it was supplied as a parameter) and also only sorting for Windows (like in current master) as 17.10 has not backported moby/moby@d98ecf2.

dmcgowan and others added 2 commits October 10, 2017 21:10
Update logic to choose manifest from manifest list to check
for os version on Windows. Separate the logic for windows
and unix to keep unix logic the same.

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
(cherry picked from commit 38aef56)
Signed-off-by: John Stephens <johnstep@docker.com>
Signed-off-by: John Stephens <johnstep@docker.com>
(cherry picked from commit 8ed8f4a)

Conflicts:
	components/engine/distribution/pull_v2_windows.go

Signed-off-by: John Stephens <johnstep@docker.com>
@andrewhsu andrewhsu added this to the 17.10.0 milestone Oct 11, 2017
@andrewhsu
Copy link
Contributor

cc @jhowardmsft

@andrewhsu
Copy link
Contributor

Hmm...just one test failure on s390x tests:

00:17:35.613 [DockerSwarmSuite] FAIL: docker_api_swarm_test.go:349: DockerSwarmSuite.TestAPISwarmRaftQuorum
00:17:35.613 [DockerSwarmSuite] 
00:17:35.613 [DockerSwarmSuite] [dbebcec8a582e] waiting for daemon to start
00:17:35.613 [DockerSwarmSuite] [dbebcec8a582e] daemon started
00:17:35.613 [DockerSwarmSuite] 
00:17:35.613 [DockerSwarmSuite] [d4082a6823d4b] waiting for daemon to start
00:17:35.613 [DockerSwarmSuite] [d4082a6823d4b] daemon started
00:17:35.613 [DockerSwarmSuite] 
00:17:35.613 [DockerSwarmSuite] [dc38ecf2ae08d] waiting for daemon to start
00:17:35.613 [DockerSwarmSuite] [dc38ecf2ae08d] daemon started
00:17:35.613 [DockerSwarmSuite] 
00:17:35.613 [DockerSwarmSuite] [d4082a6823d4b] exiting daemon
00:17:35.613 [DockerSwarmSuite] [dc38ecf2ae08d] exiting daemon
00:17:35.613 [DockerSwarmSuite] docker_api_swarm_test.go:374:
00:17:35.613 [DockerSwarmSuite]     c.Assert(err.Error(), checker.Contains, "deadline exceeded")
00:17:35.613 [DockerSwarmSuite] ... obtained string = "Error response from daemon: rpc error: code = Internal desc = transport is closing"
00:17:35.613 [DockerSwarmSuite] ... substring string = "deadline exceeded"

Copy link
Contributor

@seemethere seemethere left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

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

LGTM

@andrewhsu andrewhsu merged commit 6f56735 into docker:17.10 Oct 11, 2017
@johnstep johnstep deleted the windows-manifest-lists branch October 24, 2017 18:09
docker-jenkins pushed a commit that referenced this pull request Nov 28, 2018
[18.09] Add libseccomp requirement for rpm packages
Upstream-commit: 9a7acfb9942edb67ce429f6d78c228036b0d465f
Component: packaging
docker-jenkins pushed a commit that referenced this pull request Jun 11, 2019
…d_gid_mappings

[19.03 backport] Stop sorting uid and gid ranges in id maps
Upstream-commit: a8d1b4a1ab031ace6d47d53366ee252b655d6538
Component: engine
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Feb 3, 2020
[17.10] Backport version fix for Windows manifest lists
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants