-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Add CRD - DNSEnpoint to the helm chart template #2640
Comments
…to helm deliverables Since DNSEndpoint can be used as one of the sources for external-dns, it would be handy to be able to deploy its definition file together with the main deployment. More info: - piggy-backing on the make crd to create such template file under charts/external-dns/templates/ - by default it's the same as before and the CRD is not deployed using helm chart - to enable the CRD deployment, one has to set helm value deployCrd=true Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
…to helm deliverables Since DNSEndpoint can be used as one of the sources for external-dns, it would be handy to be able to deploy its definition file together with the main deployment. More info: - piggy-backing on the make crd to create such template file under charts/external-dns/templates/ - by default it's the same as before and the CRD is not deployed using helm chart - to enable the CRD deployment, one has to set helm value deployCrd=true Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
I think there have been a number of discussions in this repo regarding the stability of the CRD in the repo and the suitability of making it an official component of the Helm chart. I'd like an explicit approval that it can be included by a maintainer before approving adding it to the chart. The chart does already currently support the CRD workflow once the CRD is installed. Remember that Helm shouldn't be used to install CRDs as it can't handle the lifecycle and the only reason to add the CRD is to allow easy prototyping and as a lookup for the CRD. |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
/remove-lifecycle rotten |
I think it would be really helpful to have the CRD in the helm chart. Currently, I use the bitnami helm chart (https://github.com/bitnami/charts/tree/master/bitnami/external-dns) just for that reason. However, I would really prefer to use the official chart. |
This is more of a problem with pre-release CRDs as if the CRD doesn't change not being able to update the CRD is a non-issue. AFAIK the CRD in question is pre-release and hasn't progressed recently, this might want addressing before we add the CRD to the chart and people start using Helm to install it. |
There are two possibilities for the CRD:
Nevertheless, both approaches are much more convenient than cloning the repo, generating the CRD and applying it by hand. |
@snorwin option 2 is off the table as an anti-pattern, this is how Helm v2 hacked CRDs before the addition of support for the FYI the CRD is available at https://raw.githubusercontent.com/kubernetes-sigs/external-dns/master/docs/contributing/crd-source/crd-manifest.yaml so there is no need to clone and build the CRD. |
@stevehipwell I totally agree, I also prefer option 1. What is still missing for the CRD to become GA? Is there an issue tracking the missing features (i.e. error message in the status, support for TXT records, ...)? I would be happy to contribute. |
@snorwin I've not had much luck on the topic of the CRD when I've brought it up, I think this is because it's a contrib component and the maintainers here are snowed under. I'd suggest a new issue about making the CRD GA would be the best place to start, you might also want to look in on #2529 and link the two together. |
@stevehipwell I opened another issue to make the CRD GA: #2941 |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues. This bot triages un-triaged issues according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
* Add `dnsendpoint` CRD to Helm chart Fixes: #2640 * Update charts/external-dns/crds/dnsendpoint.yaml Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com> * Update charts/external-dns/crds/dnsendpoint.yaml Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com> * Update charts/external-dns/crds/dnsendpoint.yaml Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com> * Update CHANGELOG.md * add required annotation back to fix helm lint --------- Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
What would you like to be added:
CRD DNSEnpoint include in the helm chart template
Probably this was wrongly closed
https://github.com/kubernetes-sigs/external-dns/blob/master/docs/contributing/crd-source.md
Why is this needed:
Helps to ensure no separate CRD deployment is needed when using the helm
The text was updated successfully, but these errors were encountered: