Skip to content

Adding desired_auto_created_endpoints Virtual field to Memorystore instance#13819

Merged
NickElliot merged 9 commits intoGoogleCloudPlatform:mainfrom
NA2047:MemorystoreEndpoints_test
May 19, 2025
Merged

Adding desired_auto_created_endpoints Virtual field to Memorystore instance#13819
NickElliot merged 9 commits intoGoogleCloudPlatform:mainfrom
NA2047:MemorystoreEndpoints_test

Conversation

@NA2047
Copy link
Copy Markdown
Contributor

@NA2047 NA2047 commented Apr 29, 2025

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

memorystore: added field `desired_auto_created_endpoints` for `google_memorystore_instance` resource

@NA2047 NA2047 marked this pull request as draft April 29, 2025 22:04
@github-actions
Copy link
Copy Markdown

Hello! I am a robot. Tests will require approval from a repository maintainer to run. Googlers: see go/terraform-auto-test-runs to set up automatic test runs.

@NickElliot, 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.

@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 6 files changed, 125 insertions(+), 7 deletions(-))
google-beta provider: Diff ( 6 files changed, 125 insertions(+), 7 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 39 insertions(+), 1 deletion(-))
Open in Cloud Shell: Diff ( 2 files changed, 2 insertions(+), 2 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 19
Passed tests: 3
Skipped tests: 0
Affected tests: 16

Click here to see the affected service packages
  • memorystore

Action taken

Found 16 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstanceDatasourceConfig
  • TestAccMemorystoreInstanceDesiredUserCreatedEndpoints_memorystoreInstanceDesiredUserAndAutoCreatedEndpointsExample
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
Collaborator

🟢 Tests passed during RECORDING mode:
TestAccMemorystoreInstanceDesiredUserCreatedEndpoints_memorystoreInstanceDesiredUserAndAutoCreatedEndpointsExample [Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.


🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstanceDatasourceConfig [Error message] [Debug log]
TestAccMemorystoreInstance_automatedBackupConfig [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Error message] [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Error message] [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Error message] [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Error message] [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Error message] [Debug log]
TestAccMemorystoreInstance_updateNodeType [Error message] [Debug log]
TestAccMemorystoreInstance_updatePersistence [Error message] [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Error message] [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Error message] [Debug log]
TestAccMemorystoreInstance_updateShardCount [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@NA2047 NA2047 force-pushed the MemorystoreEndpoints_test branch 2 times, most recently from 3725e21 to 52d1bc7 Compare April 30, 2025 22:53
@NA2047 NA2047 force-pushed the MemorystoreEndpoints_test branch from 52d1bc7 to f1965ed Compare April 30, 2025 22:54
@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 7 files changed, 145 insertions(+), 29 deletions(-))
google-beta provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 39 insertions(+), 1 deletion(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
google-beta provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 39 insertions(+), 1 deletion(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

1 similar comment
@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
google-beta provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 39 insertions(+), 1 deletion(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 19
Passed tests: 4
Skipped tests: 0
Affected tests: 15

Click here to see the affected service packages
  • memorystore

Action taken

Found 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstanceDatasourceConfig
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
google-beta provider: Diff ( 7 files changed, 144 insertions(+), 25 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 39 insertions(+), 1 deletion(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 19
Passed tests: 4
Skipped tests: 0
Affected tests: 15

Click here to see the affected service packages
  • memorystore

Action taken

Found 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstanceDatasourceConfig
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

2 similar comments
@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 19
Passed tests: 4
Skipped tests: 0
Affected tests: 15

Click here to see the affected service packages
  • memorystore

Action taken

Found 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstanceDatasourceConfig
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 19
Passed tests: 4
Skipped tests: 0
Affected tests: 15

Click here to see the affected service packages
  • memorystore

Action taken

Found 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstanceDatasourceConfig
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
Collaborator

🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstanceDatasourceConfig [Error message] [Debug log]
TestAccMemorystoreInstance_automatedBackupConfig [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Error message] [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Error message] [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Error message] [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Error message] [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Error message] [Debug log]
TestAccMemorystoreInstance_updateNodeType [Error message] [Debug log]
TestAccMemorystoreInstance_updatePersistence [Error message] [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Error message] [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Error message] [Debug log]
TestAccMemorystoreInstance_updateShardCount [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@modular-magician
Copy link
Copy Markdown
Collaborator

🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstanceDatasourceConfig [Error message] [Debug log]
TestAccMemorystoreInstance_automatedBackupConfig [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Error message] [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Error message] [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Error message] [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Error message] [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Error message] [Debug log]
TestAccMemorystoreInstance_updateNodeType [Error message] [Debug log]
TestAccMemorystoreInstance_updatePersistence [Error message] [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Error message] [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Error message] [Debug log]
TestAccMemorystoreInstance_updateShardCount [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@modular-magician
Copy link
Copy Markdown
Collaborator

🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstanceDatasourceConfig [Error message] [Debug log]
TestAccMemorystoreInstance_automatedBackupConfig [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Error message] [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Error message] [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Error message] [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Error message] [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Error message] [Debug log]
TestAccMemorystoreInstance_updateNodeType [Error message] [Debug log]
TestAccMemorystoreInstance_updatePersistence [Error message] [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Error message] [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Error message] [Debug log]
TestAccMemorystoreInstance_updateShardCount [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@modular-magician
Copy link
Copy Markdown
Collaborator

🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstanceDatasourceConfig [Error message] [Debug log]
TestAccMemorystoreInstance_automatedBackupConfig [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Error message] [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Error message] [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Error message] [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Error message] [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Error message] [Debug log]
TestAccMemorystoreInstance_updateNodeType [Error message] [Debug log]
TestAccMemorystoreInstance_updatePersistence [Error message] [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Error message] [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Error message] [Debug log]
TestAccMemorystoreInstance_updateShardCount [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 8 files changed, 148 insertions(+), 29 deletions(-))
google-beta provider: Diff ( 8 files changed, 148 insertions(+), 29 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 39 insertions(+), 1 deletion(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 19
Passed tests: 4
Skipped tests: 0
Affected tests: 15

Click here to see the affected service packages
  • memorystore

Action taken

Found 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstanceDatasourceConfig
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
Collaborator

🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstanceDatasourceConfig [Error message] [Debug log]
TestAccMemorystoreInstance_automatedBackupConfig [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Error message] [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Error message] [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Error message] [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Error message] [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Error message] [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Error message] [Debug log]
TestAccMemorystoreInstance_updateNodeType [Error message] [Debug log]
TestAccMemorystoreInstance_updatePersistence [Error message] [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Error message] [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Error message] [Debug log]
TestAccMemorystoreInstance_updateShardCount [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 8 files changed, 278 insertions(+), 118 deletions(-))
google-beta provider: Diff ( 8 files changed, 278 insertions(+), 118 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 59 insertions(+), 22 deletions(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 20
Passed tests: 5
Skipped tests: 0
Affected tests: 15

Click here to see the affected service packages
  • memorystore

Action taken

Found 15 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
  • TestAccMemorystoreInstance_automatedBackupConfig
  • TestAccMemorystoreInstance_deprecatedDesiredPscAutoConnections
  • TestAccMemorystoreInstance_memorystoreInstanceBasicExample
  • TestAccMemorystoreInstance_memorystoreInstanceFullExample
  • TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample
  • TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample
  • TestAccMemorystoreInstance_switchoverAndDetachSecondary
  • TestAccMemorystoreInstance_updateDeletionProtection
  • TestAccMemorystoreInstance_updateEngineVersion
  • TestAccMemorystoreInstance_updateInstanceEndpoints
  • TestAccMemorystoreInstance_updateNodeType
  • TestAccMemorystoreInstance_updatePersistence
  • TestAccMemorystoreInstance_updateRedisConfigs
  • TestAccMemorystoreInstance_updateReplicaCount
  • TestAccMemorystoreInstance_updateShardCount

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
Collaborator

🟢 Tests passed during RECORDING mode:
TestAccMemorystoreInstance_automatedBackupConfig [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceBasicExample [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceFullExample [Debug log]
TestAccMemorystoreInstance_memorystoreInstancePersistenceAofExample [Debug log]
TestAccMemorystoreInstance_memorystoreInstanceSecondaryInstanceExample [Debug log]
TestAccMemorystoreInstance_switchoverAndDetachSecondary [Debug log]
TestAccMemorystoreInstance_updateDeletionProtection [Debug log]
TestAccMemorystoreInstance_updateEngineVersion [Debug log]
TestAccMemorystoreInstance_updateInstanceEndpoints [Debug log]
TestAccMemorystoreInstance_updateNodeType [Debug log]
TestAccMemorystoreInstance_updatePersistence [Debug log]
TestAccMemorystoreInstance_updateRedisConfigs [Debug log]
TestAccMemorystoreInstance_updateReplicaCount [Debug log]
TestAccMemorystoreInstance_updateShardCount [Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.


🔴 Tests failed during RECORDING mode:
TestAccMemorystoreInstance_deprecatedDesiredPscAutoConnections [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 8 files changed, 282 insertions(+), 117 deletions(-))
google-beta provider: Diff ( 8 files changed, 282 insertions(+), 117 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 62 insertions(+), 22 deletions(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 20
Passed tests: 19
Skipped tests: 0
Affected tests: 1

Click here to see the affected service packages
  • memorystore

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
  • TestAccMemorystoreInstance_deprecatedDesiredPscAutoConnections

Get to know how VCR tests work

@modular-magician
Copy link
Copy Markdown
Collaborator

🟢 Tests passed during RECORDING mode:
TestAccMemorystoreInstance_deprecatedDesiredPscAutoConnections [Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.


🟢 All tests passed!

View the build log or the debug log for each test

@NA2047 NA2047 marked this pull request as ready for review May 7, 2025 20:52
@NA2047 NA2047 requested a review from NickElliot May 7, 2025 20:52
@modular-magician
Copy link
Copy Markdown
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.

google provider: Diff ( 8 files changed, 284 insertions(+), 119 deletions(-))
google-beta provider: Diff ( 8 files changed, 284 insertions(+), 119 deletions(-))
terraform-google-conversion: Diff ( 1 file changed, 62 insertions(+), 22 deletions(-))
Open in Cloud Shell: Diff ( 5 files changed, 10 insertions(+), 10 deletions(-))

Missing doc report (experimental)

The following resources have fields missing in documents.

  • google_memorystore_instance
    • Expected Document Path: /website/docs/r/memorystore_instance.html.markdown
    • Fields: [desired_auto_created_endpoints.network desired_auto_created_endpoints.project_id]

@modular-magician
Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 20
Passed tests: 20
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages
  • memorystore

🟢 All tests passed!

View the build log

@github-actions
Copy link
Copy Markdown

@NickElliot This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.

@github-actions
Copy link
Copy Markdown

@GoogleCloudPlatform/terraform-team @NickElliot This PR has been waiting for review for 1 week. Please take a look! Use the label disable-review-reminders to disable these notifications.

Copy link
Copy Markdown
Contributor

@NickElliot NickElliot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some questions and potentially a needed change:

  1. could you re-add a test that uses desired_psc_auto_connections for the time being, as it has not been totally removed yet
  2. is it possible to do the in-place upgrade to desired_auto_created_endpoints from desired_psc_auto_connections? If so could you add an upgrade test that includes this process (i.e. step 1 use deprecated field, step 2 swap to new field)

@NA2047
Copy link
Copy Markdown
Contributor Author

NA2047 commented May 16, 2025

some questions and potentially a needed change:

  1. could you re-add a test that uses desired_psc_auto_connections for the time being, as it has not been totally removed yet
  2. is it possible to do the in-place upgrade to desired_auto_created_endpoints from desired_psc_auto_connections? If so could you add an upgrade test that includes this process (i.e. step 1 use deprecated field, step 2 swap to new field)

Hey @NickElliot

  1. I have added the following to cover this TestAccMemorystoreInstance_deprecatedDesiredPscAutoConnections
  2. These are both set to immutable so it will recreate this instance, If you would still like me to write a test for this, please let me know :)

@github-actions github-actions bot requested a review from NickElliot May 16, 2025 20:39
Copy link
Copy Markdown
Contributor

@NickElliot NickElliot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, sorry missed that, LGTM!

@NickElliot NickElliot added this pull request to the merge queue May 19, 2025
Merged via the queue into GoogleCloudPlatform:main with commit cbda0f1 May 19, 2025
24 checks passed
BBBmau pushed a commit to BBBmau/magic-modules that referenced this pull request May 20, 2025
NandiniAgrawal15 pushed a commit to NandiniAgrawal15/magic-modules that referenced this pull request Jun 5, 2025
jingqizz pushed a commit to jingqizz/magic-modules that referenced this pull request Jul 9, 2025
NandiniAgrawal15 pushed a commit to NandiniAgrawal15/magic-modules that referenced this pull request Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants