Skip to content

Commit

Permalink
Merge pull request #367 from fluxcd/oci-autologin-flag-log
Browse files Browse the repository at this point in the history
oci/auth: Remove log message about autologin flag
  • Loading branch information
stefanprodan authored Oct 11, 2022
2 parents d29f3fb + ba6c11f commit 4078e4f
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion oci/auth/aws/auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,5 @@ func (c *Client) Login(ctx context.Context, autoLogin bool, image string) (authn
auth := authn.FromConfig(authConfig)
return auth, nil
}
ctrl.LoggerFrom(ctx).Info("ECR authentication is not enabled. To enable, set the controller flag --aws-autologin-for-ecr")
return nil, fmt.Errorf("ECR authentication failed: %w", oci.ErrUnconfiguredProvider)
}
1 change: 0 additions & 1 deletion oci/auth/azure/auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,5 @@ func (c *Client) Login(ctx context.Context, autoLogin bool, image string, ref na
auth := authn.FromConfig(authConfig)
return auth, nil
}
ctrl.LoggerFrom(ctx).Info("ACR authentication is not enabled. To enable, set the controller flag --azure-autologin-for-acr")
return nil, fmt.Errorf("ACR authentication failed: %w", oci.ErrUnconfiguredProvider)
}
1 change: 0 additions & 1 deletion oci/auth/gcp/auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,5 @@ func (c *Client) Login(ctx context.Context, autoLogin bool, image string, ref na
auth := authn.FromConfig(authConfig)
return auth, nil
}
ctrl.LoggerFrom(ctx).Info("GCR authentication is not enabled. To enable, set the controller flag --gcp-autologin-for-gcr")
return nil, fmt.Errorf("GCR authentication failed: %w", oci.ErrUnconfiguredProvider)
}

0 comments on commit 4078e4f

Please sign in to comment.