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

Issue with k8s.io/docs/getting-started-guides/fedora/fedora_manual_config/ #7417

Closed
dennis1987 opened this Issue Feb 15, 2018 · 1 comment

Comments

Projects
None yet
3 participants
@dennis1987
Copy link

dennis1987 commented Feb 15, 2018

This is a...

  • Feature Request
  • Bug Report

Problem:
In the node configuration the line
KUBELET_API_SERVER="--api-servers=http://fed-master:8080"
is removed in kublet v1.6, see here

Proposed Solution:
Add this line instead:
KUBELET_ARGS="--cgroup-driver=systemd --kubeconfig=/etc/kubernetes/master-kubeconfig.yaml --require-kubeconfig"

And add new config file /etc/kubernetes/master-kubeconfig.yaml with content:

kind: Config
clusters:
- name: local
  cluster:
    server: http://fed-master:8080
users:
- name: kubelet
contexts:
- context:
    cluster: local
    user: kubelet
  name: kubelet-context
current-context: kubelet-context

Page to Update:
https://kubernetes.io/docs/getting-started-guides/fedora/fedora_manual_config/

Kubernetes Version:
Kubernetes v1.9.1

Additional Information:
Fedora 27

@scriptonist

This comment has been minimized.

Copy link
Contributor

scriptonist commented Feb 20, 2018

I'm happy to work on this. 😍

scriptonist added a commit to scriptonist/website that referenced this issue Mar 4, 2018

k8s-ci-robot added a commit that referenced this issue Mar 5, 2018

remore added a commit to remore/website that referenced this issue May 13, 2018

Fix minor errors in fedora_manual_config.md
This fix is basically for a few errors caused by kubernetes#7417 . To complete kubernetes#7417, following errors should be fixed:
- duplicated KUBELET_ARGS arg declaration should be deleted
- what to do about yaml should be explained
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.