Skip to content

Commit e8e0441

Browse files
authored
fix: ECR Keyless access - deps bumped and changed to k8schain (#78)
ECR Keyless access - deps bumped and changed to k8schain Switched to k8schain from Kubernetes pkg Signed-off-by: Rashed Kamal <[email protected]>
1 parent 45f22f3 commit e8e0441

File tree

7 files changed

+170
-153
lines changed

7 files changed

+170
-153
lines changed

.github/workflows/ci.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -474,7 +474,7 @@ jobs:
474474
kapp list -A
475475
echo "##[endgroup]"
476476
echo "##[group]Package changesets"
477-
kapp app-change list -a source-controller-ctrl
477+
kapp app-change list -a source-controller.app
478478
echo "##[endgroup]"
479479
echo "##[group]kapp-controller logs"
480480
kubectl logs -n kapp-controller -l app=kapp-controller -c kapp-controller --tail 1000
@@ -505,7 +505,7 @@ jobs:
505505
set -o nounset
506506
set -o pipefail
507507
508-
deploys=$(kapp app-change list -a source-controller-ctrl --json | jq '.Tables[0].Rows | length')
508+
deploys=$(kapp app-change list -a source-controller.app --json | jq '.Tables[0].Rows | length')
509509
if [ "$((deploys))" -gt 1 ]; then
510510
echo "Too many app changes for the source-controller package, expected 1 found ${deploys}"
511511
exit 1

.ko.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
defaultBaseImage: paketobuildpacks/run-jammy-tiny@sha256:35702d19f93e06041db1573b1140742df2182494cc93f646fd57c6d8922dc7a7
1+
defaultBaseImage: paketobuildpacks/run-jammy-tiny@sha256:c831f67d4fc0e8fcbcbd5afaae4d5fabe052248ce8482d6f36b1db76c5d78957

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ KAPP_ARGS ?= "--yes=false"
99
CONTROLLER_GEN ?= go run -modfile hack/go.mod sigs.k8s.io/controller-tools/cmd/controller-gen
1010
DIEGEN ?= go run -modfile hack/go.mod dies.dev/diegen
1111
GOIMPORTS ?= go run -modfile hack/go.mod golang.org/x/tools/cmd/goimports
12-
KAPP ?= go run -modfile hack/kapp/github.com/vmware-tanzu/carvel-kapp/cmd/kapp
12+
KAPP ?= go run -modfile hack/kapp/go.mod github.com/vmware-tanzu/carvel-kapp/cmd/kapp
1313
KO ?= go run -modfile hack/ko/go.mod github.com/google/ko
1414
KUSTOMIZE ?= go run -modfile hack/kustomize/go.mod sigs.k8s.io/kustomize/kustomize/v4
1515
YTT ?= go run -modfile hack/ytt/go.mod github.com/k14s/ytt/cmd/ytt

controllers/imagerepository_controller.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ import (
3232
"strings"
3333

3434
"github.com/go-logr/logr"
35-
"github.com/google/go-containerregistry/pkg/authn/kubernetes"
35+
"github.com/google/go-containerregistry/pkg/authn/k8schain"
3636
"github.com/google/go-containerregistry/pkg/name"
3737
"github.com/google/go-containerregistry/pkg/v1/remote"
3838
"github.com/vmware-labs/reconciler-runtime/apis"
@@ -171,7 +171,7 @@ func ImageRepositoryImageDigestSyncReconciler() reconcilers.SubReconciler {
171171
if pullSecrets == nil {
172172
return nil
173173
}
174-
keychain, err := kubernetes.NewFromPullSecrets(ctx, pullSecrets)
174+
keychain, err := k8schain.NewFromPullSecrets(ctx, pullSecrets)
175175
if err != nil {
176176
return err
177177
}
@@ -244,7 +244,7 @@ func ImageRepositoryPullImageSyncReconciler(httpRootDir, httpHost string, now fu
244244
if pullSecrets == nil {
245245
return nil
246246
}
247-
keychain, err := kubernetes.NewFromPullSecrets(ctx, pullSecrets)
247+
keychain, err := k8schain.NewFromPullSecrets(ctx, pullSecrets)
248248
if err != nil {
249249
return err
250250
}

go.mod

Lines changed: 47 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -7,76 +7,79 @@ require (
77
github.com/go-logr/logr v1.2.3
88
github.com/google/go-cmp v0.5.9
99
github.com/google/go-containerregistry v0.13.0
10-
github.com/google/go-containerregistry/pkg/authn/kubernetes v0.0.0-20230209165335-3624968304fd
10+
github.com/google/go-containerregistry/pkg/authn/k8schain v0.0.0-20230310164735-e94d40893b2d
1111
github.com/vmware-labs/reconciler-runtime v0.11.0
1212
github.com/vmware-tanzu/carvel-imgpkg v0.36.0
1313
go.uber.org/zap v1.24.0
14-
k8s.io/api v0.26.1
15-
k8s.io/apimachinery v0.26.1
16-
k8s.io/client-go v0.26.1
14+
k8s.io/api v0.26.2
15+
k8s.io/apimachinery v0.26.2
16+
k8s.io/client-go v0.26.2
1717
sigs.k8s.io/controller-runtime v0.14.4
1818
)
1919

2020
require (
21-
cloud.google.com/go/compute v1.14.0 // indirect
21+
cloud.google.com/go/compute v1.18.0 // indirect
2222
cloud.google.com/go/compute/metadata v0.2.3 // indirect
23-
github.com/Azure/azure-sdk-for-go v55.0.0+incompatible // indirect
23+
github.com/Azure/azure-sdk-for-go v68.0.0+incompatible // indirect
2424
github.com/Azure/go-autorest v14.2.0+incompatible // indirect
25-
github.com/Azure/go-autorest/autorest v0.11.27 // indirect
26-
github.com/Azure/go-autorest/autorest/adal v0.9.20 // indirect
27-
github.com/Azure/go-autorest/autorest/azure/auth v0.5.2 // indirect
28-
github.com/Azure/go-autorest/autorest/azure/cli v0.4.1 // indirect
25+
github.com/Azure/go-autorest/autorest v0.11.28 // indirect
26+
github.com/Azure/go-autorest/autorest/adal v0.9.22 // indirect
27+
github.com/Azure/go-autorest/autorest/azure/auth v0.5.12 // indirect
28+
github.com/Azure/go-autorest/autorest/azure/cli v0.4.6 // indirect
2929
github.com/Azure/go-autorest/autorest/date v0.3.0 // indirect
3030
github.com/Azure/go-autorest/logger v0.2.1 // indirect
3131
github.com/Azure/go-autorest/tracing v0.6.0 // indirect
3232
github.com/VividCortex/ewma v1.1.1 // indirect
33-
github.com/aws/aws-sdk-go-v2 v1.7.1 // indirect
34-
github.com/aws/aws-sdk-go-v2/config v1.5.0 // indirect
35-
github.com/aws/aws-sdk-go-v2/credentials v1.3.1 // indirect
36-
github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.3.0 // indirect
37-
github.com/aws/aws-sdk-go-v2/internal/ini v1.1.1 // indirect
38-
github.com/aws/aws-sdk-go-v2/service/ecr v1.4.1 // indirect
39-
github.com/aws/aws-sdk-go-v2/service/ecrpublic v1.4.1 // indirect
40-
github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.2.1 // indirect
41-
github.com/aws/aws-sdk-go-v2/service/sso v1.3.1 // indirect
42-
github.com/aws/aws-sdk-go-v2/service/sts v1.6.0 // indirect
43-
github.com/aws/smithy-go v1.6.0 // indirect
44-
github.com/awslabs/amazon-ecr-credential-helper/ecr-login v0.0.0-20220517224237-e6f29200ae04 // indirect
33+
github.com/aws/aws-sdk-go-v2 v1.17.5 // indirect
34+
github.com/aws/aws-sdk-go-v2/config v1.18.15 // indirect
35+
github.com/aws/aws-sdk-go-v2/credentials v1.13.15 // indirect
36+
github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.12.23 // indirect
37+
github.com/aws/aws-sdk-go-v2/internal/configsources v1.1.29 // indirect
38+
github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.4.23 // indirect
39+
github.com/aws/aws-sdk-go-v2/internal/ini v1.3.30 // indirect
40+
github.com/aws/aws-sdk-go-v2/service/ecr v1.18.5 // indirect
41+
github.com/aws/aws-sdk-go-v2/service/ecrpublic v1.15.4 // indirect
42+
github.com/aws/aws-sdk-go-v2/service/internal/presigned-url v1.9.23 // indirect
43+
github.com/aws/aws-sdk-go-v2/service/sso v1.12.4 // indirect
44+
github.com/aws/aws-sdk-go-v2/service/ssooidc v1.14.4 // indirect
45+
github.com/aws/aws-sdk-go-v2/service/sts v1.18.5 // indirect
46+
github.com/aws/smithy-go v1.13.5 // indirect
47+
github.com/awslabs/amazon-ecr-credential-helper/ecr-login v0.0.0-20230228174139-39c3d18f0af1 // indirect
4548
github.com/beorn7/perks v1.0.1 // indirect
4649
github.com/cespare/xxhash/v2 v2.1.2 // indirect
4750
github.com/cheggaaa/pb/v3 v3.1.0 // indirect
48-
github.com/chrismellard/docker-credential-acr-env v0.0.0-20220327082430-c57b701bfc08 // indirect
51+
github.com/chrismellard/docker-credential-acr-env v0.0.0-20230304212654-82a0ddb27589 // indirect
4952
github.com/containerd/stargz-snapshotter/estargz v0.12.1 // indirect
5053
github.com/cppforlife/color v1.9.1-0.20200716202919-6706ac40b835 // indirect
5154
github.com/cppforlife/go-cli-ui v0.0.0-20220425131040-94f26b16bc14 // indirect
5255
github.com/davecgh/go-spew v1.1.1 // indirect
53-
github.com/dimchansky/utfbom v1.1.0 // indirect
54-
github.com/docker/cli v20.10.20+incompatible // indirect
56+
github.com/dimchansky/utfbom v1.1.1 // indirect
57+
github.com/docker/cli v23.0.1+incompatible // indirect
5558
github.com/docker/distribution v2.8.1+incompatible // indirect
56-
github.com/docker/docker v20.10.20+incompatible // indirect
59+
github.com/docker/docker v23.0.1+incompatible // indirect
5760
github.com/docker/docker-credential-helpers v0.7.0 // indirect
58-
github.com/emicklei/go-restful/v3 v3.9.0 // indirect
61+
github.com/emicklei/go-restful/v3 v3.10.2 // indirect
5962
github.com/evanphx/json-patch v4.12.0+incompatible // indirect
6063
github.com/evanphx/json-patch/v5 v5.6.0 // indirect
6164
github.com/fatih/color v1.13.0 // indirect
6265
github.com/fsnotify/fsnotify v1.6.0 // indirect
6366
github.com/go-logr/zapr v1.2.3 // indirect
64-
github.com/go-openapi/jsonpointer v0.19.5 // indirect
65-
github.com/go-openapi/jsonreference v0.20.0 // indirect
67+
github.com/go-openapi/jsonpointer v0.19.6 // indirect
68+
github.com/go-openapi/jsonreference v0.20.2 // indirect
6669
github.com/go-openapi/swag v0.22.3 // indirect
6770
github.com/gogo/protobuf v1.3.2 // indirect
68-
github.com/golang-jwt/jwt/v4 v4.2.0 // indirect
71+
github.com/golang-jwt/jwt/v4 v4.5.0 // indirect
6972
github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect
70-
github.com/golang/protobuf v1.5.2 // indirect
73+
github.com/golang/protobuf v1.5.3 // indirect
7174
github.com/google/gnostic v0.6.9 // indirect
75+
github.com/google/go-containerregistry/pkg/authn/kubernetes v0.0.0-20230309011546-ff810c186c77 // indirect
7276
github.com/google/gofuzz v1.2.0 // indirect
7377
github.com/google/uuid v1.3.0 // indirect
7478
github.com/imdario/mergo v0.3.12 // indirect
7579
github.com/jmespath/go-jmespath v0.4.0 // indirect
7680
github.com/josharian/intern v1.0.0 // indirect
7781
github.com/json-iterator/go v1.1.12 // indirect
78-
github.com/klauspost/compress v1.15.11 // indirect
79-
github.com/kr/pretty v0.2.1 // indirect
82+
github.com/klauspost/compress v1.16.0 // indirect
8083
github.com/mailru/easyjson v0.7.7 // indirect
8184
github.com/mattn/go-colorable v0.1.12 // indirect
8285
github.com/mattn/go-isatty v0.0.17 // indirect
@@ -86,7 +89,6 @@ require (
8689
github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
8790
github.com/modern-go/reflect2 v1.0.2 // indirect
8891
github.com/munnerz/goautoneg v0.0.0-20191010083416-a7dc8b61c822 // indirect
89-
github.com/onsi/ginkgo v1.14.0 // indirect
9092
github.com/opencontainers/go-digest v1.0.0 // indirect
9193
github.com/opencontainers/image-spec v1.1.0-rc2 // indirect
9294
github.com/pkg/errors v0.9.1 // indirect
@@ -101,28 +103,28 @@ require (
101103
github.com/vito/go-interact v1.0.1 // indirect
102104
go.uber.org/atomic v1.7.0 // indirect
103105
go.uber.org/multierr v1.6.0 // indirect
104-
golang.org/x/crypto v0.1.0 // indirect
105-
golang.org/x/net v0.7.0 // indirect
106-
golang.org/x/oauth2 v0.1.0 // indirect
106+
golang.org/x/crypto v0.7.0 // indirect
107+
golang.org/x/net v0.8.0 // indirect
108+
golang.org/x/oauth2 v0.6.0 // indirect
107109
golang.org/x/sync v0.1.0 // indirect
108-
golang.org/x/sys v0.5.0 // indirect
109-
golang.org/x/term v0.5.0 // indirect
110-
golang.org/x/text v0.7.0 // indirect
110+
golang.org/x/sys v0.6.0 // indirect
111+
golang.org/x/term v0.6.0 // indirect
112+
golang.org/x/text v0.8.0 // indirect
111113
golang.org/x/time v0.3.0 // indirect
112114
gomodules.xyz/jsonpatch/v2 v2.2.0 // indirect
113115
gomodules.xyz/jsonpatch/v3 v3.0.1 // indirect
114116
gomodules.xyz/orderedmap v0.1.0 // indirect
115117
google.golang.org/appengine v1.6.7 // indirect
116-
google.golang.org/protobuf v1.28.1 // indirect
118+
google.golang.org/protobuf v1.29.0 // indirect
117119
gopkg.in/inf.v0 v0.9.1 // indirect
118120
gopkg.in/yaml.v2 v2.4.0 // indirect
119121
gopkg.in/yaml.v3 v3.0.1 // indirect
120122
k8s.io/apiextensions-apiserver v0.26.1 // indirect
121123
k8s.io/component-base v0.26.1 // indirect
122-
k8s.io/klog/v2 v2.80.1 // indirect
123-
k8s.io/kube-openapi v0.0.0-20221012153701-172d655c2280 // indirect
124-
k8s.io/utils v0.0.0-20221128185143-99ec85e7a448 // indirect
125-
sigs.k8s.io/json v0.0.0-20220713155537-f223a00ba0e2 // indirect
124+
k8s.io/klog/v2 v2.90.1 // indirect
125+
k8s.io/kube-openapi v0.0.0-20230308215209-15aac26d736a // indirect
126+
k8s.io/utils v0.0.0-20230308161112-d77c459e9343 // indirect
127+
sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd // indirect
126128
sigs.k8s.io/structured-merge-diff/v4 v4.2.3 // indirect
127129
sigs.k8s.io/yaml v1.3.0 // indirect
128130
)

0 commit comments

Comments
 (0)