Skip to content

Commit

Permalink
Merge pull request #6494 from Shubham82/fix_cloudproviders_builder
Browse files Browse the repository at this point in the history 
Add informer argument to the CloudProviders builder.
  • Loading branch information
@k8s-ci-robot
k8s-ci-robot committed Mar 6, 2024
2 parents 9dc3b9e + bbda0ba commit 4383f4c
Show file tree
Hide file tree
Showing 28 changed files with 56 additions and 28 deletions.
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_alicloud.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/alicloud"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for alicloud-only build is alicloud.
const DefaultCloudProvider = cloudprovider.AlicloudProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.AlicloudProviderName:
return alicloud.BuildAlicloud(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_aws.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/aws"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for AWS-only build is AWS.
const DefaultCloudProvider = cloudprovider.AwsProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.AwsProviderName:
return aws.BuildAWS(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_azure.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/azure"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider on Azure-only build is Azure.
const DefaultCloudProvider = cloudprovider.AzureProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.AzureProviderName:
return azure.BuildAzure(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_baiducloud.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/baiducloud"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for baiducloud-only build is baiducloud.
const DefaultCloudProvider = cloudprovider.BaiducloudProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.BaiducloudProviderName:
return baiducloud.BuildBaiducloud(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_bizflycloud.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/bizflycloud"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the Bizflycloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider build is Bizflycloud..
const DefaultCloudProvider = cloudprovider.BizflyCloudProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.BizflyCloudProviderName:
return bizflycloud.BuildBizflyCloud(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_brightbox.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/brightbox"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the brightbox cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider is Brightbox
const DefaultCloudProvider = cloudprovider.BrightboxProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.BrightboxProviderName:
return brightbox.BuildBrightbox(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_cherry.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
cherry "k8s.io/autoscaler/cluster-autoscaler/cloudprovider/cherryservers"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for Cherry-only build is Cherry
const DefaultCloudProvider = cherry.ProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cherry.ProviderName:
return cherry.BuildCherry(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_civo.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/civo"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the digtalocean cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for civo-only build is Civo.
const DefaultCloudProvider = cloudprovider.CivoProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.CivoProviderName:
return civo.BuildCivo(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_cloudstack.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/cloudstack"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for cloudstack-only build is cloudstack.
const DefaultCloudProvider = cloudprovider.CloudStackProviderName

func BuildCloudStack(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func BuildCloudStack(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.CloudStackProviderName:
return cloudstack.BuildCloudStack(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_digitalocean.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/digitalocean"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the digtalocean cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for do-only build is DigitalOcean.
const DefaultCloudProvider = cloudprovider.DigitalOceanProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.DigitalOceanProviderName:
return digitalocean.BuildDigitalOcean(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_exoscale.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/exoscale"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the Exoscale cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider is Exoscale.
const DefaultCloudProvider = cloudprovider.ExoscaleProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.ExoscaleProviderName:
return exoscale.BuildExoscale(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_externalgrpc.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/externalgrpc"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for externalgrpc-only build is externalgrpc.
const DefaultCloudProvider = cloudprovider.ExternalGrpcProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.ExternalGrpcProviderName:
return externalgrpc.BuildExternalGrpc(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_gce.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/gce"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider is GCE.
const DefaultCloudProvider = cloudprovider.GceProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.GceProviderName:
return gce.BuildGCE(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_hetzner.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/hetzner"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the Hetzner cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider is Hetzner.
const DefaultCloudProvider = cloudprovider.HetznerProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.HetznerProviderName:
return hetzner.BuildHetzner(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_huaweicloud.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/huaweicloud"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for huaweicloud-only build is huaweicloud.
const DefaultCloudProvider = cloudprovider.HuaweicloudProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.HuaweicloudProviderName:
return huaweicloud.BuildHuaweiCloud(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_ionoscloud.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/ionoscloud"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for IonosCloud-only build is ionoscloud.
const DefaultCloudProvider = cloudprovider.IonoscloudProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.IonoscloudProviderName:
return ionoscloud.BuildIonosCloud(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_kamatera.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/kamatera"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the Kamaterea cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider is Kamatera.
const DefaultCloudProvider = cloudprovider.KamateraProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.KamateraProviderName:
return kamatera.BuildKamatera(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_kubemark.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/kubemark"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for Kubemark-only build is Kubemark.
const DefaultCloudProvider = cloudprovider.KubemarkProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.KubemarkProviderName:
return kubemark.BuildKubemark(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_linode.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/linode"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for linode-only build is linode.
const DefaultCloudProvider = cloudprovider.LinodeProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.LinodeProviderName:
return linode.BuildLinode(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_magnum.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider/magnum"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for OpenStack-only build is OpenStack.
const DefaultCloudProvider = cloudprovider.MagnumProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.MagnumProviderName:
return magnum.BuildMagnum(opts, do, rl)
Expand Down
3 changes: 2 additions & 1 deletion cluster-autoscaler/cloudprovider/builder/builder_oci.go
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import (
"k8s.io/autoscaler/cluster-autoscaler/cloudprovider"
oci "k8s.io/autoscaler/cluster-autoscaler/cloudprovider/oci/instancepools"
"k8s.io/autoscaler/cluster-autoscaler/config"
"k8s.io/client-go/informers"
)

// AvailableCloudProviders supported by the cloud provider builder.
Expand All @@ -33,7 +34,7 @@ var AvailableCloudProviders = []string{
// DefaultCloudProvider for oci-only build is oci.
const DefaultCloudProvider = cloudprovider.OracleCloudProviderName

func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter) cloudprovider.CloudProvider {
func buildCloudProvider(opts config.AutoscalingOptions, do cloudprovider.NodeGroupDiscoveryOptions, rl *cloudprovider.ResourceLimiter, _ informers.SharedInformerFactory) cloudprovider.CloudProvider {
switch opts.CloudProviderName {
case cloudprovider.OracleCloudProviderName:
return oci.BuildOCI(opts, do, rl)
Expand Down

0 comments on commit 4383f4c

Please sign in to comment.