Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[1.12] Update docs for Kubernetes nameresolver #3556

Closed
ItalyPaleAle opened this issue Jun 15, 2023 · 3 comments · Fixed by #3785
Closed

[1.12] Update docs for Kubernetes nameresolver #3556

ItalyPaleAle opened this issue Jun 15, 2023 · 3 comments · Fixed by #3785
Assignees
Labels
content/missing-information More information requested or needed P0
Milestone

Comments

@ItalyPaleAle
Copy link
Contributor

Docs for the Kubernetes nameresolver needs to be updated to add templating. See: dapr/components-contrib#2883

@ItalyPaleAle ItalyPaleAle added the content/missing-information More information requested or needed label Jun 15, 2023
@hhunter-ms hhunter-ms added this to the 1.12 milestone Jun 19, 2023
@hhunter-ms
Copy link
Collaborator

@ItalyPaleAle friendly bump

@hhunter-ms hhunter-ms added the P0 label Sep 25, 2023
@ItalyPaleAle ItalyPaleAle removed their assignment Sep 26, 2023
@hhunter-ms
Copy link
Collaborator

@MaxSchaefer could you add this to the docs for 1.12?

@mukundansundar mukundansundar self-assigned this Oct 3, 2023
@mukundansundar
Copy link
Contributor

mukundansundar commented Oct 5, 2023

@ItalyPaleAle Is this a correct configuration that can be set for k8s name resolver if cluster domain or template (mutually exclusive) parameters are to be set:

apiVersion: dapr.io/v1alpha1
kind: Configuration
metadata:
  name: appconfig
spec:
  nameResolution:
    component: "kubernetes"
    configuration:
      "clusterDomain": "testDomain"
      "template": "{{.ID}}-{{.Data.region}}.internal:{{.Port}}"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content/missing-information More information requested or needed P0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants