Skip to content

fix(pci-databases-analytics): use /topicAcl endpoint for Kafka topic ACLs - #23068

Open
abstrask wants to merge 1 commit into
ovh:masterfrom
abstrask:fix/kafka-topic-acl-endpoint
Open

abstrask wants to merge 1 commit into
ovh:masterfrom
abstrask:fix/kafka-topic-acl-endpoint

Conversation

@abstrask

Copy link
Copy Markdown

Description

The Public Cloud Databases / Analytics app failed to load Kafka Topic ACLs: the data layer (topicAcl.api.ts) called the removed /cloud/project/{projectId}/database/{engine}/{serviceId}/acl endpoint.

This updates the three calls (getTopicAcls, addTopicAcl, deleteTopicAcl) to /topicAcl. The response model is unchanged, so nothing else changes.

Ticket Reference: Fixes #23066

Additional Information

  • Evidence: the current API only documents /topicAcl (console);
    OVH's schema marks the legacy /acl path deprecated/deleted with x-replacement-route: .../topicAcl. The official ovhcloud-cli already uses /topicAcl.
  • Tested locally against a real Kafka cluster: the Topic ACLs tab lists existing ACLs and add/delete work.
image

…ACLs

Signed-off-by: Rasmus Rask <rra@cerivo.com>
@abstrask
abstrask requested a review from a team as a code owner July 28, 2026 13:04
@github-actions

Copy link
Copy Markdown
Contributor

This Pull Request is stale due to inactivity since 30 days. If no activity happens for 15 more days from now, it will be closed.

@github-actions github-actions Bot added the stale No activity since a long time label Aug 28, 2026
@abstrask

Copy link
Copy Markdown
Author

Hello?

@github-actions github-actions Bot removed the stale No activity since a long time label Aug 29, 2026
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.

Kafka topic ACLs fail to load: PCI Databases app calls removed /acl endpoint

1 participant