Container: Cannot connect to regional cluster about python-container

[tseaver]

@mholleville Thanks for the report. This is due to a strangeness in the proto file for the Cluster service, which deprecates the three original atttributes of the ListNodePoolsRequest message (project_id, zone, cluster_id) and makes the fourth one (parent) required.

As a workaround, you can pass None for the three deprecated arguments, e.g.:

parent = '/projects/{}/locations/{}/clusters/{}'.format(project_id, zone, cluster_id)
pool_iter = client.list_node_pools(parent=parent, project_id=None, zone=None, cluster_id=None)
for pool in pool_iter:
   do_something_with(pool)

@lukesneeringer I don't understand how that is supposed to work, but it definitely trips up our code generator: we generated the deprecated fields as required positional arguments! This issue actually infects pretty much every method of the Python container client.

[mholleville]

Trying but google.api_core.exceptions.InvalidArgument: 400 Location "locations" does not exist. :(

[lukesneeringer]

The location should probably be either global or a specific location (e.g. us-central-1).

[tseaver]

@noahdietz, can you recommend a way forward? It looks like the GAPIC YAML file is b0rked.

[tseaver]

A little bird says maybe I should ask @dazuma instead for help here.

[dijitali]

@mholleville - I think there might have been a typo in the comment above (slash before projects), this works for me with both a regional and zonal cluster:

parent = 'projects/{}/locations/{}/clusters/{}'.format(project_id, location, cluster_id)
print (client.list_node_pools(parent=parent, project_id=None, zone=None, cluster_id=None))

[askmeegs]

@mholleville - I think there might have been a typo in the comment above (slash before projects), this works for me with both a regional and zonal cluster:

parent = 'projects/{}/locations/{}/clusters/{}'.format(project_id, location, cluster_id)
print (client.list_node_pools(parent=parent, project_id=None, zone=None, cluster_id=None))

Following up on this issue - dijitali@'s example is working for me, google_cloud_container 2.4.0, with us-central1 set as the location --

from google.cloud.container_v1.services.cluster_manager import ClusterManagerClient
from google.cloud.container_v1.types import cluster_service
import google.auth

credentials, project_id = google.auth.default()
location = "us-central1"
cluster_id="regional-cluster"

client = ClusterManagerClient(credentials=credentials)

parent = 'projects/{}/locations/{}/clusters/{}'.format(project_id, location, cluster_id)
print (client.list_node_pools(parent=parent, project_id=None, zone=None, cluster_id=None))

Output:

node_pools {
  name: "default-pool"
  config {
    machine_type: "e2-medium"
...

Closing this issue given that there's a suitable workaround for the deprecated fields.