wso2 / kubernetes-apim

Kubernetes and Helm resources for WSO2 API Manager
Apache License 2.0
112 stars 216 forks source link

KeyManager service URL should be the k8s service name instead of localhost #543

Open renuka-fernando opened 2 years ago

renuka-fernando commented 2 years ago

Description: Need $subjuect when configuring Choreo Connect as the gateway with APIM as the control plane.

In deployment toml file of Single Node and Pattern1, service_url of [apim.key_manager] is commented (using default).

    #[apim.key_manager]
    #service_url = "https://localhost:${mgt.transport.https.port}/services/"

Should be templated, for example in single node

  [apim.key_manager]
  service_url = "https://{{ template "am-single-node.resource.prefix" . }}-am-service.{{ .Release.Namespace }}:${mgt.transport.https.port}/services/"

Suggested Labels:

Suggested Assignees:

Affected Product Version:

OS, DB, other environment details and versions:

Steps to reproduce:

Related Issues:

renuka-fernando commented 2 years ago

As a resolution in Choreo Connect QSG, update the whole deployment toml from a file kept in https://github.com/wso2/kubernetes-microgateway/blob/22167143aaab685a1a9efdd89d8d5782244068ea/resources/controlplane-deployment.toml