Skip to content

Commit cd4c089

Browse files
authored
Merge pull request moby#32626 from aaronlehmann/vendor-swarmkit-61a92e8
Vendor swarmkit 61a92e8
2 parents 6493256 + 2b5ef9b commit cd4c089

File tree

3 files changed

+60
-17
lines changed

3 files changed

+60
-17
lines changed

integration-cli/docker_cli_swarm_test.go

+8-7
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import (
1515
"strings"
1616
"time"
1717

18+
"github.com/cloudflare/cfssl/helpers"
1819
"github.com/docker/docker/api/types"
1920
"github.com/docker/docker/api/types/swarm"
2021
"github.com/docker/docker/integration-cli/checker"
@@ -1221,10 +1222,6 @@ func (s *DockerSwarmSuite) TestSwarmJoinPromoteLocked(c *check.C) {
12211222
c.Assert(getNodeStatus(c, d), checker.Equals, swarm.LocalNodeStateActive)
12221223
}
12231224

1224-
// get d3's cert
1225-
d3cert, err := ioutil.ReadFile(filepath.Join(d3.Folder, "root", "swarm", "certificates", "swarm-node.crt"))
1226-
c.Assert(err, checker.IsNil)
1227-
12281225
// demote manager back to worker - workers are not locked
12291226
outs, err = d1.Cmd("node", "demote", d3.Info.NodeID)
12301227
c.Assert(err, checker.IsNil)
@@ -1237,12 +1234,16 @@ func (s *DockerSwarmSuite) TestSwarmJoinPromoteLocked(c *check.C) {
12371234
// is set to autolock)
12381235
waitAndAssert(c, defaultReconciliationTimeout, d3.CheckControlAvailable, checker.False)
12391236
waitAndAssert(c, defaultReconciliationTimeout, func(c *check.C) (interface{}, check.CommentInterface) {
1240-
cert, err := ioutil.ReadFile(filepath.Join(d3.Folder, "root", "swarm", "certificates", "swarm-node.crt"))
1237+
certBytes, err := ioutil.ReadFile(filepath.Join(d3.Folder, "root", "swarm", "certificates", "swarm-node.crt"))
12411238
if err != nil {
12421239
return "", check.Commentf("error: %v", err)
12431240
}
1244-
return string(cert), check.Commentf("cert: %v", string(cert))
1245-
}, checker.Not(checker.Equals), string(d3cert))
1241+
certs, err := helpers.ParseCertificatesPEM(certBytes)
1242+
if err == nil && len(certs) > 0 && len(certs[0].Subject.OrganizationalUnit) > 0 {
1243+
return certs[0].Subject.OrganizationalUnit[0], nil
1244+
}
1245+
return "", check.Commentf("could not get organizational unit from certificate")
1246+
}, checker.Equals, "swarm-worker")
12461247

12471248
// by now, it should *never* be locked on restart
12481249
d3.Restart(c)

vendor.conf

+1-1
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ github.com/docker/containerd 9048e5e50717ea4497b757314bad98ea3763c145
106106
github.com/tonistiigi/fifo 1405643975692217d6720f8b54aeee1bf2cd5cf4
107107

108108
# cluster
109-
github.com/docker/swarmkit b19d028de0a6e9ca281afeb76cea2544b9edd839
109+
github.com/docker/swarmkit 61a92e8ec074df5769decda985df4a3ab43c77eb
110110
github.com/gogo/protobuf 8d70fb3182befc465c4a1eac8ad4d38ff49778e2
111111
github.com/cloudflare/cfssl 7fb22c8cba7ecaf98e4082d22d65800cf45e042a
112112
github.com/google/certificate-transparency d90e65c3a07988180c5b1ece71791c0b6506826e

vendor/github.com/docker/swarmkit/node/node.go

+51-9
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)