Skip to content

Conversation

@zhangyangyu
Copy link
Collaborator

No description provided.

@zhangyangyu zhangyangyu requested a review from Copilot September 3, 2025 14:09
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the TiDB Cloud API from "Serverless" to "Starter and Essential", expanding support to allow creating essential clusters on AWS. The changes reflect a rebranding and expansion of the API to support both Starter (formerly Serverless) and Essential cluster types.

Key changes include:

  • API documentation updates to reflect new naming and support for both cluster types
  • Removal of cloud storage related models (S3, GCS, OSS, Azure Blob storage configurations)
  • Simplification of audit log configuration by removing cloud storage options
  • Updates to model structures for partial cluster updates

Reviewed Changes

Copilot reviewed 87 out of 87 changed files in this pull request and generated no comments.

Show a summary per file
File Description
utils.go, response.go, model_v1beta1_region_cloud_provider.go Updated API documentation header comments from "Serverless" to "Starter and Essential"
model_v1beta1_partial_update_cluster_request_partial_update_cluster.go Updated struct comments to be more descriptive and user-friendly
model_v1beta1_cluster_service_partial_update_cluster_body*.go Added new model files for partial cluster updates with simplified structure
model_v1beta1_cluster__cloud_storage.go Removed cloud storage models (S3, GCS, OSS, Azure Blob)
model_v1beta1_cluster_audit_log_config.go Simplified audit log config by removing cloud storage and rotation options
model_v1beta1_cluster_auto_scaling.go, model_v1beta1_cluster_automated_backup_policy.go Updated field documentation to be more descriptive
Comments suppressed due to low confidence (1)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@zhangyangyu zhangyangyu merged commit 0da77c4 into main Sep 3, 2025
7 checks passed
@zhangyangyu zhangyangyu deleted the allow-aws-essential branch September 3, 2025 16:11
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.

2 participants