Skip to content

Commit aa62a01

Browse files
authored
Merge pull request #156 from spowelljr/updateMocking
Use mock registry if `MOCK_GOOGLE_TOKEN` is true
2 parents fc2bfe7 + cdea91b commit aa62a01

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

server.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,19 +118,25 @@ func createPullSecret(clientset *kubernetes.Clientset, ns *corev1.Namespace, cre
118118
}
119119
}
120120

121-
// The MOCK_GOOGLE_TOKEN env var prevents using credentials to fetch the token. Instead the token will be mocked.
121+
registries := append(gcr_config.DefaultGCRRegistries[:], gcr_config.DefaultARRegistries[:]...)
122+
123+
// The MOCK_GOOGLE_TOKEN env var prevents using credentials to fetch the
124+
// token. Instead the token will be mocked. It also sets a mock registry
125+
// due to pulls to Artifact Registry for publicly available images with
126+
// mock credentials causing unauthorized errors. See:
127+
// https://github.com/kubernetes/minikube/issues/19714
122128
mockToken, _ := strconv.ParseBool(os.Getenv("MOCK_GOOGLE_TOKEN"))
123129
var token *oauth2.Token
124130
if mockToken {
125131
token = &oauth2.Token{AccessToken: "mock_access_token"}
132+
registries = []string{"mock-registry"}
126133
} else {
127134
token, err = creds.TokenSource.Token()
128135
if err != nil {
129136
return err
130137
}
131138
}
132139
var dockercfg string
133-
registries := append(gcr_config.DefaultGCRRegistries[:], gcr_config.DefaultARRegistries[:]...)
134140
for _, reg := range registries {
135141
dockercfg += fmt.Sprintf(`"https://%s":{"username":"oauth2accesstoken","password":"%s","email":"none"},`, reg, token.AccessToken)
136142
}

0 commit comments

Comments
 (0)