Credential provider Provide takes image (clouds)

k3s-v1.15.3
tiffany jernigan 2019-03-22 07:50:05 +00:00
parent 847cb24aa1
commit ecbb090f32
6 changed files with 15 additions and 15 deletions

View File

@ -173,7 +173,7 @@ func (a *acrProvider) Enabled() bool {
return true
}
func (a *acrProvider) Provide() credentialprovider.DockerConfig {
func (a *acrProvider) Provide(image string) credentialprovider.DockerConfig {
cfg := credentialprovider.DockerConfig{}
ctx, cancel := getContextWithCancel()
defer cancel()
@ -246,6 +246,6 @@ func getACRDockerEntryFromARMToken(a *acrProvider, loginServer string) (*credent
}, nil
}
func (a *acrProvider) LazyProvide() *credentialprovider.DockerConfigEntry {
func (a *acrProvider) LazyProvide(image string) *credentialprovider.DockerConfigEntry {
return nil
}

View File

@ -74,7 +74,7 @@ func Test(t *testing.T) {
}
provider.loadConfig(bytes.NewBufferString(configStr))
creds := provider.Provide()
creds := provider.Provide("")
if len(creds) != len(result)+1 {
t.Errorf("Unexpected list: %v, expected length %d", creds, len(result)+1)

View File

@ -130,12 +130,12 @@ func (g *metadataProvider) Enabled() bool {
}
// LazyProvide implements DockerConfigProvider. Should never be called.
func (g *dockerConfigKeyProvider) LazyProvide() *credentialprovider.DockerConfigEntry {
func (g *dockerConfigKeyProvider) LazyProvide(image string) *credentialprovider.DockerConfigEntry {
return nil
}
// Provide implements DockerConfigProvider
func (g *dockerConfigKeyProvider) Provide() credentialprovider.DockerConfig {
func (g *dockerConfigKeyProvider) Provide(image string) credentialprovider.DockerConfig {
// Read the contents of the google-dockercfg metadata key and
// parse them as an alternate .dockercfg
if cfg, err := credentialprovider.ReadDockerConfigFileFromUrl(dockerConfigKey, g.Client, metadataHeader); err != nil {
@ -148,12 +148,12 @@ func (g *dockerConfigKeyProvider) Provide() credentialprovider.DockerConfig {
}
// LazyProvide implements DockerConfigProvider. Should never be called.
func (g *dockerConfigUrlKeyProvider) LazyProvide() *credentialprovider.DockerConfigEntry {
func (g *dockerConfigUrlKeyProvider) LazyProvide(image string) *credentialprovider.DockerConfigEntry {
return nil
}
// Provide implements DockerConfigProvider
func (g *dockerConfigUrlKeyProvider) Provide() credentialprovider.DockerConfig {
func (g *dockerConfigUrlKeyProvider) Provide(image string) credentialprovider.DockerConfig {
// Read the contents of the google-dockercfg-url key and load a .dockercfg from there
if url, err := credentialprovider.ReadUrl(dockerConfigUrlKey, g.Client, metadataHeader); err != nil {
klog.Errorf("while reading 'google-dockercfg-url' metadata: %v", err)
@ -258,12 +258,12 @@ type tokenBlob struct {
}
// LazyProvide implements DockerConfigProvider. Should never be called.
func (g *containerRegistryProvider) LazyProvide() *credentialprovider.DockerConfigEntry {
func (g *containerRegistryProvider) LazyProvide(image string) *credentialprovider.DockerConfigEntry {
return nil
}
// Provide implements DockerConfigProvider
func (g *containerRegistryProvider) Provide() credentialprovider.DockerConfig {
func (g *containerRegistryProvider) Provide(image string) credentialprovider.DockerConfig {
cfg := credentialprovider.DockerConfig{}
tokenJsonBlob, err := credentialprovider.ReadUrl(metadataToken, g.Client, metadataHeader)

View File

@ -91,7 +91,7 @@ func TestDockerKeyringFromGoogleDockerConfigMetadata(t *testing.T) {
t.Errorf("Provider is unexpectedly disabled")
}
keyring.Add(provider.Provide())
keyring.Add(provider.Provide(""))
creds, ok := keyring.Lookup(registryUrl)
if !ok {
@ -169,7 +169,7 @@ func TestDockerKeyringFromGoogleDockerConfigMetadataUrl(t *testing.T) {
t.Errorf("Provider is unexpectedly disabled")
}
keyring.Add(provider.Provide())
keyring.Add(provider.Provide(""))
creds, ok := keyring.Lookup(registryUrl)
if !ok {
@ -253,7 +253,7 @@ func TestContainerRegistryBasics(t *testing.T) {
t.Errorf("Provider is unexpectedly disabled")
}
keyring.Add(provider.Provide())
keyring.Add(provider.Provide(""))
creds, ok := keyring.Lookup(registryUrl)
if !ok {

View File

@ -79,12 +79,12 @@ func (p *rancherProvider) Enabled() bool {
}
// LazyProvide implements DockerConfigProvider. Should never be called.
func (p *rancherProvider) LazyProvide() *credentialprovider.DockerConfigEntry {
func (p *rancherProvider) LazyProvide(image string) *credentialprovider.DockerConfigEntry {
return nil
}
// Provide implements DockerConfigProvider.Provide, refreshing Rancher tokens on demand
func (p *rancherProvider) Provide() credentialprovider.DockerConfig {
func (p *rancherProvider) Provide(image string) credentialprovider.DockerConfig {
cfg := credentialprovider.DockerConfig{}
for _, cred := range p.credGetter.getCredentials() {
entry := credentialprovider.DockerConfigEntry{

View File

@ -82,7 +82,7 @@ func TestRancherCredentialsProvide(t *testing.T) {
}
keyring := &credentialprovider.BasicDockerKeyring{}
keyring.Add(provider.Provide())
keyring.Add(provider.Provide(""))
for _, registry := range serverAddresses {
fullImagePath := path.Join(registry, image)