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

Please create kubernetes/klog (for forking glog) #195

Closed
dims opened this issue Oct 26, 2018 · 25 comments · Fixed by kubernetes/community#3033
Closed

Please create kubernetes/klog (for forking glog) #195

dims opened this issue Oct 26, 2018 · 25 comments · Fixed by kubernetes/community#3033
Assignees
Labels
area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository kind/feature Categorizes issue or PR as related to a new feature. sig/architecture Categorizes an issue or PR as relevant to SIG Architecture. sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation.

Comments

@dims
Copy link
Member

dims commented Oct 26, 2018

New Repo, Staging Repo, or migrate existing

Import https://github.com/dims/klog into kubernetes org

Requested name for new repository

klog

Which Organization should it reside

kubernetes

If not a staging repo, who should have admin access

@dims @thockin

If not a staging repo, who should have write access

@tallclair @justinsb, @erictune @lavalamp

If a new repo, who should be listed as approvers in OWNERS

@tallclair @justinsb, @erictune @lavalamp @dims @thockin

If a new repo, who should be listed in SECURITY_CONTACTS

@dims @thockin

What should the repo description be

Leveled execution logs for Go (fork of https://github.com/golang/glog)

What SIG and subproject does this fall under in sigs.yaml

sig-architecture

Approvals

Please prove you have followed the appropriate approval process for this new
repo by including links to the relevant approvals (meeting minutes, e-mail
thread, etc.)
Discussion in sig-arch on Oct 25, 2018 ( http://bit.ly/sig-architecture )

Authoritative requirements are here: https://git.k8s.io/community/github-management/kubernetes-repositories.md

tl;dr (but really you should read the linked doc, this may be stale)

  • If this is a core repository, then sig-architecture must approve
  • If this is a SIG repository, then this must follow the procedure spelled out
    in that SIG's charter

Additional context for request

Any additional information or context to describe the request.

/area github-repo

@k8s-ci-robot k8s-ci-robot added the area/github-repo Creating, migrating or deleting a Kubernetes GitHub Repository label Oct 26, 2018
@dims
Copy link
Member Author

dims commented Oct 26, 2018

Also started an email thread in sig-arch:
https://groups.google.com/d/msg/kubernetes-sig-architecture/wCWiWf3Juzs/hXRVBH90CgAJ

@erictune
Copy link
Member

/lgtm

@tallclair
Copy link
Member

Are we still intending for this to be a transitional step towards our logging end goal? If so, can we call that out in the description & readme, and discourage others from using this? My concern is that as soon as we put out a "kubernetes-log" repo, other projects will start cargo-culting it...
Along similar lines, I wonder if we should stick with "glog" for the name? It further emphasizes that this is a fork of glog, and not a new kubernetes-endorced log library. It also means fewer lines need to be touched in the migration.

@cblecker
Copy link
Member

hey @dims, can you ping me back on this when consensus at sig-arch has been achieved? Then I can coordinate with you on a repo transfer.

@dims
Copy link
Member Author

dims commented Oct 26, 2018

Will do. thanks @cblecker.

I am going to assume @thockin and @tallclair are approvers on the plan from @kubernetes/sig-architecture-feature-requests

@k8s-ci-robot k8s-ci-robot added sig/architecture Categorizes an issue or PR as relevant to SIG Architecture. kind/feature Categorizes issue or PR as related to a new feature. labels Oct 26, 2018
@dims
Copy link
Member Author

dims commented Oct 31, 2018

/sig instrumentation

@k8s-ci-robot k8s-ci-robot added the sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation. label Oct 31, 2018
@dims
Copy link
Member Author

dims commented Oct 31, 2018

@kubernetes/sig-architecture-feature-requests @thockin @bgrant0607 @tallclair @justinsb sig-arch folks, please +1 this request

@jdumars
Copy link
Member

jdumars commented Oct 31, 2018

/lgtm

2 similar comments
@lavalamp
Copy link
Member

/lgtm

@erictune
Copy link
Member

/lgtm

@BenTheElder
Copy link
Member

Very interested in seeing this happen -- Minor note about glog vs klog, klog sounds a lot like clog which is perhaps not the greatest association for logging? :^)
If we are just doing a glog fork, k8s.io/glog might be good enough and easier to swap with dep etc.

@thockin
Copy link
Member

thockin commented Oct 31, 2018

@dims thoughts on @BenTheElder's comment?

@thockin
Copy link
Member

thockin commented Oct 31, 2018

/lgtm
/approve

@dims
Copy link
Member Author

dims commented Oct 31, 2018

@BenTheElder @thockin klog gives us some wiggle room which can be valuable. we can still use dep + wrapper over klog ( like https://github.com/dims/glog-minimal ) when needed. So let's go with klog for now please.

@dims
Copy link
Member Author

dims commented Nov 1, 2018

/assign @cblecker

@cblecker
Copy link
Member

cblecker commented Nov 1, 2018

@dims So this isn't going to be a staging repo, right? Just moving over your existing fork? If so, can you please add me as an admin of the existing repo.

@dims
Copy link
Member Author

dims commented Nov 1, 2018

Ack @cblecker doing that now

@justinsb
Copy link
Member

justinsb commented Nov 1, 2018

/lgtm

1 similar comment
@jbeda
Copy link

jbeda commented Nov 1, 2018

/lgtm

@tallclair
Copy link
Member

/lgtm
I like the idea of putting the glog-compatible interface under k8s.io/klog/glog, so that 1) only the import lines need to touched, and 2) it is easy to tell which files have been migrated to the new logging interface (once we get to that).

@dims
Copy link
Member Author

dims commented Nov 1, 2018

@tallclair affirmative! will do

@cblecker
Copy link
Member

cblecker commented Nov 1, 2018

This is moved into https://github.com/kubernetes/klog . Haven't fully finished all the automation pieces, but it's in progress :)

@neolit123
Copy link
Member

neolit123 commented Nov 2, 2018

hm, so we forked glog in https://github.com/kubernetes/klog, but forked yaml in https://github.com/kubernetes-sigs/yaml. 🤔

@dims @cblecker what was the reason for the separate orgs for these brand new forks?

@dims
Copy link
Member Author

dims commented Nov 2, 2018

@neolit123 that was my call. klog's initial sponsor was sig-arch and the code was coming from a trusted org so i felt it was ok to bring it directly into the main kubernetes org. the yaml one was coming from a personal github repo (ghodss) and it was coming in under sig-api-machinery, so it seemed better to leave it under kubernetes-sigs.

@cblecker
Copy link
Member

@dims Can you add this to sigs.yaml? Then we can close this out.

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 kind/feature Categorizes issue or PR as related to a new feature. sig/architecture Categorizes an issue or PR as relevant to SIG Architecture. sig/instrumentation Categorizes an issue or PR as relevant to SIG Instrumentation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.