bigtable: fix TestAccBigtableTable_familyType immutable-field ExpectError regex - #18760
bigtable: fix TestAccBigtableTable_familyType immutable-field ExpectError regex#18760SirGitsalot wants to merge 1 commit into
Conversation
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit c3c0ce5: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode 🟢 All tests passed in Replaying mode! No Recording was needed. View the replaying VCR build log @SirGitsalot, @annguy3n VCR tests complete for c3c0ce5! |
|
@modular-magician reassign-reviewer |
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @ScottSuarez, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
Fixes nightly acceptance test failure for
TestAccBigtableTable_familyType.Root Cause & Fix
When the test attempts to change a column family's aggregate type (
intmax->intmin), the Bigtable API rejects the modification and returns an error indicating immutable fields. Previously, the error message only listed a single field:Immutable fields 'value_type.aggregate_type' cannot be updated.The API can now include additional immutable fields in the message (e.g.,
Immutable fields 'value_type.aggregate_type,sql_type' cannot be updated.).This fix loosens the
ExpectErrorregex inmmv1/third_party/terraform/services/bigtable/resource_bigtable_table_test.goto tolerate additional fields while continuing to verify thatvalue_type.aggregate_typeis reported as immutable.