Skip to content

Commit

Permalink
Bump the docker group across 1 directory with 3 updates (#316)
Browse files Browse the repository at this point in the history
* Bump the docker group across 1 directory with 3 updates

Bumps the docker group with 3 updates in the / directory: [github.com/docker/cli](https://github.com/docker/cli), [github.com/docker/docker](https://github.com/docker/docker) and [github.com/moby/moby](https://github.com/moby/moby).


Updates `github.com/docker/cli` from 25.0.4+incompatible to 26.1.1+incompatible
- [Commits](docker/cli@v25.0.4...v26.1.1)

Updates `github.com/docker/docker` from 25.0.5+incompatible to 26.1.1+incompatible
- [Release notes](https://github.com/docker/docker/releases)
- [Commits](moby/moby@v25.0.5...v26.1.1)

Updates `github.com/moby/moby` from 25.0.4+incompatible to 26.1.1+incompatible
- [Release notes](https://github.com/moby/moby/releases)
- [Commits](moby/moby@v25.0.4...v26.1.1)

---
updated-dependencies:
- dependency-name: github.com/docker/cli
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: docker
- dependency-name: github.com/docker/docker
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: docker
- dependency-name: github.com/moby/moby
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: docker
...

Signed-off-by: dependabot[bot] <[email protected]>

* fix breaking changes

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jake Coffman <[email protected]>
  • Loading branch information
dependabot[bot] and jakecoffman authored May 8, 2024
1 parent c200aad commit 4b82dea
Show file tree
Hide file tree
Showing 6 changed files with 22 additions and 19 deletions.
9 changes: 5 additions & 4 deletions go.mod
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
module github.com/dependabot/cli

go 1.22.1
go 1.22.3

require (
github.com/MakeNowJust/heredoc v1.0.0
github.com/docker/cli v25.0.4+incompatible
github.com/docker/docker v25.0.5+incompatible
github.com/docker/cli v26.1.1+incompatible
github.com/docker/docker v26.1.1+incompatible
github.com/goware/prefixer v0.0.0-20160118172347-395022866408
github.com/hexops/gotextdiff v1.0.3
github.com/moby/moby v25.0.4+incompatible
github.com/moby/moby v26.1.1+incompatible
github.com/moby/sys/signal v0.7.0
github.com/spf13/cobra v1.8.0
gopkg.in/yaml.v3 v3.0.1
Expand All @@ -30,6 +30,7 @@ require (
github.com/inconshreveable/mousetrap v1.1.0 // indirect
github.com/klauspost/compress v1.17.8 // indirect
github.com/kr/text v0.2.0 // indirect
github.com/moby/docker-image-spec v1.3.1 // indirect
github.com/moby/patternmatcher v0.6.0 // indirect
github.com/moby/sys/sequential v0.5.0 // indirect
github.com/moby/sys/user v0.1.0 // indirect
Expand Down
14 changes: 8 additions & 6 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@ github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c
github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
github.com/distribution/reference v0.6.0 h1:0IXCQ5g4/QMHHkarYzh5l+u8T3t73zM5QvfrDyIgxBk=
github.com/distribution/reference v0.6.0/go.mod h1:BbU0aIcezP1/5jX/8MP0YiH4SdvB5Y4f/wlDRiLyi3E=
github.com/docker/cli v25.0.4+incompatible h1:DatRkJ+nrFoYL2HZUzjM5Z5sAmcA5XGp+AW0oEw2+cA=
github.com/docker/cli v25.0.4+incompatible/go.mod h1:JLrzqnKDaYBop7H2jaqPtU4hHvMKP+vjCwu2uszcLI8=
github.com/docker/docker v25.0.5+incompatible h1:UmQydMduGkrD5nQde1mecF/YnSbTOaPeFIeP5C4W+DE=
github.com/docker/docker v25.0.5+incompatible/go.mod h1:eEKB0N0r5NX/I1kEveEz05bcu8tLC/8azJZsviup8Sk=
github.com/docker/cli v26.1.1+incompatible h1:bE1/uE2tCa08fMv+7ikLR/RDPoCqytwrLtkIkSzxLvw=
github.com/docker/cli v26.1.1+incompatible/go.mod h1:JLrzqnKDaYBop7H2jaqPtU4hHvMKP+vjCwu2uszcLI8=
github.com/docker/docker v26.1.1+incompatible h1:oI+4kkAgIwwb54b9OC7Xc3hSgu1RlJA/Lln/DF72djQ=
github.com/docker/docker v26.1.1+incompatible/go.mod h1:eEKB0N0r5NX/I1kEveEz05bcu8tLC/8azJZsviup8Sk=
github.com/docker/go-connections v0.5.0 h1:USnMq7hx7gwdVZq1L49hLXaFtUdTADjXGp+uj1Br63c=
github.com/docker/go-connections v0.5.0/go.mod h1:ov60Kzw0kKElRwhNs9UlUHAE/F9Fe6GLaXnqyDdmEXc=
github.com/docker/go-units v0.5.0 h1:69rxXcBk27SvSaaxTtLh/8llcHD8vYHT7WSdRZ/jvr4=
Expand Down Expand Up @@ -60,8 +60,10 @@ github.com/kr/pty v1.1.1/go.mod h1:pFQYn66WHrOpPYNljwOMqo10TkYh1fy3cYio2l3bCsQ=
github.com/kr/text v0.1.0/go.mod h1:4Jbv+DJW3UT/LiOwJeYQe1efqtUx/iVham/4vfdArNI=
github.com/kr/text v0.2.0 h1:5Nx0Ya0ZqY2ygV366QzturHI13Jq95ApcVaJBhpS+AY=
github.com/kr/text v0.2.0/go.mod h1:eLer722TekiGuMkidMxC/pM04lWEeraHUUmBw8l2grE=
github.com/moby/moby v25.0.4+incompatible h1:vea1J80wDM5x5geaZSaywFkfFxLABJIQ3mmR4ewZGbU=
github.com/moby/moby v25.0.4+incompatible/go.mod h1:fDXVQ6+S340veQPv35CzDahGBmHsiclFwfEygB/TWMc=
github.com/moby/docker-image-spec v1.3.1 h1:jMKff3w6PgbfSa69GfNg+zN/XLhfXJGnEx3Nl2EsFP0=
github.com/moby/docker-image-spec v1.3.1/go.mod h1:eKmb5VW8vQEh/BAr2yvVNvuiJuY6UIocYsFu/DxxRpo=
github.com/moby/moby v26.1.1+incompatible h1:p86pRiZUnIQZQsHwigk0tK78A38U6xigJFC05Gq8yDY=
github.com/moby/moby v26.1.1+incompatible/go.mod h1:fDXVQ6+S340veQPv35CzDahGBmHsiclFwfEygB/TWMc=
github.com/moby/patternmatcher v0.6.0 h1:GmP9lR19aU5GqSSFko+5pRqHi+Ohk1O69aFiKkVGiPk=
github.com/moby/patternmatcher v0.6.0/go.mod h1:hDPoyOpDY7OrrMDLaYoY3hf52gNCR/YOUYxkhApJIxc=
github.com/moby/sys/sequential v0.5.0 h1:OPvI35Lzn9K04PBbCLW0g4LcFAJgHsvXsRyewg5lXtc=
Expand Down
4 changes: 2 additions & 2 deletions internal/infra/open_telemetry.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ func NewCollector(ctx context.Context, cli *client.Client, net *Networks, params
return nil, fmt.Errorf("failed to copy cert to container: %w", err)
}

if err = cli.ContainerStart(ctx, collectorContainer.ID, types.ContainerStartOptions{}); err != nil {
if err = cli.ContainerStart(ctx, collectorContainer.ID, container.StartOptions{}); err != nil {
collector.Close()
return nil, fmt.Errorf("failed to start collector container: %w", err)
}
Expand All @@ -109,7 +109,7 @@ func (c *Collector) Close() error {
timeout := 30
_ = c.cli.ContainerStop(context.Background(), c.containerID, container.StopOptions{Timeout: &timeout})

err := c.cli.ContainerRemove(context.Background(), c.containerID, types.ContainerRemoveOptions{Force: true})
err := c.cli.ContainerRemove(context.Background(), c.containerID, container.RemoveOptions{Force: true})
if err != nil {
return fmt.Errorf("failed to remove collector container: %w", err)
}
Expand Down
6 changes: 3 additions & 3 deletions internal/infra/proxy.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ func NewProxy(ctx context.Context, cli *client.Client, params *RunParams, nets *
}
}

if err = cli.ContainerStart(ctx, proxyContainer.ID, types.ContainerStartOptions{}); err != nil {
if err = cli.ContainerStart(ctx, proxyContainer.ID, container.StartOptions{}); err != nil {
_ = proxy.Close()
return nil, fmt.Errorf("failed to start container: %w", err)
}
Expand Down Expand Up @@ -152,7 +152,7 @@ func putProxyConfig(ctx context.Context, cli *client.Client, config *Config, id
}

func (p *Proxy) TailLogs(ctx context.Context, cli *client.Client) {
out, err := cli.ContainerLogs(ctx, p.containerID, types.ContainerLogsOptions{
out, err := cli.ContainerLogs(ctx, p.containerID, container.LogsOptions{
ShowStdout: true,
ShowStderr: true,
Follow: true,
Expand All @@ -170,7 +170,7 @@ func (p *Proxy) TailLogs(ctx context.Context, cli *client.Client) {

func (p *Proxy) Close() (err error) {
defer func() {
removeErr := p.cli.ContainerRemove(context.Background(), p.containerID, types.ContainerRemoveOptions{Force: true})
removeErr := p.cli.ContainerRemove(context.Background(), p.containerID, container.RemoveOptions{Force: true})
if removeErr != nil {
err = fmt.Errorf("failed to remove proxy container: %w", removeErr)
}
Expand Down
4 changes: 2 additions & 2 deletions internal/infra/tty.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,14 @@ package infra

import (
"context"
"github.com/docker/docker/api/types/container"
"log"
"os"
gosignal "os/signal"
"runtime"
"time"

"github.com/docker/cli/cli/streams"
"github.com/docker/docker/api/types"
"github.com/moby/moby/client"
"github.com/moby/sys/signal"
)
Expand All @@ -23,7 +23,7 @@ func resizeTtyTo(ctx context.Context, c *client.Client, id string, height, width
return nil
}

options := types.ResizeOptions{
options := container.ResizeOptions{
Height: height,
Width: width,
}
Expand Down
4 changes: 2 additions & 2 deletions internal/infra/updater.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ func NewUpdater(ctx context.Context, cli *client.Client, net *Networks, params *
return nil, err
}

if err = cli.ContainerStart(ctx, updaterContainer.ID, types.ContainerStartOptions{}); err != nil {
if err = cli.ContainerStart(ctx, updaterContainer.ID, container.StartOptions{}); err != nil {
updater.Close()
return nil, fmt.Errorf("failed to start updater container: %w", err)
}
Expand Down Expand Up @@ -295,7 +295,7 @@ func (u *Updater) Wait(ctx context.Context, condition container.WaitCondition) e
// Close kills and deletes the container and deletes updater mount paths related to the run.
func (u *Updater) Close() (err error) {
defer func() {
removeErr := u.cli.ContainerRemove(context.Background(), u.containerID, types.ContainerRemoveOptions{Force: true})
removeErr := u.cli.ContainerRemove(context.Background(), u.containerID, container.RemoveOptions{Force: true})
if removeErr != nil {
err = fmt.Errorf("failed to remove proxy container: %w", removeErr)
}
Expand Down

0 comments on commit 4b82dea

Please sign in to comment.