Skip to content
This repository was archived by the owner on Sep 30, 2020. It is now read-only.

[v0.14.x] Allow injection of CoreDNS configuration for non-root zones #1875

Merged

Conversation

kfr2
Copy link
Contributor

@kfr2 kfr2 commented Jun 29, 2020

The existing kubeDns.autoscaler.extraCoreDNSConfig allows the user to
inject additional values into the coredns configmap's root zone (".").
This commit allows the user to additionally specify a string to
kubeDns.autoscaler.additionalZoneCoreDNSConfig which will be injected
into the configmap after the root zone to allow the user to specify
configuration for additional zones. As an example, this might be used
to forward traffic for the .global zone to the istiocoredns service.

The existing `kubeDns.autoscaler.extraCoreDNSConfig`  allows the user to
inject additional values into the coredns configmap's root zone (".").
This commit allows the user to additionally specify a string to
`kubeDns.autoscaler.additionalZoneCoreDNSConfig` which will be injected
into the configmap after the root zone to allow the user to specify
configuration for additional zones. As an example, this might be used
to forward traffic for the .global zone to the istiocoredns service.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 29, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign mumoshu
You can assign the PR to them by writing /assign @mumoshu in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 29, 2020
@dominicgunn
Copy link
Contributor

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 30, 2020
@dominicgunn dominicgunn merged commit cd68fa8 into kubernetes-retired:v0.14.x Jun 30, 2020
@dominicgunn dominicgunn changed the title Allow injection of CoreDNS configuration for non-root zones [v0.14.x] Allow injection of CoreDNS configuration for non-root zones Jul 3, 2020
@dominicgunn dominicgunn added this to the v0.14.6 milestone Jul 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants