Skip to content

Sync 5.0.0 with main #8945

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 57 commits into from
Sep 14, 2023
Merged

Sync 5.0.0 with main #8945

merged 57 commits into from
Sep 14, 2023

Conversation

shuyama1
Copy link
Member

Release Note Template for Downstream PRs (will be copied)


sahsagar-google and others added 30 commits September 6, 2023 11:48
…8832)

* Adding Terraform resources for Tenancy APIs in GKEHub

* Segregating MembershipBinding and MembershipRBACRoleBinding to keep things simpler in the review

* Fixing the docu URIs

* Adding TF support for Tenancy API for Membership Binding

* Adding dependent membership binding to the same commit chain

* Making Scope un-updatable and replacing hard coded project number with the one from test env

* Making Scope RRBAC updatable

* Making Namespace immutable

* Adding update test cases

* Removing all memberships field from Scope since it is no longer supported

* Removing all_memberships field for Scope from all test cases

* Enhancing namespace resource with the labels fields

* Adding more keys to test out the sorted comparison of keys in the map of labels

* Adding more keys to test out the sorted comparison of keys in the update map
* Fix import of bigquery transfer config location

* Fix import of bigquery transfer config when name does not contain location

* Fix indentation in bigquery_data_transfer_self_link_as_name_set_location.go.erb
Co-authored-by: Edward Sun <[email protected]>
* Add BigLake Table Resource

* add: biglake table example

* add: biglake table update test

* update: hive_options

* fix: wrong directory

* trim: trailing whitespace

* fix: incorrect example location

* delete: incorrect file

* add: acc tests for biglake table

* Add SDK provider configuration tests, part 2 (#7723)

* Add tests for `providerConfigure` testing `billing_project` config

* Fix `billing_project` test

* Add tests for `providerConfigure` testing `region` config

* Add tests for `providerConfigure` testing `zone` config

* Add tests for `providerConfigure` testing `user_project_override` config

* Remove old redundant `TestHandleSDKDefaults_*` tests

* Update code following creation of `transport` package

* Reposition `transport_tpg` in imports

* Add WIP of scopes test

* Fix defect in test case

* Fix scopes test

* Implement test cases for user config including zero values, and update tests to asset when field should be unset but is found in config

* Make test error messages clearer w.r.t provider config from user versus derived Config struct

* Fix import issue after rebase

* Add comments separating types of test case, add cases for empty strings overwritten by ENVs

* Rebase PR to pull in changes, including service package split

* Add test cases that show empty strings are ignored and ENVs are used instead

* Update text in test failure messages, add comments to signpost different types of test case

* Make lack of error more explicit in test case

* Fix import issues from rebasing branch

* update use of `ioutil` to `os`

* Make imports match what's on main

* Update `testFakeCredentialsPath` to `transport_tpg.TestFakeCredentialsPath`

* Add missing comma

* Move file into `provider_test` package, update imports and remove duplicate code

* Fix defect when trying to access `CredentialsFromJSON` function

* Add missing function calls to set ENVs used in tests

* Update tests for region field, add test case for use of self links

* Aad test case for zone field, self links are not shortened and usable

* Remove test case; `ConflictsWith` on fields not testable at this level

* Update test case names, add comments

* TeamCity: Usability improvements : tag builds to distinguish nightly builds vs ad hoc builds, add project descriptions (#8685)

* Add ability to tag TeamCity builds based on whether they're automated or ad-hoc. Nightly builds tagged with the date.

* Add ability to set project descriptions using a context parameter

* Refactor how date is formatted, to avoid problem where TeamCity interprets `%Y-%` as interpolating a `Y-` parameter

* Remove use of `TRIGGERED_BY`; value in build didn't match UI and isn't useful

* Update tag for nightly test builds to be static/consistent

* Add ignore read on reservedIpRange field Filestore Instance (#8520)

Co-authored-by: Shuya Ma <[email protected]>

* b-283271112 Add "additionalScopes" under webSsoConfig for the "groups" (#8744)

* Made breaking change detector own its own setup and build process (#8705)

* Made breaking change detector own its own setup and build process

* Moved breaking change detector unit tests to github action

* Corrected breaking change detector unit test setup

* Added back package name updates for tpgb new

* made unit tests use a shallow clone

* Limit breaking change detector unit tests to runs that modify the tool

Co-authored-by: Scott Suarez <[email protected]>

* Update .ci/scripts/go-plus/github-differ/generate_comment.sh

Co-authored-by: Scott Suarez <[email protected]>

* Minor Cleanup

* Intentionally broke breaking change detector

* Revert "Intentionally broke breaking change detector"

This reverts commit bcb6ba8.

* Intentionally caused panic in breaking change detector at runtime

* Made a breaking change

* Added additional logging

* Removed export in generate_comment.sh

export hides the exit code of the command being run; assignment on its own does not. Export is not required in the context of a shell script

* Made failure get set to 1 instead of $?

* Added bin/ cleanup

* Revert "Intentionally caused panic in breaking change detector at runtime"

This reverts commit a16c0cd.

* Fixed package name replacement for google-beta

* Re-added export of TPG/TPGB BREAKING

* Added comment explaining the export location

* Revert "Made a breaking change"

This reverts commit 2deecd7.

---------

Co-authored-by: Scott Suarez <[email protected]>

* Go changelog (#8727)

* Add copy of go-changelog to tools

* Add changelog checker to pre-build validation

* Log errors instead of commenting

* Move check changelog to separate workflow

* Mark lines for removal

* Remove unused go-changelog tools

* Apply suggestions from code review

Co-authored-by: Scott Suarez <[email protected]>

* Make old changelog checker a no-op

---------

Co-authored-by: Scott Suarez <[email protected]>

* Fix acctest import specifying the beta version instead of GA (#8736)

* Cleanup deprecated folders (#8720)

* Remove folder resources and data_sources

* Remove utility files

* Move more utility files

* Fix tgc

* Import provider package in tgc

* Remove utils files

* Remove test lines from changelog checker (#8763)

* Adds the synthetic monitor target type as an option for uptime checks. (#8709)

* Adds synthetic_monitor type that lives alongside resource_group and monitored_resource
* removes requirement for either tcp_check or http_check to be provided, as neither is required when synthetic_monitor is provided
* Adds acceptance test, and example. A new test fixutre w/ zip file is provided for these flows.

* Removed guard around operation WithResponse method & improved error handling (#8762)

* Removed guard around operation WithResponse method

Resolved hashicorp/terraform-provider-google#15618

* Handle missing resource inside response more gracefully

* Add zli82016 to the vacation list (#8761)

* Update example for google_compute_addresses data source (#8765)

* Remove myself from vacation list (#8770)

* Document database flags type change (#8769)

* Added some missing what's next links (#8733)

* Added some missing what's next links

* reverted changes to run-tests.md

* Add: BigLake API Catalog Resource (#8751)

Co-authored-by: Shuya Ma <[email protected]>

* Add workflow for membership checker unit tests (#8760)

* Add workflow for membership checker unit tests

* Apply suggestions from code review

Co-authored-by: Scott Suarez <[email protected]>

---------

Co-authored-by: Scott Suarez <[email protected]>

* Key upload (#8714)

* add oidc key upload

* fix a bug

* fix a typo

* add example

* Update mmv1/products/iamworkforcepool/WorkforcePoolProvider.yaml

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>

* remove unnecessary test

* add a new line

* fix a bug

* Update mmv1/products/iamworkforcepool/WorkforcePoolProvider.yaml

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>

---------

Co-authored-by: Stephen Lewis (Burrows) <[email protected]>

* Marks `template.volumes.secret.items.mode` field not required in Cloud Run V2 resources (#8771)

* removed require

* remove required for job

* Removed MembershipRBACRoleBinding from ga provider (#8776)

* make distribution_policy_target_shape updatable (#8774)

Co-authored-by: Edward Sun <[email protected]>

* feat(google_container_cluster): support fqdn network policy (#8461)

Signed-off-by: Tsubasa Nagasawa <[email protected]>

* Redis cluster terraform support (#8567)

* Redis cluster terraform support

* updating cluster resource

* updating cluster resource

* updating cluster resource

* updating cluster resource

* updating cluster resource

* updating cluster resource

* update

* update

* update

* update

* update

* update

* update

* update

* update SCPolicy

* Update Service connection policy

* Update Service connection policy

---------

Co-authored-by: Himani Khanduja <[email protected]>

* move: Table.yaml to `biglake`

* update: switch to parent id convention

* update: example to use parent id convention

* update: continue to update to parent convention

* add: biglake table custom import

* fix: correct usage of names

* fix: extra comma

* fix: parameter spec.

* fix: parameters spec

* delete: remove location param

* fix: build

* fix: correct url

* add: spec custom import

* update: switch to id_format over custom import

* fix: remove location from resource test.

* fix: test names

* add: serde_info to acc test

* fix: serde_info is a struct

* remove: serde_info due to api bug.

* delete: unsupported serdeinfo field

* fix: use example person names

* update: mark `database` immutable

* update: change more fields in update test

---------

Signed-off-by: Tsubasa Nagasawa <[email protected]>
Co-authored-by: Sarah French <[email protected]>
Co-authored-by: Baruch Steinberg <[email protected]>
Co-authored-by: Shuya Ma <[email protected]>
Co-authored-by: vaibhav-google <[email protected]>
Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
Co-authored-by: Scott Suarez <[email protected]>
Co-authored-by: Thomas Rodgers <[email protected]>
Co-authored-by: Ryan Oaks <[email protected]>
Co-authored-by: Zhenhua Li <[email protected]>
Co-authored-by: Daniel Koss <[email protected]>
Co-authored-by: Ryan White <[email protected]>
Co-authored-by: Riley Karson <[email protected]>
Co-authored-by: Sean McGivern <[email protected]>
Co-authored-by: bohengy <[email protected]>
Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
Co-authored-by: Yanwei Guo <[email protected]>
Co-authored-by: Edward Sun <[email protected]>
Co-authored-by: Edward Sun <[email protected]>
Co-authored-by: Tsubasa Nagasawa <[email protected]>
Co-authored-by: himanikh <[email protected]>
Co-authored-by: Himani Khanduja <[email protected]>
…dles `credentials` values (#8801)

* Add initial version of plugin framework provider config test affected by inaccessible functions

* Refactor provider config tests to use plugin-framework types

* Add test case about handling of Unknown values for `project`

* Update tests to check values in BOTH the data model and provider config struct after `LoadAndValidateFramework` runs

* Add some tests for `credentials` in plugin framework provider, including test case that fails

* Update `LoadAndValidateFramework` to take a pointer to the provider data model, so mutations to the data within the function change the original struct

This enables tests to track how the data is mutated

* Add remaining `credentials` test cases to check PF/SDK config parity

* Make tests unset ADC ENV automatically, update comments to tests setting ADC ENV

* Add test for behaviour when credentials value is unknown

* Add comment referring devs to where unknown value test is implemented

* Remove duplicated test case

* Fix filename so it's generated correctly

* Remove fmt line

* Update `project` tests that are affected by `LoadAndValidateFramework` now taking a pointer as an argument

* Update mmv1/third_party/terraform/fwtransport/framework_config_test.go.erb

Co-authored-by: Riley Karson <[email protected]>

---------

Co-authored-by: Riley Karson <[email protected]>
* Implement apigee target server resource

* fix : yaml format

* fix : add missing resource fields in test

* test : include more than one element in list

* docs : update descriptions and hcl example

* fix : add default value for protocol field

* test : add test step for omitting protocol field

* fix : replace default_value with default_from_api

* test : split clientAuthEnabled to seperate test

* Add comment explaining why acc test is skipped due to incompatibility with VCR

---------

Co-authored-by: Sarah French <[email protected]>
* Adding Terraform resources for Tenancy APIs in GKEHub

* Segregating MembershipBinding and MembershipRBACRoleBinding to keep things simpler in the review

* Fixing the docu URIs

* Adding TF support for Tenancy API for Membership Binding

* Adding dependent membership binding to the same commit chain

* Making Scope un-updatable and replacing hard coded project number with the one from test env

* Making Scope RRBAC updatable

* Making Namespace immutable

* Adding update test cases

* Removing all memberships field from Scope since it is no longer supported

* Removing all_memberships field for Scope from all test cases

* Add labels field to the Membership Binding resource
…dles `billing_project` values (#8861)

* Add initial version of plugin framework provider config test affected by inaccessible functions

* Refactor provider config tests to use plugin-framework types

* Add test case about handling of Unknown values for `project`

* Update tests to check values in BOTH the data model and provider config struct after `LoadAndValidateFramework` runs

* Add some tests for `credentials` in plugin framework provider, including test case that fails

* Update `LoadAndValidateFramework` to take a pointer to the provider data model, so mutations to the data within the function change the original struct

This enables tests to track how the data is mutated

* Add remaining `credentials` test cases to check PF/SDK config parity

* Make tests unset ADC ENV automatically, update comments to tests setting ADC ENV

* Add test for behaviour when credentials value is unknown

* Add comment referring devs to where unknown value test is implemented

* Remove duplicated test case

* Fix filename so it's generated correctly

* Remove fmt line

* Update `project` tests that are affected by `LoadAndValidateFramework` now taking a pointer as an argument

* Change strings in SDK version of billing_project tests

* Add plugin framework version of tests for billing_project

* Fix defect in PF code where (FrameworkProviderConfig).BillingProject value never set
* add upgrade_policy in vmware_cluster

* add disable_bundled_ingress in vmware_cluster

* add upgrade_policy field in bare_metal_cluster

* add binary_authorization field for bare_metal_cluster

* remove disable_bundled_ingress field as the API change it not yet released

* fix resource name conflicts in tests

* remove unspecified enum item

* Add new fields in update test

* fix yaml format

* Test the new fields changes in update test
…connection to 5.0 guide (#8889)

* Add the note using Create method to create google_service_networking_connection to 5.0 guide

* Improve the note
…dles `region` and `zone` values (#8856)

* Add initial version of plugin framework provider config test affected by inaccessible functions

* Refactor provider config tests to use plugin-framework types

* Add test case about handling of Unknown values for `project`

* Update tests to check values in BOTH the data model and provider config struct after `LoadAndValidateFramework` runs

* Add some tests for `credentials` in plugin framework provider, including test case that fails

* Update `LoadAndValidateFramework` to take a pointer to the provider data model, so mutations to the data within the function change the original struct

This enables tests to track how the data is mutated

* Add remaining `credentials` test cases to check PF/SDK config parity

* Make tests unset ADC ENV automatically, update comments to tests setting ADC ENV

* Add test for behaviour when credentials value is unknown

* Add comment referring devs to where unknown value test is implemented

* Remove duplicated test case

* Fix filename so it's generated correctly

* Remove fmt line

* Update `project` tests that are affected by `LoadAndValidateFramework` now taking a pointer as an argument

* Improve test case names in SDK tests

* Change string values in tests - no impact on test outcome

* Add tests for plugin framework handling of `region`

* Add tests for plugin framework handling of `zone`

* Add issue link in comment
…dles `access_token` values (#8814)

* Add initial version of plugin framework provider config test affected by inaccessible functions

* Refactor provider config tests to use plugin-framework types

* Add test case about handling of Unknown values for `project`

* Update tests to check values in BOTH the data model and provider config struct after `LoadAndValidateFramework` runs

* Add some tests for `credentials` in plugin framework provider, including test case that fails

* Update `LoadAndValidateFramework` to take a pointer to the provider data model, so mutations to the data within the function change the original struct

This enables tests to track how the data is mutated

* Add remaining `credentials` test cases to check PF/SDK config parity

* Make tests unset ADC ENV automatically, update comments to tests setting ADC ENV

* Add test for behaviour when credentials value is unknown

* Add comment referring devs to where unknown value test is implemented

* Remove duplicated test case

* Fix filename so it's generated correctly

* Remove fmt line

* Update `project` tests that are affected by `LoadAndValidateFramework` now taking a pointer as an argument

* Improve test case names in SDK tests

* Change string values in tests - no impact on test outcome

* Add tests for plugin framework handling of `region`

* Add tests for plugin framework handling of `zone`

* Add issue link in comment

* Update test names in SDK access_token unit tests

* Update PF provider config tests to log more unexpected error info

* Add initial version of the PF tests for `access_token`
…dles `user_project_override` values, fix potential bug (#8862)

* Add initial version of plugin framework provider config test affected by inaccessible functions

* Refactor provider config tests to use plugin-framework types

* Add test case about handling of Unknown values for `project`

* Update tests to check values in BOTH the data model and provider config struct after `LoadAndValidateFramework` runs

* Add some tests for `credentials` in plugin framework provider, including test case that fails

* Update `LoadAndValidateFramework` to take a pointer to the provider data model, so mutations to the data within the function change the original struct

This enables tests to track how the data is mutated

* Add remaining `credentials` test cases to check PF/SDK config parity

* Make tests unset ADC ENV automatically, update comments to tests setting ADC ENV

* Add test for behaviour when credentials value is unknown

* Add comment referring devs to where unknown value test is implemented

* Remove duplicated test case

* Fix filename so it's generated correctly

* Remove fmt line

* Update `project` tests that are affected by `LoadAndValidateFramework` now taking a pointer as an argument

* Update `FrameworkProviderConfig` to use plugin framework types package for `UserProjectOverride` boolean

* Add plugin framework version of tests for user_project_override, update test case names in SDK version of tests

* Add `user_project_override` test case for handling of unknown value
…e resources (#8432)

* Address added ipVersion parameter for reserving both ipv4 and ipv6 addresses
Instance added ipv6Address for assigning static ipv6 address to compute instance resource
ComputeInstance added test for internal ipv6 new field

* Fix yaml syntax issues

* Fixes

* Add ipv6PrefixLength and fix tests

* added ipv6PrefixLength and ipv6Address to compute_instance_template

* added ipv6PrefixLength and ipv6Address to region_compute_instance_template

---------

Co-authored-by: Luca Prete <[email protected]>
…dles `impersonate_service_account`, `impersonate_service_account_delegates` values (#8818)

* Add initial version of plugin framework provider config test affected by inaccessible functions

* Refactor provider config tests to use plugin-framework types

* Add test case about handling of Unknown values for `project`

* Update tests to check values in BOTH the data model and provider config struct after `LoadAndValidateFramework` runs

* Add some tests for `credentials` in plugin framework provider, including test case that fails

* Update `LoadAndValidateFramework` to take a pointer to the provider data model, so mutations to the data within the function change the original struct

This enables tests to track how the data is mutated

* Add remaining `credentials` test cases to check PF/SDK config parity

* Make tests unset ADC ENV automatically, update comments to tests setting ADC ENV

* Add test for behaviour when credentials value is unknown

* Add comment referring devs to where unknown value test is implemented

* Remove duplicated test case

* Fix filename so it's generated correctly

* Remove fmt line

* Update `project` tests that are affected by `LoadAndValidateFramework` now taking a pointer as an argument

* Add initial version of tests for `impersonate_service_account`

* Add test case for when impersonate_service_account is Unknown

* Add initial version of tests for `impersonate_service_account_delegates`
…secret_manager_secret` resource (#8838)

* Added support for automatic_replication field in google_secret_manager_secret resource

* Replaced deprecated field automatic with automatic_replication in terraform configs

* Improved the deprecation_message

* Changed the name of field from automatic_replication to auto

* Adjusted white spaces

* Added test case to cover the automatic field
IchordeDionysos and others added 5 commits September 12, 2023 13:42
* Adjust Firestore database etag description

* Adjust Firestore database create_time description

* Add update_time field to Firestore database

* Add uid field to Firestore database
* Fix issue #12883

Make one CloudResourceManager call per monitored_project terraform
resource rather than per (monitored_project * "sibling" projects)

* Add comment

* Correct build errors

* Fix build errors

* Add debug logs

* Fix printf

* Fix match between TF resource and API response

* Fix from non-matching to matching condition

* Correct documentation

* Correct documentation

* Fix go build issues

* Convert projectNumber to string

* Use correct string conversion

* Move comment

* Add debug logs to monitored project encoder

* correct delete_url
@google-cla
Copy link

google-cla bot commented Sep 13, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@modular-magician
Copy link
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 121 files changed, 15855 insertions(+), 1015 deletions(-))
Terraform Beta: Diff ( 136 files changed, 16216 insertions(+), 1036 deletions(-))
TF Conversion: Diff ( 18 files changed, 2489 insertions(+), 6 deletions(-))
TF OiCS: Diff ( 48 files changed, 1182 insertions(+), 12 deletions(-))

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_bigquery_table (33 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_bigquery_table" "primary" {
  materialized_view {
    allow_non_incremental_definition = # value needed
  }
  table_constraints {
    foreign_keys {
      column_references {
        referenced_column  = # value needed
        referencing_column = # value needed
      }
      name = # value needed
      referenced_table {
        dataset_id = # value needed
        project_id = # value needed
        table_id   = # value needed
      }
    }
    primary_key {
      columns = # value needed
    }
  }
}

Resource: google_compute_instance (203 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
  }
}

Resource: google_compute_instance_from_machine_image (5 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_from_machine_image" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_instance_from_template (10 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_from_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_instance_template (128 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_region_instance_template (46 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_region_instance_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_data_pipeline_pipeline (5 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_data_pipeline_pipeline" "primary" {
  scheduler_service_account_email = # value needed
  workload {
    dataflow_flex_template_request {
      launch_parameter {
        environment {
          flexrs_goal      = # value needed
          ip_configuration = # value needed
          kms_key_name     = # value needed
          subnetwork       = # value needed
        }
        launch_options = # value needed
      }
      validate_only = # value needed
    }
    dataflow_launch_template_request {
      launch_parameters {
        environment {
          ip_configuration = # value needed
          kms_key_name     = # value needed
          subnetwork       = # value needed
        }
      }
      validate_only = # value needed
    }
  }
}

@modular-magician
Copy link
Collaborator

Tests analytics

Total tests: 2942
Passed tests 2590
Skipped tests: 296
Affected tests: 56

Action taken

Found 56 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
TestAccApigeeTargetServer_apigeeTargetServerTest_update|TestAccApigeeTargetServer_apigeeTargetServerTestExample|TestAccBiglakeTable_biglakeTable_update|TestAccBiglakeTable_biglakeTableExample|TestAccCloudSchedulerJob_schedulerJobHttpExample|TestAccCloudSchedulerJob_schedulerJobPausedExample|TestAccComputeAddress_internalIpv6|TestAccComputeInstance_internalIPv6|TestAccContainerNodePool_withMultiNicNetworkConfig|TestAccContainerAttachedCluster_update|TestAccContainerAttachedCluster_containerAttachedClusterFullExample|TestAccDataPipelinePipeline_basicFlexTemplate|TestAccDataPipelinePipeline_basicLaunchTemplate|TestAccDataPipelinePipeline_dataPipelinePipelineExample|TestAccDialogflowCXTestCase_dialogflowcxTestCaseFullExample|TestAccDialogflowCXTestCase_update|TestAccFirestoreDatabase_updatePitrEnablement|TestAccFirestoreDatabase_updateConcurrencyMode|TestAccFirestoreDatabase_firestoreDefaultDatabaseInDatastoreModeExample|TestAccFirestoreDatabase_firestoreDefaultDatabaseExample|TestAccFirestoreDatabase_firestoreDatabaseInDatastoreModeExample|TestAccFirestoreDatabase_firestoreDatabaseExample|TestAccGKEHub2Scope_gkehubScopeBasicExample_update|TestAccGKEHub2Namespace_gkehubNamespaceBasicExample_update|TestAccGKEHub2Scope_gkehubScopeBasicExample|TestAccGKEHub2ScopeIamBindingGenerated|TestAccGKEHub2ScopeIamMemberGenerated|TestAccGKEHub2ScopeIamPolicyGenerated|TestAccGKEHub2Namespace_gkehubNamespaceBasicExample|TestAccGKEHub2MembershipBinding_gkehubMembershipBindingBasicExample_update|TestAccGKEHub2MembershipBinding_gkehubMembershipBindingBasicExample|TestAccGkeonpremBareMetalCluster_bareMetalClusterUpdateBasic|TestAccGkeonpremBareMetalCluster_bareMetalClusterUpdateBgpLb|TestAccGkeonpremBareMetalCluster_bareMetalClusterUpdateManualLb|TestAccGkeonpremVmwareCluster_gkeonpremVmwareClusterManuallbExample|TestAccGkeonpremBareMetalCluster_gkeonpremBareMetalClusterManuallbExample|TestAccMonitoringMonitoredProject_monitoringMonitoredProjectBasicExample|TestAccMonitoringMonitoredProject_projectNumLongForm|TestAccMonitoringMonitoredProject_projectNumShortForm|TestAccMonitoringMonitoredProject_monitoringMonitoredProjectLongFormExample|TestAccPubsubSubscription_pushNoWrapperEmpty|TestAccDataSourceGoogleServiceAccountAccessToken_basic|TestAccDataSourceGoogleServiceAccountIdToken_impersonation|TestAccSecretManagerSecret_automaticCmekUpdate|TestAccSecretManagerSecretVersion_secretVersionDeletionPolicyAbandonExample|TestAccSecretManagerSecretVersion_secretVersionDeletionPolicyDisableExample|TestAccSecretManagerSecret_secretWithAutomaticCmekExample|TestAccStorageBucket_basicWithAutoclass|TestAccStorageInsightsReportConfig_storageInsightsReportConfigExample|TestAccStorageInsightsReportConfig_update|TestAccVertexAIIndexEndpoint_vertexAiIndexEndpointWithPublicEndpointExample|TestAccWorkstationsWorkstationIamBindingGenerated|TestAccWorkstationsWorkstation_update|TestAccWorkstationsWorkstation_workstationBasicExample|TestAccWorkstationsWorkstationIamMemberGenerated|TestAccWorkstationsWorkstationIamPolicyGenerated

Get to know how VCR tests work

@modular-magician
Copy link
Collaborator

$\textcolor{green}{\textsf{Tests passed during RECORDING mode:}}$
TestAccApigeeTargetServer_apigeeTargetServerTest_update[Debug log]
TestAccApigeeTargetServer_apigeeTargetServerTestExample[Debug log]
TestAccBiglakeTable_biglakeTable_update[Debug log]
TestAccBiglakeTable_biglakeTableExample[Debug log]
TestAccCloudSchedulerJob_schedulerJobHttpExample[Debug log]
TestAccCloudSchedulerJob_schedulerJobPausedExample[Debug log]
TestAccComputeAddress_internalIpv6[Debug log]
TestAccComputeInstance_internalIPv6[Debug log]
TestAccContainerNodePool_withMultiNicNetworkConfig[Debug log]
TestAccContainerAttachedCluster_update[Debug log]
TestAccContainerAttachedCluster_containerAttachedClusterFullExample[Debug log]
TestAccDataPipelinePipeline_basicFlexTemplate[Debug log]
TestAccDataPipelinePipeline_basicLaunchTemplate[Debug log]
TestAccDataPipelinePipeline_dataPipelinePipelineExample[Debug log]
TestAccDialogflowCXTestCase_dialogflowcxTestCaseFullExample[Debug log]
TestAccDialogflowCXTestCase_update[Debug log]
TestAccFirestoreDatabase_updatePitrEnablement[Debug log]
TestAccFirestoreDatabase_updateConcurrencyMode[Debug log]
TestAccFirestoreDatabase_firestoreDefaultDatabaseInDatastoreModeExample[Debug log]
TestAccFirestoreDatabase_firestoreDefaultDatabaseExample[Debug log]
TestAccFirestoreDatabase_firestoreDatabaseInDatastoreModeExample[Debug log]
TestAccFirestoreDatabase_firestoreDatabaseExample[Debug log]
TestAccGKEHub2Scope_gkehubScopeBasicExample_update[Debug log]
TestAccGKEHub2Namespace_gkehubNamespaceBasicExample_update[Debug log]
TestAccGKEHub2Scope_gkehubScopeBasicExample[Debug log]
TestAccGKEHub2ScopeIamBindingGenerated[Debug log]
TestAccGKEHub2ScopeIamMemberGenerated[Debug log]
TestAccGKEHub2ScopeIamPolicyGenerated[Debug log]
TestAccGKEHub2Namespace_gkehubNamespaceBasicExample[Debug log]
TestAccGKEHub2MembershipBinding_gkehubMembershipBindingBasicExample_update[Debug log]
TestAccGkeonpremBareMetalCluster_bareMetalClusterUpdateBasic[Debug log]
TestAccGkeonpremBareMetalCluster_bareMetalClusterUpdateBgpLb[Debug log]
TestAccGkeonpremBareMetalCluster_bareMetalClusterUpdateManualLb[Debug log]
TestAccGkeonpremVmwareCluster_gkeonpremVmwareClusterManuallbExample[Debug log]
TestAccGkeonpremBareMetalCluster_gkeonpremBareMetalClusterManuallbExample[Debug log]
TestAccMonitoringMonitoredProject_monitoringMonitoredProjectBasicExample[Debug log]
TestAccMonitoringMonitoredProject_projectNumLongForm[Debug log]
TestAccMonitoringMonitoredProject_projectNumShortForm[Debug log]
TestAccMonitoringMonitoredProject_monitoringMonitoredProjectLongFormExample[Debug log]
TestAccPubsubSubscription_pushNoWrapperEmpty[Debug log]
TestAccDataSourceGoogleServiceAccountAccessToken_basic[Debug log]
TestAccDataSourceGoogleServiceAccountIdToken_impersonation[Debug log]
TestAccSecretManagerSecret_automaticCmekUpdate[Debug log]
TestAccSecretManagerSecretVersion_secretVersionDeletionPolicyAbandonExample[Debug log]
TestAccSecretManagerSecretVersion_secretVersionDeletionPolicyDisableExample[Debug log]
TestAccSecretManagerSecret_secretWithAutomaticCmekExample[Debug log]
TestAccStorageBucket_basicWithAutoclass[Debug log]
TestAccStorageInsightsReportConfig_update[Debug log]
TestAccVertexAIIndexEndpoint_vertexAiIndexEndpointWithPublicEndpointExample[Debug log]
TestAccWorkstationsWorkstationIamBindingGenerated[Debug log]
TestAccWorkstationsWorkstation_update[Debug log]
TestAccWorkstationsWorkstation_workstationBasicExample[Debug log]
TestAccWorkstationsWorkstationIamMemberGenerated[Debug log]
TestAccWorkstationsWorkstationIamPolicyGenerated[Debug log]

Rerun these tests in REPLAYING mode to catch issues

$\textcolor{green}{\textsf{No issues found for passed tests after REPLAYING rerun.}}$


$\textcolor{red}{\textsf{Tests failed during RECORDING mode:}}$
TestAccGKEHub2MembershipBinding_gkehubMembershipBindingBasicExample[Error message] [Debug log]
TestAccStorageInsightsReportConfig_storageInsightsReportConfigExample[Error message] [Debug log]

$\textcolor{red}{\textsf{Please fix these to complete your PR.}}$
View the build log or the debug log for each test

@modular-magician
Copy link
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 121 files changed, 15854 insertions(+), 1015 deletions(-))
Terraform Beta: Diff ( 136 files changed, 16215 insertions(+), 1036 deletions(-))
TF Conversion: Diff ( 18 files changed, 2489 insertions(+), 6 deletions(-))
TF OiCS: Diff ( 48 files changed, 1182 insertions(+), 12 deletions(-))

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_compute_instance (203 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
  }
}

Resource: google_compute_instance_from_machine_image (5 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_from_machine_image" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_instance_from_template (10 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_from_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_instance_template (128 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_region_instance_template (46 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_region_instance_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_data_pipeline_pipeline (5 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_data_pipeline_pipeline" "primary" {
  scheduler_service_account_email = # value needed
  workload {
    dataflow_flex_template_request {
      launch_parameter {
        environment {
          flexrs_goal      = # value needed
          ip_configuration = # value needed
          kms_key_name     = # value needed
          subnetwork       = # value needed
        }
        launch_options = # value needed
      }
      validate_only = # value needed
    }
    dataflow_launch_template_request {
      launch_parameters {
        environment {
          ip_configuration = # value needed
          kms_key_name     = # value needed
          subnetwork       = # value needed
        }
      }
      validate_only = # value needed
    }
  }
}

@modular-magician
Copy link
Collaborator

Tests analytics

Total tests: 3036
Passed tests 2724
Skipped tests: 304
Affected tests: 8

Action taken

Found 8 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
TestAccBigQueryTable_ForeignKey|TestAccBigQueryTable_MaterializedView_NonIncremental_basic|TestAccBigQueryTable_PrimaryKey|TestAccBigQueryTable_MaterializedView_DailyTimePartioning_Update|TestAccBigQueryTable_updateTableConstraints|TestAccBigQueryTable_MaterializedView_DailyTimePartioning_Basic|TestAccGKEHub2MembershipBinding_gkehubMembershipBindingBasicExample|TestAccStorageInsightsReportConfig_storageInsightsReportConfigExample

Get to know how VCR tests work

@modular-magician
Copy link
Collaborator

$\textcolor{green}{\textsf{Tests passed during RECORDING mode:}}$
TestAccBigQueryTable_ForeignKey[Debug log]
TestAccBigQueryTable_MaterializedView_NonIncremental_basic[Debug log]
TestAccBigQueryTable_PrimaryKey[Debug log]
TestAccBigQueryTable_MaterializedView_DailyTimePartioning_Update[Debug log]
TestAccBigQueryTable_updateTableConstraints[Debug log]
TestAccBigQueryTable_MaterializedView_DailyTimePartioning_Basic[Debug log]
TestAccGKEHub2MembershipBinding_gkehubMembershipBindingBasicExample[Debug log]
TestAccStorageInsightsReportConfig_storageInsightsReportConfigExample[Debug log]

Rerun these tests in REPLAYING mode to catch issues

$\textcolor{green}{\textsf{No issues found for passed tests after REPLAYING rerun.}}$


$\textcolor{green}{\textsf{All tests passed!}}$
View the build log or the debug log for each test

@shuyama1 shuyama1 requested a review from c2thorn September 14, 2023 15:14
@shuyama1
Copy link
Member Author

tgc failed with error:

Step #17 - "tgc-test": github.com/GoogleCloudPlatform/terraform-google-conversion/v2/cai2hcl imports
Step #17 - "tgc-test":     github.com/hashicorp/terraform-provider-google-beta/google-beta: module github.com/hashicorp/terraform-provider-google-beta@latest found (v1.20.0, replaced by github.com/modular-magician/terraform-provider-google-beta@v1.20.1-0.20230914035502-a4946bde9df0), but does not contain package github.com/hashicorp/terraform-provider-google-beta/google-beta
Step #17 - "tgc-test": GO111MODULE=on go test -short ./...
Step #17 - "tgc-test": go: updates to go.mod needed; to update it:
Step #17 - "tgc-test":     go mod tidy
Step #17 - "tgc-test": make: *** [Makefile:7: test] Error 1
Step #17 - "tgc-test":   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Step #17 - "tgc-test":                                  Dload  Upload   Total   Spent    Left  Speed

I think it might be related to some recent change and GoogleCloudPlatform/terraform-google-conversion@d1699f0 should fix it (?). I cherry-pick it the 5.0.0 branch in tgc repo. Rerun the build now.

/gcbrun

@modular-magician
Copy link
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 121 files changed, 15854 insertions(+), 1015 deletions(-))
Terraform Beta: Diff ( 136 files changed, 16215 insertions(+), 1036 deletions(-))
TF Conversion: Diff ( 18 files changed, 2489 insertions(+), 6 deletions(-))
TF OiCS: Diff ( 48 files changed, 1182 insertions(+), 12 deletions(-))

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_compute_instance (203 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
  }
}

Resource: google_compute_instance_from_machine_image (5 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_from_machine_image" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_instance_from_template (10 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_from_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_instance_template (128 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_instance_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_compute_region_instance_template (46 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_region_instance_template" "primary" {
  network_interface {
    internal_ipv6_prefix_length = # value needed
    ipv6_address                = # value needed
  }
}

Resource: google_data_pipeline_pipeline (5 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_data_pipeline_pipeline" "primary" {
  scheduler_service_account_email = # value needed
  workload {
    dataflow_flex_template_request {
      launch_parameter {
        environment {
          flexrs_goal      = # value needed
          ip_configuration = # value needed
          kms_key_name     = # value needed
          subnetwork       = # value needed
        }
        launch_options = # value needed
      }
      validate_only = # value needed
    }
    dataflow_launch_template_request {
      launch_parameters {
        environment {
          ip_configuration = # value needed
          kms_key_name     = # value needed
          subnetwork       = # value needed
        }
      }
      validate_only = # value needed
    }
  }
}

@modular-magician
Copy link
Collaborator

Tests analytics

Total tests: 3036
Passed tests 2731
Skipped tests: 304
Affected tests: 1

Action taken

Found 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
TestAccBigtableAppProfile_bigtableAppProfileSingleclusterExample

Get to know how VCR tests work

@modular-magician
Copy link
Collaborator

$\textcolor{green}{\textsf{Tests passed during RECORDING mode:}}$
TestAccBigtableAppProfile_bigtableAppProfileSingleclusterExample[Debug log]

Rerun these tests in REPLAYING mode to catch issues

$\textcolor{green}{\textsf{No issues found for passed tests after REPLAYING rerun.}}$


$\textcolor{green}{\textsf{All tests passed!}}$
View the build log or the debug log for each test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.