feat: [google-cloud-os-login] added field ImportSshPublicKeyRequest.regions (#12168)

gcf-owl-bot[bot] · web-flow · commit d30f83d88766 · 2024-01-09T08:37:05.000-05:00
- [ ] Regenerate this pull request now. PiperOrigin-RevId: 596735856 Source-Link: googleapis/googleapis@87e7305 Source-Link: googleapis/googleapis-gen@114ca4d Copy-Tag: eyJwIjoicGFja2FnZXMvZ29vZ2xlLWNsb3VkLW9zLWxvZ2luLy5Pd2xCb3QueWFtbCIsImgiOiIxMTRjYTRkMzc2ZTg0NTMzMTUwMDg4ZGY4YWM4NmUwOTgyNDFlOTY2In0= BEGIN_NESTED_COMMIT feat: [google-cloud-os-login] add regions field to ImportSshPublicKeyRequest PiperOrigin-RevId: 596667704 Source-Link: googleapis/googleapis@c340584 Source-Link: googleapis/googleapis-gen@53aa884 Copy-Tag: eyJwIjoicGFja2FnZXMvZ29vZ2xlLWNsb3VkLW9zLWxvZ2luLy5Pd2xCb3QueWFtbCIsImgiOiI1M2FhODg0MTY3NTcxMDQ0NTAxZDA3NDIyMGU2M2FiY2Q5OGM2YWMxIn0= END_NESTED_COMMIT --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
diff --git a/packages/google-cloud-os-login/google/cloud/oslogin/gapic_version.py b/packages/google-cloud-os-login/google/cloud/oslogin/gapic_version.py
@@ -13,4 +13,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-__version__ = "2.12.0"  # {x-release-please-version}
+__version__ = "0.0.0"  # {x-release-please-version}
diff --git a/packages/google-cloud-os-login/google/cloud/oslogin_v1/gapic_version.py b/packages/google-cloud-os-login/google/cloud/oslogin_v1/gapic_version.py
@@ -13,4 +13,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-__version__ = "2.12.0"  # {x-release-please-version}
+__version__ = "0.0.0"  # {x-release-please-version}
diff --git a/packages/google-cloud-os-login/google/cloud/oslogin_v1/types/oslogin.py b/packages/google-cloud-os-login/google/cloud/oslogin_v1/types/oslogin.py
@@ -187,6 +187,11 @@ class ImportSshPublicKeyRequest(proto.Message):
         project_id (str):
             The project ID of the Google Cloud Platform
             project.
+        regions (MutableSequence[str]):
+            Optional. The regions to which to assert that
+            the key was written. If unspecified, defaults to
+            all regions. Regions are listed at
+            https://cloud.google.com/about/locations#region.
     """
 
     parent: str = proto.Field(
@@ -202,6 +207,10 @@ class ImportSshPublicKeyRequest(proto.Message):
         proto.STRING,
         number=3,
     )
+    regions: MutableSequence[str] = proto.RepeatedField(
+        proto.STRING,
+        number=5,
+    )
 
 
 class ImportSshPublicKeyResponse(proto.Message):
diff --git a/packages/google-cloud-os-login/samples/generated_samples/snippet_metadata_google.cloud.oslogin.v1.json b/packages/google-cloud-os-login/samples/generated_samples/snippet_metadata_google.cloud.oslogin.v1.json
@@ -8,7 +8,7 @@
     ],
     "language": "PYTHON",
     "name": "google-cloud-os-login",
-    "version": "2.12.0"
+    "version": "0.1.0"
   },
   "snippets": [
     {
diff --git a/packages/google-cloud-os-login/scripts/fixup_oslogin_v1_keywords.py b/packages/google-cloud-os-login/scripts/fixup_oslogin_v1_keywords.py
@@ -44,7 +44,7 @@ class osloginCallTransformer(cst.CSTTransformer):
         'delete_ssh_public_key': ('name', ),
         'get_login_profile': ('name', 'project_id', 'system_id', ),
         'get_ssh_public_key': ('name', ),
-        'import_ssh_public_key': ('parent', 'ssh_public_key', 'project_id', ),
+        'import_ssh_public_key': ('parent', 'ssh_public_key', 'project_id', 'regions', ),
         'update_ssh_public_key': ('name', 'ssh_public_key', 'update_mask', ),
     }
 
diff --git a/packages/google-cloud-os-login/tests/unit/gapic/oslogin_v1/test_os_login_service.py b/packages/google-cloud-os-login/tests/unit/gapic/oslogin_v1/test_os_login_service.py
@@ -4039,7 +4039,12 @@ def test_import_ssh_public_key_rest_required_fields(
         credentials=ga_credentials.AnonymousCredentials()
     ).import_ssh_public_key._get_unset_required_fields(jsonified_request)
     # Check that path parameters and body parameters are not mixing in.
-    assert not set(unset_fields) - set(("project_id",))
+    assert not set(unset_fields) - set(
+        (
+            "project_id",
+            "regions",
+        )
+    )
     jsonified_request.update(unset_fields)
 
     # verify required fields with non-default values are left alone
@@ -4094,7 +4099,15 @@ def test_import_ssh_public_key_rest_unset_required_fields():
     )
 
     unset_fields = transport.import_ssh_public_key._get_unset_required_fields({})
-    assert set(unset_fields) == (set(("projectId",)) & set(("parent",)))
+    assert set(unset_fields) == (
+        set(
+            (
+                "projectId",
+                "regions",
+            )
+        )
+        & set(("parent",))
+    )
 
 
 @pytest.mark.parametrize("null_interceptor", [True, False])

Original file line number	Diff line number	Diff line change
`@@ -13,4 +13,4 @@`
`13`	`13`	`# See the License for the specific language governing permissions and`
`14`	`14`	`# limitations under the License.`
`15`	`15`	`#`
`16`		`-__version__ = "2.12.0" # {x-release-please-version}`
	`16`	`+__version__ = "0.0.0" # {x-release-please-version}`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`],`
`9`	`9`	`"language": "PYTHON",`
`10`	`10`	`"name": "google-cloud-os-login",`
`11`		`- "version": "2.12.0"`
	`11`	`+ "version": "0.1.0"`
`12`	`12`	`},`
`13`	`13`	`"snippets": [`
`14`	`14`	`{`
Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ class osloginCallTransformer(cst.CSTTransformer):`
`44`	`44`	`'delete_ssh_public_key': ('name', ),`
`45`	`45`	`'get_login_profile': ('name', 'project_id', 'system_id', ),`
`46`	`46`	`'get_ssh_public_key': ('name', ),`
`47`		`- 'import_ssh_public_key': ('parent', 'ssh_public_key', 'project_id', ),`
	`47`	`+ 'import_ssh_public_key': ('parent', 'ssh_public_key', 'project_id', 'regions', ),`
`48`	`48`	`'update_ssh_public_key': ('name', 'ssh_public_key', 'update_mask', ),`
`49`	`49`	`}`
`50`	`50`