From d05048cedd56842f3ef1defec625c11b17149664 Mon Sep 17 00:00:00 2001 From: Patrick Decat Date: Sat, 19 Oct 2024 17:27:11 +0200 Subject: [PATCH] fix: instances column of gcp_compute_instance_group errors for regional groups --- gcp/table_gcp_compute_instance_group.go | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/gcp/table_gcp_compute_instance_group.go b/gcp/table_gcp_compute_instance_group.go index fb4b5357..d86888bc 100644 --- a/gcp/table_gcp_compute_instance_group.go +++ b/gcp/table_gcp_compute_instance_group.go @@ -259,14 +259,21 @@ func getComputeInstanceGroupInstances(ctx context.Context, d *plugin.QueryData, return nil, err } - resp, err := service.InstanceGroups.ListInstances(project, getLastPathElement(types.SafeString(instanceGroup.Zone)), instanceGroup.Name, &compute.InstanceGroupsListInstancesRequest{}).Do() - - if err != nil { - plugin.Logger(ctx).Error("gcp_compute_instance_group.getComputeInstanceGroupInstances", "api_err", err) - return nil, err + if instanceGroup.Zone == "" { + resp, err := service.RegionInstanceGroups.ListInstances(project, getLastPathElement(types.SafeString(instanceGroup.Region)), instanceGroup.Name, &compute.RegionInstanceGroupsListInstancesRequest{}).Do() + if err != nil { + plugin.Logger(ctx).Error("gcp_compute_instance_group.getComputeInstanceGroupInstances", "api_err", err) + return nil, err + } + return &resp.Items, nil + } else { + resp, err := service.InstanceGroups.ListInstances(project, getLastPathElement(types.SafeString(instanceGroup.Zone)), instanceGroup.Name, &compute.InstanceGroupsListInstancesRequest{}).Do() + if err != nil { + plugin.Logger(ctx).Error("gcp_compute_instance_group.getComputeInstanceGroupInstances", "api_err", err) + return nil, err + } + return &resp.Items, nil } - - return &resp.Items, nil } //// TRANSFORM FUNCTIONS