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

REQUEST: Creation of KEP 14 repos #56

Closed
sttts opened this issue Sep 4, 2018 · 23 comments
Closed

REQUEST: Creation of KEP 14 repos #56

sttts opened this issue Sep 4, 2018 · 23 comments
Assignees
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository

Comments

@sttts
Copy link

sttts commented Sep 4, 2018

To implement KEP 14, we need a couple of repos.

New Repo

  • cloud-controller-manager
  • kube-controller-manager
  • kube-scheduler
  • kubelet
  • kube-proxy

Which Organization should it reside

kubernetes

Note that these are "staging repositories", i.e. they are published by the https://github.com/kubernetes/publishing-bot and therefore must be in the kubernetes org.

Who should be provided access to this repository

We need admin access for the stage-bots team, compare:

bildschirmfoto 2018-09-03 um 13 54 46

We need branch settings like:

bildschirmfoto 2018-09-03 um 13 55 26

Approvals

KEP 14: https://github.com/kubernetes/community/blob/master/keps/sig-cluster-lifecycle/0014-20180707-componentconfig-api-types-to-staging.md

Additional context for request

The staging repositories have been created in the staging/ directory of the k/k repo, all approved by top-level approvers @jbeda and @thockin.

/cc @luxas @stewart-yu

@sttts sttts changed the title Creation of KEP 14 repos REQUEST: Creation of KEP 14 repos Sep 4, 2018
@cblecker
Copy link
Member

cblecker commented Sep 4, 2018

@sttts Has sig-arch approved of new core repos? Was there a meeting agenda item or a mailing list thread where they do?

cc: @jdumars @bgrant0607

@sttts
Copy link
Author

sttts commented Sep 5, 2018

@cblecker the KEP was approved which proposed these repos.

@cblecker
Copy link
Member

cblecker commented Sep 5, 2018

Looks like this is slated for discussion in sig-arch on Thursday
kubernetes/community#2627

@sttts
Copy link
Author

sttts commented Sep 6, 2018

@spiffxp
Copy link
Member

spiffxp commented Sep 7, 2018

/assign
I believe this is done but I'll leave this open to confirm since this is staging

@spiffxp
Copy link
Member

spiffxp commented Sep 7, 2018

It's also unclear to me whether/where these repos should fall in sigs.yaml?

@stewart-yu
Copy link

stewart-yu commented Sep 7, 2018

thanks, but seems no need to create repo for cloud-controller-manager now. @spiffxp 😀
/cc @sttts

dims added a commit to dims/publishing-bot that referenced this issue Sep 7, 2018
New repos requested in:
kubernetes/org#58
kubernetes/org#56

Change-Id: I2e71a998db2e53b8d8827a59a7b35198f52e3e13
@GeorgeGuo2018
Copy link

https://github.com/kubernetes/kube-controller-manager
this repo is empty now, what should i do if my older projects referred package from this repo?
Any reply would be appreciated,thx.

@sttts
Copy link
Author

sttts commented Sep 7, 2018

thanks, but seems no need to create repo for cloud-controller-manager now.

@stewart-yu thanks for the correction. Yes, we keep those types in the cmd package because the future of the binary is not clear.

@sttts
Copy link
Author

sttts commented Sep 7, 2018

this repo is empty now, what should i do if my older projects referred package from this repo?
Any reply would be appreciated,thx.

@GeorgeGuo2018 I will update github.com/kubernetes/publishing-bot and it will fill that repo.

Which "older projects" do you refer to?

@sttts
Copy link
Author

sttts commented Sep 7, 2018

@spiffxp we have the OWNERS files in there. Isn't that enough? Listing those repos in sigs.yaml in addition sounds equivalent.

@sttts
Copy link
Author

sttts commented Sep 7, 2018

@spiffxp as @stewart-yu correctly noted, cloud-controller-manager is not used. You can delete it again.

sttts pushed a commit to sttts/publishing-bot that referenced this issue Sep 7, 2018
New repos requested in:
kubernetes/org#58
kubernetes/org#56

