Skip to content

Commit

Permalink
Skip grpc in bump-deps.sh
Browse files Browse the repository at this point in the history
The latest version of grpc is not compatible with the version containerd
is using. Skip automatic updates.

This change also stops using the `-u` flag in `go get`. The flag causes
`go get` to update transitive depdendencies which means that grpc was
getting update whenever we tried to update containerd.

Signed-off-by: Kern Walster <[email protected]>
  • Loading branch information
Kern-- committed Sep 7, 2023
1 parent e464dda commit fe6994c
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
6 changes: 5 additions & 1 deletion .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,10 @@ updates:
schedule:
interval: "daily"
ignore:
# skip grpc because the current latest is not compatible with containerd 1.7
# skip k8s deps since they use the latest go version/features that may not be in the go version soci uses
# Also ignored in /scripts/bump-deps.sh
- dependency-name: "google.golang.org/grpc"
- dependency-name: "k8s.io/*"

# Automatic upgrade for go modules of cmd package.
Expand All @@ -17,9 +19,11 @@ updates:
schedule:
interval: "daily"
ignore:
# skip k8s deps and soci-snapshotter itself
# skip grpc because the current latest is not compatible with containerd 1.7
# skip k8s deps since they use the latest go version/features that may not be in the go version soci uses
# Also ignored in /scripts/bump-deps.sh
- dependency-name: "github.com/awslabs/soci-snapshotter"
- dependency-name: "google.golang.org/grpc"
- dependency-name: "k8s.io/*"

# Automatic upgrade for base images used in the Dockerfile
Expand Down
8 changes: 6 additions & 2 deletions scripts/bump-deps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -22,16 +22,20 @@ SOCI_SNAPSHOTTER_PROJECT_ROOT="${CUR_DIR}/.."
pushd ${SOCI_SNAPSHOTTER_PROJECT_ROOT}

# skip k8s deps since they use the latest go version/features that may not be in the go version soci uses
# skip grpc because it's not compatible with containerd 1.7
# Also ignored in /dependabot.yml
go get -u $(go list -m -f '{{if not (or .Indirect .Main)}}{{.Path}}{{end}}' all | \
go get $(go list -m -f '{{if not (or .Indirect .Main)}}{{.Path}}{{end}}' all | \
grep -v "^google.golang.org/grpc" | \
grep -v "^k8s.io/")
make vendor

pushd ./cmd
# skip k8s deps and soci-snapshotter itself
# skip grpc because it's not compatible with containerd 1.7
# Also ignored in /dependabot.yml
go get -u $(go list -m -f '{{if not (or .Indirect .Main)}}{{.Path}}{{end}}' all | \
go get $(go list -m -f '{{if not (or .Indirect .Main)}}{{.Path}}{{end}}' all | \
grep -v "^github.com/awslabs/soci-snapshotter" | \
grep -v "^google.golang.org/grpc" | \
grep -v "^k8s.io/")
popd
make vendor
Expand Down

0 comments on commit fe6994c

Please sign in to comment.