-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Determine canonical list of (run-time) dependencies for Kubernetes #2234
Comments
/sig architecture |
/milestone v1.12 |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
/remove-lifecycle stale |
/sig cluster-lifecycle Also ref: kubernetes/sig-release#400 for the release team to at least have some kind of documented process to follow in its checklist |
@marpaia fyi |
kubeadm is a consumer only of 2-3 items in the external deps list:
the main consumer of these external deps is the cluster folder, which has to be booted outside of k/k eventually. |
It looks like that dependencies list includes things in the cluster/addons directory, but is missing some obvious things, like etcd. |
I suppose this issue is about run-time dependencies. There is also the question of build-time dependencies. Perhaps the title of this issue should be more explicit. |
I opened #3749 for the build-time dependencies. |
Spec'd this as run-time dependencies. #3749 is the issue for build-time deps. |
Is this still needed? If so, wouldn't it be an item for k/website if its run-time dependendencies for users? |
/unassign @jdumars |
We need a canonical list of official dependencies for Kubernetes, something we don't currently have. This should appear both in the docs and in the release notes.
For example, this section of the release notes is currently a grab-bag:
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.10.md#external-dependencies
Some of those things should be listed, some shouldn't, and there's certainly stuff that ought to be listed there that isn't.
@jdumars @bgrant0607 @nickchase @zacharysarah
The text was updated successfully, but these errors were encountered: