{"push_id":1878363739,"size":374,"distinct_size":374,"ref":"refs/heads/master","head":"4ac4c8ef4b06ffa8ccd4fd3ae0f76e4ca820c1bc","before":"397a57d3a5f057a3c787afa107964b692ec2ebe1","commits":[{"sha":"0e87588bbb2482f9635a19ac43be1c0ee9652ac2","author":{"name":"Lei Jitang","email":"9198a71242d42e547d4ff4dd9a99ebac35ae58f3@huawei.com"},"message":"Enable inspect exec if container is pause/restarting/non-running\n\nif docker exec exit and at the same the the container is pause,\nthere could be a chance the `docker exec exit` will fail\n```\n$ docker exec -ti 388c7f47a06c sh\n/ # exit\nError response from daemon: Container 388c7f47a06cce0856266ffd56a2ce2901689ca7a6b9cd741b37652418448f2b is paused, unpause the container before exec\n```\nTo reproduce this error easilly, we can add a sleep in `containerPause`\n```\n--- a/daemon/pause.go\n+++ b/daemon/pause.go\n@@ -2,6 +2,7 @@ package daemon\n\n import (\n        \"fmt\"\n+       \"time\"\n\n        \"github.com/docker/docker/container\"\n )\n@@ -25,7 +26,7 @@ func (daemon *Daemon) ContainerPause(name string) error {\n func (daemon *Daemon) containerPause(container *container.Container) error {\n        container.Lock()\n        defer container.Unlock()\n-\n+       time.Sleep(time.Second * 5)\n        // We cannot Pause the container which is not running\n        if !container.Running {\n                return errNotRunning{container.ID}\n```\n\nSigned-off-by: Lei Jitang <leijitang@huawei.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/0e87588bbb2482f9635a19ac43be1c0ee9652ac2"},{"sha":"66cfe61f71252f528ddb458d554cd241e996d9f1","author":{"name":"Stephen J Day","email":"f612758db3805c4abe0d56243c03a33f3294af3f@docker.com"},"message":"pkg: remove random package\n\nThe unnecessary `random` package has been removed in favor of using the\n`math/rand` package directly. Seeding of the random value from crypto\nhas been added to the `stringid` package to account for the change.\n\nMay need to add an equivalent seed to `namesgenerator`, but this is\noften used with `stringid` and has collision protection.\n\nSigned-off-by: Stephen J Day <stephen.day@docker.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/66cfe61f71252f528ddb458d554cd241e996d9f1"},{"sha":"d42c268fc3983d19e9dbd800948e7c4445c425f0","author":{"name":"gkze","email":"d8c2dd775e638f321570ec9e0e20ada72f845d33@bugsnag.com"},"message":"GCP Logs: send log line as jsonPayload.message instead of jsonPayload.data\n\nSigned-off-by: gkze <george@bugsnag.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/d42c268fc3983d19e9dbd800948e7c4445c425f0"},{"sha":"960b8d9294356b5651938244a62e0d485da72211","author":{"name":"Brian Goff","email":"7c85cf8ae7443f547a26fb59a1f52171e1274aa1@gmail.com"},"message":"Make TestLogsAPIStdout a bit less racey\n\nSigned-off-by: Brian Goff <cpuguy83@gmail.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/960b8d9294356b5651938244a62e0d485da72211"},{"sha":"24477e70040019ca421ec1031dc553dc780c02f1","author":{"name":"Aaron Lehmann","email":"ab01047e91402f1a4542861b1076cf31bcc38777@docker.com"},"message":"cluster: Only pass a join address when in the process of joining a cluster\n\nThis code currently passes a random manager address when creating a new\nNode. This doesn't really make sense - we should only pass a join\naddress on the initial join, or when retrying that join. An upcoming\nchange to swarmkit will pay attention to JoinAddr significant when a\nnode is already part of a cluster, so passing in the random value needs\nto be avoided.\n\nSigned-off-by: Aaron Lehmann <aaron.lehmann@docker.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/24477e70040019ca421ec1031dc553dc780c02f1"},{"sha":"48829ddf885eb656bff6dcf053bf96471b106bf0","author":{"name":"Brian Goff","email":"7c85cf8ae7443f547a26fb59a1f52171e1274aa1@gmail.com"},"message":"Add stream format details for attach/logs endpoint\n\nSigned-off-by: Brian Goff <cpuguy83@gmail.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/48829ddf885eb656bff6dcf053bf96471b106bf0"},{"sha":"2db05316d09214f5cd6de24e9f17784cbc2f2573","author":{"name":"Miklos Szegedi","email":"b93174b7542708ad3a18cd5f5b3ef7bf46ec39cc@cloudera.com"},"message":"Whitelist adjtimex get operation. Adjustment operations are gated by CAP_SYS_TIME\n\nSigned-off-by: Miklos Szegedi <miklos.szegedi@cloudera.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/2db05316d09214f5cd6de24e9f17784cbc2f2573"},{"sha":"17173efbe00d422392f91c672350266314808b28","author":{"name":"Aaron Lehmann","email":"ab01047e91402f1a4542861b1076cf31bcc38777@docker.com"},"message":"integration-cli: Replace sleeps with polling in swarm lock/unlock tests\n\nThis will hopefully make the tests more robust by replacing a fixed 3s\nsleep with a polling loop that looks at whether the key PEM file is\nencrypted or not.\n\nSigned-off-by: Aaron Lehmann <aaron.lehmann@docker.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/17173efbe00d422392f91c672350266314808b28"},{"sha":"1a6bf8248a32c160347e4daf3dd4f15023357889","author":{"name":"Peng Tao","email":"f3d91b2a395157033aa70913a4d8f4796b4824bb@gmail.com"},"message":"fix aufs mount option length calculation\n\nWe add \",dirperm1\" but only increase length by len(\"dirperm1\").\n\nSigned-off-by: Peng Tao <bergwolf@gmail.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/1a6bf8248a32c160347e4daf3dd4f15023357889"},{"sha":"bd5f92d2631df7c932b93e72e45b39cba19f2f3b","author":{"name":"Daniel Nephin","email":"3ad2504a4cf4aded6f6a35c53e1619a88aadd930@docker.com"},"message":"Remove CopyOnBuild from the daemon.\n\nAdd CreateImage() to the daemon\nRefactor daemon.Comit() and expose a Image.NewChild()\nUpdate copy to use IDMappings.\n\nSigned-off-by: Daniel Nephin <dnephin@docker.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/bd5f92d2631df7c932b93e72e45b39cba19f2f3b"},{"sha":"ecd44d23cc3c863c20467377cf568b920e2dd89c","author":{"name":"Daniel Nephin","email":"3ad2504a4cf4aded6f6a35c53e1619a88aadd930@docker.com"},"message":"Rename LazyContext to LazySource.\n\nSigned-off-by: Daniel Nephin <dnephin@docker.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/ecd44d23cc3c863c20467377cf568b920e2dd89c"},{"sha":"51360965206b0db49cc0365dabb590063a17a9df","author":{"name":"Daniel Nephin","email":"3ad2504a4cf4aded6f6a35c53e1619a88aadd930@docker.com"},"message":"Fix copy when used with scratch and images with empty RootFS\n\nCommit the rwLayer to get the correct DiffID\nRefacator copy in thebuilder\nmove more code into exportImage\ncleanup some windows tests\nRelease the newly commited layer.\nSet the imageID on the buildStage after exporting a new image.\nMove archiver to BuildManager.\nHave ReleaseableLayer.Commit return a layer\nand store the Image from exportImage in the local imageSources cache\nRemove NewChild from image interface.\n\nSigned-off-by: Daniel Nephin <dnephin@docker.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/51360965206b0db49cc0365dabb590063a17a9df"},{"sha":"7e03cebff42717a9a6f04cefdaab931fc152ba51","author":{"name":"Naveed Jamil","email":"e2a0aaedb0c5936b6827131bc098201a38d2b1d4@tenpearls.com"},"message":"Added Test Case Coverage for PKG/REEXEC\n\nSigned-off-by: Naveed Jamil <naveed.jamil@tenpearls.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/7e03cebff42717a9a6f04cefdaab931fc152ba51"},{"sha":"ebe66b1d0f52dc58a98a428d4efa4d2f2743b96e","author":{"name":"Sebastiaan van Stijn","email":"f27a17fc90c00d8c248d8510a1084221738b492b@gone.nl"},"message":"Use tempdir instead of working directory as build-context\n\nThe `makefile()` utility was used to create a temporary Dockerfile, and after\ntests completed, this file was deleted.\n\nHowever, the _build_ used the current path (`/usr/local/bin/docker`) as\nbuild-context. As a result, roughtly 20 MB was sent as build-context for each\nbuild, but none of the builds actually required a build-context.\n\nThis patch;\n\n- creates a temp-dir for the test, which can be used as build-context\n- changes the `makefile()` utility and removes the `cleanup` functionality\n- instead, the `temp-dir` is removed after the test finishes (which also removes the temporary `Dockerfile`)\n\nSigned-off-by: Sebastiaan van Stijn <github@gone.nl>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/ebe66b1d0f52dc58a98a428d4efa4d2f2743b96e"},{"sha":"5d04fe73bf9fa7cff1b99206f39536aed807efb3","author":{"name":"Sebastiaan van Stijn","email":"f27a17fc90c00d8c248d8510a1084221738b492b@gone.nl"},"message":"Simplify DockerRegistrySuite.TestUserAgentPassThrough()\n\nThis patch simplifies the test by;\n\n- re-using the registry-mock / handler\n- skipping the last `docker build`, which was only\n  used to make sure a local image was present. Instead,\n  the daemon is started with a `busybox` image loaded.\n\nAlso added a comment, explaining why the mock always\nreturns a 404 (hence, error/output-string should not\nbe checked in the test), and made the mock return a\nvalid/correctly formatted error response.\n\nSigned-off-by: Sebastiaan van Stijn <github@gone.nl>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/5d04fe73bf9fa7cff1b99206f39536aed807efb3"},{"sha":"cffa6d52a2819c126517333b8b31337ded5cc0d9","author":{"name":"Naveed Jamil","email":"e2a0aaedb0c5936b6827131bc098201a38d2b1d4@tenpearls.com"},"message":"Added Test Case Coverage for PKG/PROMISE\n\nSigned-off-by: Naveed Jamil <naveed.jamil@tenpearls.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/cffa6d52a2819c126517333b8b31337ded5cc0d9"},{"sha":"f3fce00ef3f3b7ead5ac211f9aafabfebf820c2f","author":{"name":"Sebastiaan van Stijn","email":"f27a17fc90c00d8c248d8510a1084221738b492b@gone.nl"},"message":"Make TestServiceCreateCompatiblePlatforms use actual API response\n\nThe `TestServiceCreateCompatiblePlatforms()` test was confusing, because it\ndid not actually use the mocked API response, but used a local\n`distributionInspectBody` variable to verify that the `/distribution/` endpoint\nwas called.\n\nThis flow was especially confusing, because a comment in the test describes;\n\n    \"check if the /distribution endpoint returned correct output\"\n\nIf (for whatever reason), the endpoint was not called, the test would panic,\nbecause the `distributionInspectBody` would not be set.\n\nThis patch rewrites the test to use the actual API response that is returned\nby the mock, and verifies that the information returned by the `/distribution/`\nendpoint is properly used in the service's definition.\n\nSigned-off-by: Sebastiaan van Stijn <github@gone.nl>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/f3fce00ef3f3b7ead5ac211f9aafabfebf820c2f"},{"sha":"8e71b1e210dc0eff980f39271d6c1dd48d87024e","author":{"name":"Darren Stahl","email":"18d6ca1d63992e62d6ace8ae1da7db05093f03e6@microsoft.com"},"message":"Skip evaluation of symlinks to data root on IoT Core\n\nSigned-off-by: Darren Stahl <darst@microsoft.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/8e71b1e210dc0eff980f39271d6c1dd48d87024e"},{"sha":"5b0993d6c778c18735692560538c790faa3dbbb4","author":{"name":"Wentao Zhang","email":"b3d8599bc8577c4ba1a9bd01c0face89e64d16f1@huawei.com"},"message":"When daemon is in startup process, could not start container\n\nDescription:\n When docker is in startup process and containerd sends an \"process exit\" event to docker.\n If the container config '--restart=always', restartmanager will start this container very soon.\n\n But some initialization is not done, e.g. `daemon.netController`,when visit, docker would panic.\n\nSigned-off-by: Wentao Zhang <zhangwentao234@huawei.com>","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/5b0993d6c778c18735692560538c790faa3dbbb4"},{"sha":"7fdc2e42d1d8eadf38976c3363b994040ad28fa5","author":{"name":"Vincent Demeester","email":"c4c21f2bfe4ec34ddc59cdd941972a20fc455cb0@docker.com"},"message":"Merge pull request #33637 from thaJeztah/improve-ci-build-time\n\nMinor refactor/cleanup of tests","distinct":true,"url":"https://api.github.com/repos/dsheets/moby/commits/7fdc2e42d1d8eadf38976c3363b994040ad28fa5"}]}
{"actor":{"display_login":"dsheets"}}