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

CoreDNS operator:- Addition of list of features #40

Closed
4 tasks
Abhik1998 opened this issue Feb 5, 2020 · 11 comments
Closed
4 tasks

CoreDNS operator:- Addition of list of features #40

Abhik1998 opened this issue Feb 5, 2020 · 11 comments

Comments

@Abhik1998
Copy link

Abhik1998 commented Feb 5, 2020

Description:-
(GSOC proposal 2020)
We have some new features to be added this time to the CoreDNS operator.The current operator is a proof of concept of the add-on library.

Goal:-
Define the set of milestones and criteria for productionizing the CoreDNS operator. The current operator is a proof of concept of the add-on library. The goal of the alpha operator is to provide a minimal feature set that is manageable, predictable, and reliable in a production environment.

  • Implement CoreDNS - operator smoke test.
    Implementing and validating various scenarios like creation, deletion, configuration change, etc.
    This work will require adding multiple versions to the CoreDNS version directory. Issue
  • Provide metrics in the kubebuilder-declarative-pattern library
    A user should be able to emit metrics on the Prometheus metrics endpoint. Metrics should be written to the controller-runtime Prometheus Registry. Issue
  • Integrate Kustomize with kubebuilder-declarative-pattern
    Finish and share design doc. Issue
  • Implement rolling updates for ConfigMap
    Rolling update for Configs should use hashed names (kustomize supports this) Issue

Proposed mentors: Jeff Johnson [ @johnsonj ]

@Abhik1998 Abhik1998 changed the title CoreDNS:- Addition of list of features CoreDNS operator:- Addition of list of features Feb 5, 2020
@dholbach
Copy link
Member

CCing @rajansandeep

@dholbach
Copy link
Member

@rajansandeep and @johnsonj

Looks like @Abhik1998 got the proposal added to /cncf/soc as a project - do you feel the project description above makes sense? Does this capture the ideal outcome for the operator? I feel like some coordination and planning with the coredns team about this operator should be part of this project too.

@Abhik1998
Copy link
Author

Abhik1998 commented Feb 27, 2020

@dholbach I hope this description helps. I have updated that.

@johnsonj
Copy link
Contributor

johnsonj commented Feb 27, 2020

Totally agree @dholbach - It'd be great to get their input and buy in. Our ultimate goal is to have an 'alpha quality' operator for managing CoreDNS in the cluster and they're the experts on what that really means.

@rajansandeep Can you check out this doc for more context and let me know what you think?

@rajansandeep
Copy link
Contributor

@rajansandeep Can you check out this doc for more context and let me know what you think?

@johnsonj The goals set in the doc for an 'alpha quality' operator seems reasonable to me.
The only addition which comes to mind right now is whether we would want to add the CoreDNS corefile migration support while upgrading/downgrading the CoreDNS version. (This is already been done for kubeadm)

I'd like to kick off progress for the CoreDNS operator by updating it to v2 (I was to do it some time ago, but was absent due to a long vacation), which will also cover the remaining work in #27 (comment)

@Raman-Kumar
Copy link

@rajansandeep
Can you suggest me some docs or resources to properly understand the codebase of cluster-addons and contributing to it?

@johnsonj
Copy link
Contributor

johnsonj commented Mar 3, 2020

The README in this project (cluster-addons) should cover a bit of the high-level ideas. The README for the CoreDNS operator primarily covers 'how we got to this point'.

For more information on the frameworks the operator builds on:

If the timing works for you, we have a bi-weekly meeting (see 'Cluster Addons')

@johnsonj
Copy link
Contributor

@Abhik1998 thank you for kicking off the organization of this! I duplicated this issue as #47 so that I could have edit access. I'm going to dupe this bug to that to keep things clear.

@johnsonj
Copy link
Contributor

/dupe #47

@johnsonj
Copy link
Contributor

/close

@k8s-ci-robot
Copy link
Contributor

@johnsonj: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@johnsonj johnsonj mentioned this issue Jun 10, 2020
4 tasks
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

No branches or pull requests

6 participants