Change-Id: I2e71a998db2e53b8d8827a59a7b35198f52e3e13
dims added a commit to dims/publishing-bot that referenced this issue Sep 7, 2018
New repos requested in:
kubernetes/org#58
kubernetes/org#56

Change-Id: I2e71a998db2e53b8d8827a59a7b35198f52e3e13
@spiffxp
Copy link
Member

spiffxp commented Sep 7, 2018

@sttts every other repo that comes out of staging is listed in sigs.yaml as belonging to a sig, egs:

sig-api-machinery has this entry

- name: universal-machinery # i.e., both client and server
      owners:
      - https://raw.githubusercontent.com/kubernetes/apimachinery/master/OWNERS
      - https://raw.githubusercontent.com/kubernetes/kubernetes/master/staging/src/k8s.io/apimachinery/OWNERS

sig-architecture has this entry

- name: api
      owners:
      - https://raw.githubusercontent.com/kubernetes/api/master/OWNERS
      - https://raw.githubusercontent.com/kubernetes/kubernetes/master/staging/src/k8s.io/api/OWNERS

I don't know if the intent is to move to a world where each component is owned by a different sig, eg:

  • sig-node owns kubelet
  • sig-network owns kube-proxy
  • sig-apps owns kube-controller-manager
  • sig-api-machinery owns kube-apiserver
  • sig-scheduling owns kube-scheduler

or, if sig-archtecture owns the root of each component, with individual sigs owning packages within (eg: sig-architecture owns api, but sig-apps owns the workload apis)

@sttts
Copy link
Author

sttts commented Sep 7, 2018

@spiffxp I wasn't aware that we link OWNERS files there. So of course the new repos belong into that file 👍

dims added a commit to dims/publishing-bot that referenced this issue Sep 7, 2018
New repos requested in:
kubernetes/org#58
kubernetes/org#56

Change-Id: I2e71a998db2e53b8d8827a59a7b35198f52e3e13
sttts pushed a commit to sttts/publishing-bot that referenced this issue Sep 7, 2018
New repos requested in:
kubernetes/org#58
kubernetes/org#56

Change-Id: I2e71a998db2e53b8d8827a59a7b35198f52e3e13
@cblecker cblecker removed their assignment Sep 10, 2018
@spiffxp
Copy link
Member

spiffxp commented Sep 10, 2018

Current status:

  • I removed cloud-controller-manager
  • The other repos look to be synced by publishing-bot now
  • I have yet to confirm that the repos have been added to sigs.yaml

@spiffxp
Copy link
Member

spiffxp commented Sep 10, 2018

I chatted with @sttts offline, the repo descriptions have been changed

Just waiting for a sigs.yaml PR to close this out

@luxas
Copy link
Member

luxas commented Sep 11, 2018

I don't know either whether we want the specialized SIG (e.g. node) to own all of the k8s.io/kubelet repo or if we want it to own e.g. pkg/ and cmd/ only?
That really dictates the structure of the sigs.yaml PR. Any preferences?

@spiffxp
Copy link
Member

spiffxp commented Sep 11, 2018

Having sig-arch own root and specialized sigs own different packages was one of the suggestions I made above. I have no strong preference other than an option get chosen. I will ask the sig-arch mailing list

@spiffxp
Copy link
Member

spiffxp commented Sep 11, 2018

@spiffxp
Copy link
Member

spiffxp commented Sep 17, 2018

So far I have received two responses, both of which seem to suggest the sig-arch root / sig-foo subdirs approach.

@luxas
Copy link
Member

luxas commented Nov 30, 2018

The final piece of this was fixed with kubernetes/kubernetes#70453, closing
/close

@k8s-ci-robot
Copy link
Contributor

@luxas: Closing this issue.

In response to this:

The final piece of this was fixed with kubernetes/kubernetes#70453, closing
/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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository
Projects
None yet
Development

No branches or pull requests

7 participants