Skip to content

ING-1464: Added support for reporting mixed cluster editions.#367

Merged
brett19 merged 1 commit into
masterfrom
ING-1464_mixed-cluster-editions
Apr 28, 2026
Merged

ING-1464: Added support for reporting mixed cluster editions.#367
brett19 merged 1 commit into
masterfrom
ING-1464_mixed-cluster-editions

Conversation

@brett19
Copy link
Copy Markdown
Member

@brett19 brett19 commented Apr 27, 2026

No description provided.

@brett19 brett19 requested review from Westwooo, chvck and Copilot April 27, 2026 19:47
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

This PR updates the XDCR GetClusterInfo response to correctly report “mixed” Couchbase cluster editions, aligning the gateway’s behavior with updated gocbcorex/goprotostellar capabilities.

Changes:

  • Bump github.com/couchbase/gocbcorex and github.com/couchbase/goprotostellar to versions that support mixed-edition reporting.
  • Update XDCR GetClusterInfo to use aggregated cluster info and map editions to Enterprise / Community / Mixed.
  • Expand the XDCR integration test to accept the new CLUSTER_EDITION_MIXED enum value.

Reviewed changes

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

File Description
go.mod Updates Couchbase dependency versions needed for aggregated/mixed edition support.
go.sum Locks updated module checksums for the bumped Couchbase dependencies.
gateway/dataimpl/server_v1/xdcrserver.go Switches to aggregated cluster info and adds Mixed edition selection logic.
gateway/test/xdcr_test.go Updates test assertions to allow the new Mixed edition enum.

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

@brett19 brett19 merged commit 946d29e into master Apr 28, 2026
40 of 48 checks passed
@brett19 brett19 deleted the ING-1464_mixed-cluster-editions branch April 28, 2026 17:17
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