feat: Add DefaultLogsBucketBehavior to BuildOptions (#391)

gcf-owl-bot[bot] · web-flow · commit 599c9385af35 · 2023-03-14T06:59:57.000-04:00
* feat: Add DefaultLogsBucketBehavior to BuildOptions Committer: @arvinddayal PiperOrigin-RevId: 515431035 Source-Link: googleapis/googleapis@d129b4f Source-Link: https://github.com/googleapis/googleapis-gen/commit/5f17a32e7da58a2060ecee9fd29fe36e6eb39847 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNWYxN2EzMmU3ZGE1OGEyMDYwZWNlZTlmZDI5ZmUzNmU2ZWIzOTg0NyJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
diff --git a/google/cloud/devtools/cloudbuild_v1/types/cloudbuild.py b/google/cloud/devtools/cloudbuild_v1/types/cloudbuild.py
@@ -2674,6 +2674,9 @@ class BuildOptions(proto.Message):
             Using a global volume in a build with only one
             step is not valid as it is indicative of a build
             request with an incorrect configuration.
+        default_logs_bucket_behavior (google.cloud.devtools.cloudbuild_v1.types.BuildOptions.DefaultLogsBucketBehavior):
+            Option to specify how default logs buckets
+            are setup.
     """
 
     class VerifyOption(proto.Enum):
@@ -2779,6 +2782,21 @@ class LoggingMode(proto.Enum):
         CLOUD_LOGGING_ONLY = 5
         NONE = 4
 
+    class DefaultLogsBucketBehavior(proto.Enum):
+        r"""Default GCS log bucket behavior options.
+
+        Values:
+            DEFAULT_LOGS_BUCKET_BEHAVIOR_UNSPECIFIED (0):
+                Unspecified.
+            REGIONAL_USER_OWNED_BUCKET (1):
+                Bucket is located in user-owned project in
+                the same region as the build. The builder
+                service account must have access to create and
+                write to GCS buckets in the build project.
+        """
+        DEFAULT_LOGS_BUCKET_BEHAVIOR_UNSPECIFIED = 0
+        REGIONAL_USER_OWNED_BUCKET = 1
+
     class PoolOption(proto.Message):
         r"""Details about how a build should be executed on a ``WorkerPool``.
 
@@ -2861,6 +2879,11 @@ class PoolOption(proto.Message):
         number=14,
         message="Volume",
     )
+    default_logs_bucket_behavior: DefaultLogsBucketBehavior = proto.Field(
+        proto.ENUM,
+        number=21,
+        enum=DefaultLogsBucketBehavior,
+    )
 
 
 class ReceiveTriggerWebhookRequest(proto.Message):
diff --git a/tests/unit/gapic/cloudbuild_v1/test_cloud_build.py b/tests/unit/gapic/cloudbuild_v1/test_cloud_build.py
@@ -5993,6 +5993,7 @@ def test_create_build_rest(request_type):
             "env": ["env_value1", "env_value2"],
             "secret_env": ["secret_env_value1", "secret_env_value2"],
             "volumes": {},
+            "default_logs_bucket_behavior": 1,
         },
         "log_url": "log_url_value",
         "substitutions": {},
@@ -6331,6 +6332,7 @@ def test_create_build_rest_bad_request(
             "env": ["env_value1", "env_value2"],
             "secret_env": ["secret_env_value1", "secret_env_value2"],
             "volumes": {},
+            "default_logs_bucket_behavior": 1,
         },
         "log_url": "log_url_value",
         "substitutions": {},
@@ -8078,6 +8080,7 @@ def test_create_build_trigger_rest(request_type):
                 "env": ["env_value1", "env_value2"],
                 "secret_env": ["secret_env_value1", "secret_env_value2"],
                 "volumes": {},
+                "default_logs_bucket_behavior": 1,
             },
             "log_url": "log_url_value",
             "substitutions": {},
@@ -8482,6 +8485,7 @@ def test_create_build_trigger_rest_bad_request(
                 "env": ["env_value1", "env_value2"],
                 "secret_env": ["secret_env_value1", "secret_env_value2"],
                 "volumes": {},
+                "default_logs_bucket_behavior": 1,
             },
             "log_url": "log_url_value",
             "substitutions": {},
@@ -9682,6 +9686,7 @@ def test_update_build_trigger_rest(request_type):
                 "env": ["env_value1", "env_value2"],
                 "secret_env": ["secret_env_value1", "secret_env_value2"],
                 "volumes": {},
+                "default_logs_bucket_behavior": 1,
             },
             "log_url": "log_url_value",
             "substitutions": {},
@@ -10089,6 +10094,7 @@ def test_update_build_trigger_rest_bad_request(
                 "env": ["env_value1", "env_value2"],
                 "secret_env": ["secret_env_value1", "secret_env_value2"],
                 "volumes": {},
+                "default_logs_bucket_behavior": 1,
             },
             "log_url": "log_url_value",
             "substitutions": {},
