Skip to content

Add PrimaryServiceTemplate and ReplicaServiceTemplate#315

Closed
arosh wants to merge 1 commit intocybozu-go:mainfrom
arosh:primary-or-replica-service-template
Closed

Add PrimaryServiceTemplate and ReplicaServiceTemplate#315
arosh wants to merge 1 commit intocybozu-go:mainfrom
arosh:primary-or-replica-service-template

Conversation

@arosh
Copy link
Copy Markdown
Member

@arosh arosh commented Oct 6, 2021

Motivation

In our Kubernetes cluster, we can automatically set desired external DNS name to the Service by annotating external-dns.alpha.kubernetes.io/hostname.

We want to set different annotations to the Service for primary instance and the Service for replica instances.

How

As a patch for serviceTemplate, primaryServiceTemplate and replicaServiceTemplate can be specified in the manifest. They will be merged into the serviceTemplate applied to the corresponding service and applied in the reconciliation loop.

Matter of concern

The merging of serviceTemplate and primaryServiceTemplate / replicaServiceTemplate is implemented by DeepCopyInto. This behavior may confuse kustomize users.

@arosh arosh force-pushed the primary-or-replica-service-template branch 13 times, most recently from fd6c0e7 to e1991a1 Compare October 7, 2021 07:02
@arosh arosh force-pushed the primary-or-replica-service-template branch from e1991a1 to 1c0c635 Compare October 7, 2021 07:04
@ymmt2005
Copy link
Copy Markdown
Member

ymmt2005 commented Dec 8, 2021

Let me close this.

@ymmt2005 ymmt2005 closed this Dec 8, 2021
@arosh
Copy link
Copy Markdown
Member Author

arosh commented Jan 28, 2022

Resolved by #350

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.

2 participants