feat: [google-cloud-securesourcemanager] add field private_config to message .google.cloud.securesourcemanager.v1.Instance (#12891)

- [ ] Regenerate this pull request now.

BEGIN_COMMIT_OVERRIDE
feat: add field `private_config` to message
`.google.cloud.securesourcemanager.v1.Instance`
feat: add constant `UNKNOWN` to field `State` in message
`.google.cloud.securesourcemanager.v1.Instance`
docs: A comment for field `instance` in message
`.google.cloud.securesourcemanager.v1.Repository` is updated to include
data plane vs control plane behavior.
END_COMMIT_OVERRIDE



PiperOrigin-RevId: 650735536

Source-Link:
googleapis/googleapis@847b40b

Source-Link:
googleapis/googleapis-gen@dd9deaf
Copy-Tag:
eyJwIjoicGFja2FnZXMvZ29vZ2xlLWNsb3VkLXNlY3VyZXNvdXJjZW1hbmFnZXIvLk93bEJvdC55YW1sIiwiaCI6ImRkOWRlYWY3YzlhMWQ5MmI5ZjNmMjQxY2I4YzdjY2ZhNTNiNmYyNTQifQ==

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Author: gcf-owl-bot[bot]

packages/google-cloud-securesourcemanager/google/cloud/securesourcemanager/gapic_version.py

@@ -13,4 +13,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-__version__ = "0.1.6"  # {x-release-please-version}
+__version__ = "0.0.0"  # {x-release-please-version}

packages/google-cloud-securesourcemanager/google/cloud/securesourcemanager_v1/gapic_version.py

@@ -13,4 +13,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-__version__ = "0.1.6"  # {x-release-please-version}
+__version__ = "0.0.0"  # {x-release-please-version}

packages/google-cloud-securesourcemanager/google/cloud/securesourcemanager_v1/services/secure_source_manager/async_client.py

@@ -97,6 +97,8 @@ class SecureSourceManagerAsyncClient:
     _DEFAULT_ENDPOINT_TEMPLATE = SecureSourceManagerClient._DEFAULT_ENDPOINT_TEMPLATE
     _DEFAULT_UNIVERSE = SecureSourceManagerClient._DEFAULT_UNIVERSE
 
+    ca_pool_path = staticmethod(SecureSourceManagerClient.ca_pool_path)
+    parse_ca_pool_path = staticmethod(SecureSourceManagerClient.parse_ca_pool_path)
     crypto_key_path = staticmethod(SecureSourceManagerClient.crypto_key_path)
     parse_crypto_key_path = staticmethod(
         SecureSourceManagerClient.parse_crypto_key_path
@@ -107,6 +109,12 @@ class SecureSourceManagerAsyncClient:
     parse_repository_path = staticmethod(
         SecureSourceManagerClient.parse_repository_path
     )
+    service_attachment_path = staticmethod(
+        SecureSourceManagerClient.service_attachment_path
+    )
+    parse_service_attachment_path = staticmethod(
+        SecureSourceManagerClient.parse_service_attachment_path
+    )
     common_billing_account_path = staticmethod(
         SecureSourceManagerClient.common_billing_account_path
     )

packages/google-cloud-securesourcemanager/google/cloud/securesourcemanager_v1/services/secure_source_manager/client.py

@@ -217,6 +217,28 @@ def transport(self) -> SecureSourceManagerTransport:
         """
         return self._transport
 
+    @staticmethod
+    def ca_pool_path(
+        project: str,
+        location: str,
+        ca_pool: str,
+    ) -> str:
+        """Returns a fully-qualified ca_pool string."""
+        return "projects/{project}/locations/{location}/caPools/{ca_pool}".format(
+            project=project,
+            location=location,
+            ca_pool=ca_pool,
+        )
+
+    @staticmethod
+    def parse_ca_pool_path(path: str) -> Dict[str, str]:
+        """Parses a ca_pool path into its component segments."""
+        m = re.match(
+            r"^projects/(?P<project>.+?)/locations/(?P<location>.+?)/caPools/(?P<ca_pool>.+?)$",
+            path,
+        )
+        return m.groupdict() if m else {}
+
     @staticmethod
     def crypto_key_path(
         project: str,
@@ -287,6 +309,28 @@ def parse_repository_path(path: str) -> Dict[str, str]:
         )
         return m.groupdict() if m else {}
 
+    @staticmethod
+    def service_attachment_path(
+        project: str,
+        region: str,
+        service_attachment: str,
+    ) -> str:
+        """Returns a fully-qualified service_attachment string."""
+        return "projects/{project}/regions/{region}/serviceAttachments/{service_attachment}".format(
+            project=project,
+            region=region,
+            service_attachment=service_attachment,
+        )
+
+    @staticmethod
+    def parse_service_attachment_path(path: str) -> Dict[str, str]:
+        """Parses a service_attachment path into its component segments."""
+        m = re.match(
+            r"^projects/(?P<project>.+?)/regions/(?P<region>.+?)/serviceAttachments/(?P<service_attachment>.+?)$",
+            path,
+        )
+        return m.groupdict() if m else {}
+
     @staticmethod
     def common_billing_account_path(
         billing_account: str,

packages/google-cloud-securesourcemanager/google/cloud/securesourcemanager_v1/types/secure_source_manager.py

@@ -64,6 +64,9 @@ class Instance(proto.Message):
             Output only. Update timestamp.
         labels (MutableMapping[str, str]):
             Optional. Labels as key value pairs.
+        private_config (google.cloud.securesourcemanager_v1.types.Instance.PrivateConfig):
+            Optional. Private settings for private
+            instance.
         state (google.cloud.securesourcemanager_v1.types.Instance.State):
             Output only. Current state of the instance.
         state_note (google.cloud.securesourcemanager_v1.types.Instance.StateNote):
@@ -93,12 +96,16 @@ class State(proto.Enum):
                 Instance is being deleted.
             PAUSED (4):
                 Instance is paused.
+            UNKNOWN (6):
+                Instance is unknown, we are not sure if it's
+                functioning.
         """
         STATE_UNSPECIFIED = 0
         CREATING = 1
         ACTIVE = 2
         DELETING = 3
         PAUSED = 4
+        UNKNOWN = 6
 
     class StateNote(proto.Enum):
         r"""Provides information about the current instance state.
@@ -148,6 +155,44 @@ class HostConfig(proto.Message):
             number=4,
         )
 
+    class PrivateConfig(proto.Message):
+        r"""PrivateConfig includes settings for private instance.
+
+        Attributes:
+            is_private (bool):
+                Required. Immutable. Indicate if it's private
+                instance.
+            ca_pool (str):
+                Required. Immutable. CA pool resource, resource must in the
+                format of
+                ``projects/{project}/locations/{location}/caPools/{ca_pool}``.
+            http_service_attachment (str):
+                Output only. Service Attachment for HTTP, resource is in the
+                format of
+                ``projects/{project}/regions/{region}/serviceAttachments/{service_attachment}``.
+            ssh_service_attachment (str):
+                Output only. Service Attachment for SSH, resource is in the
+                format of
+                ``projects/{project}/regions/{region}/serviceAttachments/{service_attachment}``.
+        """
+
+        is_private: bool = proto.Field(
+            proto.BOOL,
+            number=1,
+        )
+        ca_pool: str = proto.Field(
+            proto.STRING,
+            number=2,
+        )
+        http_service_attachment: str = proto.Field(
+            proto.STRING,
+            number=3,
+        )
+        ssh_service_attachment: str = proto.Field(
+            proto.STRING,
+            number=4,
+        )
+
     name: str = proto.Field(
         proto.STRING,
         number=1,
@@ -167,6 +212,11 @@ class HostConfig(proto.Message):
         proto.STRING,
         number=4,
     )
+    private_config: PrivateConfig = proto.Field(
+        proto.MESSAGE,
+        number=13,
+        message=PrivateConfig,
+    )
     state: State = proto.Field(
         proto.ENUM,
         number=5,
@@ -200,9 +250,12 @@ class Repository(proto.Message):
             Optional. Description of the repository,
             which cannot exceed 500 characters.
         instance (str):
-            Output only. The name of the instance in which the
-            repository is hosted, formatted as
+            Optional. The name of the instance in which the repository
+            is hosted, formatted as
             ``projects/{project_number}/locations/{location_id}/instances/{instance_id}``
+            For data plane CreateRepository requests, this field is
+            output only. For control plane CreateRepository requests,
+            this field is used as input.
         uid (str):
             Output only. Unique identifier of the
             repository.

packages/google-cloud-securesourcemanager/samples/generated_samples/snippet_metadata_google.cloud.securesourcemanager.v1.json

@@ -8,7 +8,7 @@
     ],
     "language": "PYTHON",
     "name": "google-cloud-securesourcemanager",
-    "version": "0.1.6"
+    "version": "0.1.0"
   },
   "snippets": [
     {

packages/google-cloud-securesourcemanager/tests/unit/gapic/securesourcemanager_v1/test_secure_source_manager.py

@@ -6607,6 +6607,12 @@ def test_create_instance_rest(request_type):
         "create_time": {"seconds": 751, "nanos": 543},
         "update_time": {},
         "labels": {},
+        "private_config": {
+            "is_private": True,
+            "ca_pool": "ca_pool_value",
+            "http_service_attachment": "http_service_attachment_value",
+            "ssh_service_attachment": "ssh_service_attachment_value",
+        },
         "state": 1,
         "state_note": 1,
         "kms_key": "kms_key_value",
@@ -10302,11 +10308,37 @@ def test_secure_source_manager_grpc_lro_async_client():
     assert transport.operations_client is transport.operations_client
 
 
-def test_crypto_key_path():
+def test_ca_pool_path():
     project = "squid"
     location = "clam"
-    key_ring = "whelk"
-    crypto_key = "octopus"
+    ca_pool = "whelk"
+    expected = "projects/{project}/locations/{location}/caPools/{ca_pool}".format(
+        project=project,
+        location=location,
+        ca_pool=ca_pool,
+    )
+    actual = SecureSourceManagerClient.ca_pool_path(project, location, ca_pool)
+    assert expected == actual
+
+
+def test_parse_ca_pool_path():
+    expected = {
+        "project": "octopus",
+        "location": "oyster",
+        "ca_pool": "nudibranch",
+    }
+    path = SecureSourceManagerClient.ca_pool_path(**expected)
+
+    # Check that the path construction is reversible.
+    actual = SecureSourceManagerClient.parse_ca_pool_path(path)
+    assert expected == actual
+
+
+def test_crypto_key_path():
+    project = "cuttlefish"
+    location = "mussel"
+    key_ring = "winkle"
+    crypto_key = "nautilus"
     expected = "projects/{project}/locations/{location}/keyRings/{key_ring}/cryptoKeys/{crypto_key}".format(
         project=project,
         location=location,
@@ -10321,10 +10353,10 @@ def test_crypto_key_path():
 
 def test_parse_crypto_key_path():
     expected = {
-        "project": "oyster",
-        "location": "nudibranch",
-        "key_ring": "cuttlefish",
-        "crypto_key": "mussel",
+        "project": "scallop",
+        "location": "abalone",
+        "key_ring": "squid",
+        "crypto_key": "clam",
     }
     path = SecureSourceManagerClient.crypto_key_path(**expected)
 
@@ -10334,9 +10366,9 @@ def test_parse_crypto_key_path():
 
 
 def test_instance_path():
-    project = "winkle"
-    location = "nautilus"
-    instance = "scallop"
+    project = "whelk"
+    location = "octopus"
+    instance = "oyster"
     expected = "projects/{project}/locations/{location}/instances/{instance}".format(
         project=project,
         location=location,
@@ -10348,9 +10380,9 @@ def test_instance_path():
 
 def test_parse_instance_path():
     expected = {
-        "project": "abalone",
-        "location": "squid",
-        "instance": "clam",
+        "project": "nudibranch",
+        "location": "cuttlefish",
+        "instance": "mussel",
     }
     path = SecureSourceManagerClient.instance_path(**expected)
 
@@ -10360,9 +10392,9 @@ def test_parse_instance_path():
 
 
 def test_repository_path():
-    project = "whelk"
-    location = "octopus"
-    repository = "oyster"
+    project = "winkle"
+    location = "nautilus"
+    repository = "scallop"
     expected = (
         "projects/{project}/locations/{location}/repositories/{repository}".format(
             project=project,
@@ -10376,9 +10408,9 @@ def test_repository_path():
 
 def test_parse_repository_path():
     expected = {
-        "project": "nudibranch",
-        "location": "cuttlefish",
-        "repository": "mussel",
+        "project": "abalone",
+        "location": "squid",
+        "repository": "clam",
     }
     path = SecureSourceManagerClient.repository_path(**expected)
 
@@ -10387,6 +10419,34 @@ def test_parse_repository_path():
     assert expected == actual
 
 
+def test_service_attachment_path():
+    project = "whelk"
+    region = "octopus"
+    service_attachment = "oyster"
+    expected = "projects/{project}/regions/{region}/serviceAttachments/{service_attachment}".format(
+        project=project,
+        region=region,
+        service_attachment=service_attachment,
+    )
+    actual = SecureSourceManagerClient.service_attachment_path(
+        project, region, service_attachment
+    )
+    assert expected == actual
+
+
+def test_parse_service_attachment_path():
+    expected = {
+        "project": "nudibranch",
+        "region": "cuttlefish",
+        "service_attachment": "mussel",
+    }
+    path = SecureSourceManagerClient.service_attachment_path(**expected)
+
+    # Check that the path construction is reversible.
+    actual = SecureSourceManagerClient.parse_service_attachment_path(path)
+    assert expected == actual
+
+
 def test_common_billing_account_path():
     billing_account = "winkle"
     expected = "billingAccounts/{billing_account}".format(