Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

Commit

Permalink
Fix missing vpc id in Alicloud infra controller
Browse files Browse the repository at this point in the history
```improvement user
An issue that prevent the Alicloud infrastructure controller from reconciling the infrastructure in rare cases has been fixed.
```
  • Loading branch information
rfranzke committed Jun 4, 2019
1 parent 26b6bef commit e243969
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ import (

extensionsv1alpha1 "github.com/gardener/gardener/pkg/apis/extensions/v1alpha1"
"github.com/gardener/gardener/pkg/chartrenderer"
"github.com/gardener/gardener/pkg/operation/terraformer"
"github.com/go-logr/logr"
apierrors "k8s.io/apimachinery/pkg/api/errors"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
Expand Down Expand Up @@ -142,7 +143,7 @@ func (a *actuator) getConfigAndCredentialsForInfra(ctx context.Context, infra *e
func (a *actuator) fetchEIPInternetChargeType(vpcClient alicloudclient.VPC, tf extensionsterraformer.Interface) (string, error) {
stateVariables, err := tf.GetStateOutputVariables(TerraformerOutputKeyVPCID)
if err != nil {
if apierrors.IsNotFound(err) {
if apierrors.IsNotFound(err) || terraformer.IsVariablesNotFoundError(err) {
return alicloudclient.DefaultInternetChargeType, nil
}
return "", err
Expand Down

0 comments on commit e243969

Please sign in to comment.