feat: (storageinsights) added field parquet_options in google_storage_insights_report_config (#13178)

Author: gurusai-voleti

mmv1/products/storageinsights/ReportConfig.yaml

@@ -100,11 +100,23 @@ properties:
             type: Integer
             description: 'The year to stop generating inventory reports'
             required: true
+  - name: 'parquetOptions'
+    type: NestedObject
+    description: |
+      An option for outputting inventory reports as parquet files.
+    exactly_one_of:
+      - parquet_options
+      - csv_options
+    allow_empty_object: true
+    send_empty_value: true
+    properties: []
   - name: 'csvOptions'
     type: NestedObject
     description: |
       Options for configuring the format of the inventory report CSV file.
-    required: true
+    exactly_one_of:
+      - parquet_options
+      - csv_options
     properties:
       - name: 'recordSeparator'
         type: String

mmv1/third_party/terraform/services/storageinsights/resource_storage_insights_report_config_test.go

@@ -43,6 +43,147 @@ func TestAccStorageInsightsReportConfig_update(t *testing.T) {
 	})
 }
 
+func TestAccStorageInsightsReportConfig_parquet(t *testing.T) {
+	t.Parallel()
+
+	context := map[string]interface{}{
+		"random_suffix": acctest.RandString(t, 10),
+	}
+
+	acctest.VcrTest(t, resource.TestCase{
+		PreCheck:                 func() { acctest.AccTestPreCheck(t) },
+		ProtoV5ProviderFactories: acctest.ProtoV5ProviderFactories(t),
+		Steps: []resource.TestStep{
+			{
+				Config: testAccStorageInsightsReportConfig_parquet(context),
+			},
+			{
+				ResourceName:            "google_storage_insights_report_config.config",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"location"},
+			},
+			{
+				Config: testAccStorageInsightsReportConfig_updateCsv(context),
+			},
+			{
+				ResourceName:            "google_storage_insights_report_config.config",
+				ImportState:             true,
+				ImportStateVerify:       true,
+				ImportStateVerifyIgnore: []string{"location"},
+			},
+		},
+	})
+}
+
+func testAccStorageInsightsReportConfig_parquet(context map[string]interface{}) string {
+	return acctest.Nprintf(`
+data "google_project" "project" {
+}
+
+resource "google_storage_insights_report_config" "config" {
+  display_name = "Test Report Config"
+  location = "us-central1"
+  frequency_options {
+    frequency = "WEEKLY"
+    start_date {
+      day = 15
+      month = 3
+      year = 2050
+    }
+    end_date {
+      day = 15
+      month = 4
+      year = 2050
+    }
+  }
+  parquet_options {}
+  object_metadata_report_options {
+    metadata_fields = ["bucket", "name", "project"]
+    storage_filters {
+      bucket = google_storage_bucket.report_bucket.name
+    }
+    storage_destination_options {
+      bucket = google_storage_bucket.report_bucket.name
+      destination_path = "test-insights-reports"
+    }
+  }
+  depends_on = [
+	google_storage_bucket_iam_member.admin,
+  ]
+}
+
+resource "google_storage_bucket" "report_bucket" {
+  name                        = "tf-test-my-bucket%{random_suffix}"
+  location                    = "us-central1"
+  force_destroy               = true
+  uniform_bucket_level_access = true
+}
+
+resource "google_storage_bucket_iam_member" "admin" {
+  bucket = google_storage_bucket.report_bucket.name
+  role   = "roles/storage.admin"
+  member = "serviceAccount:service-${data.google_project.project.number}@gcp-sa-storageinsights.iam.gserviceaccount.com"
+}
+`, context)
+}
+
+func testAccStorageInsightsReportConfig_updateCsv(context map[string]interface{}) string {
+	return acctest.Nprintf(`
+data "google_project" "project" {
+}
+
+resource "google_storage_insights_report_config" "config" {
+  display_name = "Test Report Config"
+  location = "us-central1"
+  frequency_options {
+    frequency = "WEEKLY"
+    start_date {
+      day = 15
+      month = 3
+      year = 2050
+    }
+    end_date {
+      day = 15
+      month = 4
+      year = 2050
+    }
+  }
+  csv_options {
+    record_separator = "\n"
+    delimiter = ","
+    header_required = false
+  }
+  object_metadata_report_options {
+    metadata_fields = ["bucket", "name", "project"]
+    storage_filters {
+      bucket = google_storage_bucket.report_bucket.name
+    }
+    storage_destination_options {
+      bucket = google_storage_bucket.report_bucket.name
+      destination_path = "test-insights-reports"
+    }
+  }
+  depends_on = [
+	google_storage_bucket_iam_member.admin,
+  ]
+}
+
+resource "google_storage_bucket" "report_bucket" {
+  name                        = "tf-test-my-bucket%{random_suffix}"
+  location                    = "us-central1"
+  force_destroy               = true
+  uniform_bucket_level_access = true
+}
+
+resource "google_storage_bucket_iam_member" "admin" {
+  bucket = google_storage_bucket.report_bucket.name
+  role   = "roles/storage.admin"
+  member = "serviceAccount:service-${data.google_project.project.number}@gcp-sa-storageinsights.iam.gserviceaccount.com"
+}
+`, context)
+}
+
 func testAccStorageInsightsReportConfig_full(context map[string]interface{}) string {
 	return acctest.Nprintf(`
 data "google_project" "project" {
@@ -120,11 +261,7 @@ resource "google_storage_insights_report_config" "config" {
       year = 2040
     }
   }
-  csv_options {
-    record_separator = "\r\n"
-    delimiter = "."
-    header_required = true
-  }
+  parquet_options {}
   object_metadata_report_options {
     metadata_fields = ["bucket", "name", "project"]
     storage_filters {