Skip to content

ING-1431: Implement cluster edition in XDCR GetClusterInfo.#357

Merged
brett19 merged 1 commit into
masterfrom
ING-1431_cluster-edition
Mar 17, 2026
Merged

ING-1431: Implement cluster edition in XDCR GetClusterInfo.#357
brett19 merged 1 commit into
masterfrom
ING-1431_cluster-edition

Conversation

@brett19
Copy link
Copy Markdown
Member

@brett19 brett19 commented Mar 17, 2026

No description provided.

chvck
chvck previously approved these changes Mar 17, 2026
Copy link
Copy Markdown

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

Updates the XDCR GetClusterInfo response to include the cluster edition (enterprise vs community), aligning the gateway’s XDCR API with the newer goprotostellar proto definitions.

Changes:

  • Bump github.com/couchbase/goprotostellar to a newer pseudo-version containing the cluster edition field.
  • Populate GetClusterInfoResponse.Edition based on clusterInfo.IsEnterprise.
  • Extend XDCR integration test to validate the edition field is present.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
go.mod Updates goprotostellar dependency to version that includes cluster edition support.
go.sum Syncs checksums for the updated goprotostellar version.
gateway/dataimpl/server_v1/xdcrserver.go Sets Edition in GetClusterInfo responses based on cluster info.
gateway/test/xdcr_test.go Adds an assertion that Edition is non-nil in XDCR GetClusterInfo test.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread gateway/test/xdcr_test.go
@brett19 brett19 merged commit 66cdf41 into master Mar 17, 2026
29 of 30 checks passed
@brett19 brett19 deleted the ING-1431_cluster-edition branch March 17, 2026 20:39
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.

3 participants