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

Conversation

@nishanttotla
Copy link
Contributor

Cherry picking moby/moby#33279 into components/cli/vendor/github.com/docker/docker.

This is related to the changes in #13

cc @andrewhsu @mlaventure @thaJeztah

@mavenugo mavenugo mentioned this pull request Jun 6, 2017
23 tasks
@andrewhsu andrewhsu force-pushed the cpick-moby-33279-17.06 branch 2 times, most recently from 9be5bec to 8f1ef90 Compare June 6, 2017 05:07
@andrewhsu
Copy link
Contributor

Two errors with this PR which are reproducible:

00:06:40 FAIL: docker_cli_service_logs_test.go:24: DockerSwarmSuite.TestServiceLogs
00:06:40 
00:06:40 [d693c720905ab] waiting for daemon to start
00:06:40 [d693c720905ab] daemon started
00:06:40 
00:06:40 docker_cli_service_logs_test.go:43:
00:06:40     // make sure task has been deployed.
00:06:40     waitAndAssert(c, defaultReconciliationTimeout,
00:06:40         d.CheckRunningTaskImages, checker.DeepEquals,
00:06:40         map[string]int{"busybox": len(services)})
00:06:40 docker_utils_test.go:471:
00:06:40     c.Assert(v, checker, args...)
00:06:40 ... obtained map[string]int = map[string]int{"busybox:latest":2}
00:06:40 ... expected map[string]int = map[string]int{"busybox":2}

and

00:22:57 FAIL: docker_cli_swarm_unix_test.go:56: DockerSwarmSuite.TestSwarmNetworkPluginV2
00:22:57 
00:22:57 [d88ef2b0e0aea] waiting for daemon to start
00:22:57 [d88ef2b0e0aea] daemon started
00:22:57 
00:22:57 [d65246fc1d5e2] waiting for daemon to start
00:22:57 [d65246fc1d5e2] daemon started
00:22:57 
00:22:57 docker_cli_swarm_unix_test.go:103:
00:22:57     waitAndAssert(c, defaultReconciliationTimeout, d1.CheckRunningTaskImages, checker.DeepEquals,
00:22:57         map[string]int{image: 1})
00:22:57 docker_utils_test.go:471:
00:22:57     c.Assert(v, checker, args...)
00:22:57 ... obtained map[string]int = map[string]int{"busybox:latest":1}
00:22:57 ... expected map[string]int = map[string]int{"busybox":1}

@andrewhsu
Copy link
Contributor

To run the tests from your branch, you'll need to build the docker-dev image from the components/engine and the cli from components/cli and pass the cli location into the test-integration-cli target with DOCKER_CLI_PATH. You can also restrict the tests run with TESTFLAGS, something like this:

$ docker run --rm -it \
  -e DOCKER_CLI_PATH=/home/ubuntu/src/docker-ce/components/cli/build/docker \
  -e TESTFLAGS='-check.f DockerSwarmSuite.TestServiceLogs'  \
  $docker-dev-image ./hack/make.sh binary test-integration-cli

@nishanttotla
Copy link
Contributor Author

nishanttotla commented Jun 6, 2017

@andrewhsu ah I see why they're failing. The cherry-pick makes sure that all service images have a default latest tag if one is not provided. The tests need to be fixed.

Can that be done in this PR?

@nishanttotla
Copy link
Contributor Author

@andrewhsu I've added a fix for those two tests as an extra commit. PTAL.

@nishanttotla nishanttotla force-pushed the cpick-moby-33279-17.06 branch from 25acdc2 to 02cc54a Compare June 6, 2017 18:39
@andrewhsu andrewhsu mentioned this pull request Jun 6, 2017
40 tasks
@nishanttotla
Copy link
Contributor Author

@andrewhsu now that RC2 is out, could we move ahead with this PR?

nishanttotla and others added 3 commits June 10, 2017 00:27
Signed-off-by: Nishant Totla <nishanttotla@gmail.com>
(cherry picked from commit 5efcec7)

Conflicts resolved:
components/cli/vendor/github.com/docker/docker/client/service_create.go

Signed-off-by: Andrew Hsu <andrewhsu@docker.com>
Because of cherry-pick from commit 5efcec7
into components/cli/vendor/github.com/docker/docker

Signed-off-by: Andrew Hsu <andrewhsu@docker.com>
Because of cherry-pick from commit
5efcec7 into
components/cli/vendor/github.com/docker/docker

Signed-off-by: Nishant Totla <nishanttotla@gmail.com>
@andrewhsu
Copy link
Contributor

LGTM

@andrewhsu andrewhsu merged commit e92c400 into docker-archive:17.06 Jun 10, 2017
@nishanttotla nishanttotla deleted the cpick-moby-33279-17.06 branch June 13, 2017 21:39
@andrewhsu andrewhsu modified the milestone: 17.06.0 Jul 12, 2017
andrewhsu added a commit that referenced this pull request Aug 30, 2017
[ppc64le] add deb support for ubuntu-xenial
Upstream-commit: 53b3255
Component: packaging
docker-jenkins pushed a commit that referenced this pull request Aug 7, 2018
[18.06] Bump SwarmKit to 8852e88
Upstream-commit: 5150e8235c56517af44c7902e71a2e954a05a4ac
Component: engine
docker-jenkins pushed a commit that referenced this pull request May 3, 2019
1. There is no need to persist DOCKER_GITCOMMIT,
as it's not needed for runtime, only for build.
So, remove ENV.

2. In case $GITCOMMIT is not defined during build time
(and it happens if .git directory is not present),
we still need to have some value set, so set it to
`undefined`. Otherwise we'll have something like

>  => ERROR [builder 2/3] RUN hack/make.sh build-integration-test-binary
> ------
>  > [builder 2/3] RUN hack/make.sh build-integration-test-binary:
> #32 0.488
> #32 0.505 error: .git directory missing and DOCKER_GITCOMMIT not specified
> #32 0.505   Please either build with the .git directory accessible, or specify the
> #32 0.505   exact (--short) commit hash you are building using DOCKER_GITCOMMIT for
> #32 0.505   future accountability in diagnosing build issues.  Thanks!

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
Upstream-commit: c3b24944ca10edb835de6e806ad6cd9e952ebc11
Component: engine
docker-jenkins pushed a commit that referenced this pull request May 14, 2019
1. There is no need to persist DOCKER_GITCOMMIT,
as it's not needed for runtime, only for build.
So, remove ENV.

2. In case $GITCOMMIT is not defined during build time
(and it happens if .git directory is not present),
we still need to have some value set, so set it to
`undefined`. Otherwise we'll have something like

>  => ERROR [builder 2/3] RUN hack/make.sh build-integration-test-binary
> ------
>  > [builder 2/3] RUN hack/make.sh build-integration-test-binary:
> #32 0.488
> #32 0.505 error: .git directory missing and DOCKER_GITCOMMIT not specified
> #32 0.505   Please either build with the .git directory accessible, or specify the
> #32 0.505   exact (--short) commit hash you are building using DOCKER_GITCOMMIT for
> #32 0.505   future accountability in diagnosing build issues.  Thanks!

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
(cherry picked from commit c3b24944ca10edb835de6e806ad6cd9e952ebc11)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Upstream-commit: 14bb71d508980769d581010a73565a3c7535a787
Component: engine
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Jan 29, 2020
…79-17.06

[cherry-pick] vendor: Ensure service images get default tag and print familiar strings
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Jan 30, 2020
…rt-xenial

[ppc64le] add deb support for ubuntu-xenial
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Feb 3, 2020
…79-17.06

[cherry-pick] vendor: Ensure service images get default tag and print familiar strings
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants