Skip to content

Backport of Fix catalog service endpoint when querying for a peer service into release/1.20.x#22261

Closed
hc-github-team-consul-core wants to merge 4 commits intorelease/1.20.xfrom
backport/f-catalog-service-peer/ultimately-optimal-opossum
Closed

Backport of Fix catalog service endpoint when querying for a peer service into release/1.20.x#22261
hc-github-team-consul-core wants to merge 4 commits intorelease/1.20.xfrom
backport/f-catalog-service-peer/ultimately-optimal-opossum

Conversation

@hc-github-team-consul-core
Copy link
Collaborator

Backport

This PR is auto-generated from #22189 to be assessed for backporting due to the inclusion of the label backport/1.20.

🚨

Warning automatic cherry-pick of commits failed. If the first commit failed,
you will see a blank no-op commit below. If at least one commit succeeded, you
will see the cherry-picked commits up to, not including, the commit where
the merge conflict occurred.

The person who merged in the original PR is:
@nitin-sachdev-29
This person should manually cherry-pick the original PR into a new backport PR,
and close this one when the manual backport PR is merged in.

merge conflict error: unable to process merge commit: "6cdaef235356f6ec61fdd690f55eaf4bb844c67b", automatic backport requires rebase workflow

The below text is copied from the body of the original PR.


When calling /catalog/service/<service>?peer=<peer-name> no services are returned even when they are configured as exported in the peer and shown correctly in the UI.

Making the same call on health /health/service/<service>?peer=<peer-name> works fine.

This fixes #20792


Overview of commits

Copy link
Collaborator

Choose a reason for hiding this comment

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

Auto approved Consul Bot automated PR

@nitin-sachdev-29 nitin-sachdev-29 marked this pull request as ready for review June 4, 2025 12:17
@nitin-sachdev-29 nitin-sachdev-29 requested review from a team as code owners June 4, 2025 12:17
@nitin-sachdev-29 nitin-sachdev-29 enabled auto-merge (squash) June 4, 2025 12:17
@nitin-sachdev-29 nitin-sachdev-29 disabled auto-merge June 4, 2025 12:31
@nitin-sachdev-29
Copy link
Contributor

closing this PR since CE release/1.20.x branch is no longer maintained

@nitin-sachdev-29 nitin-sachdev-29 deleted the backport/f-catalog-service-peer/ultimately-optimal-opossum branch June 10, 2025 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/docs Documentation needs to be created/updated/clarified

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants