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

Added note re ip_forward and updated dashboard url #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chrisbu
Copy link

@chrisbu chrisbu commented Apr 1, 2020

Description

For my install (raspbian buster) I had to also set net.ipv4.ip_forward=1 to get networking working between nodes.
Also updated the alternative dashboard url to the current one.

Motivation and Context

Fixed problems accessing dashboard when it was deployed anywhere other than the master node.

  • I have raised an issue to propose this change (required)

How Has This Been Tested?

Tested using raspbian buster fresh install and a 3 node cluster. Not tested on earlier versions of raspbian.

Types of changes

  • [ x] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [ x] My code follows the code style of this project.
  • [ x] I have tested this change and prove it works
  • [ x] I've read the CONTRIBUTION guide
  • [ x] I have signed-off my commits with git commit -s

Signed-off-by: Chris Buckett <chrisbuckett@gmail.com>
Signed-off-by: Chris Buckett <chrisbuckett@gmail.com>
@derek derek bot added the new-contributor label Apr 1, 2020
@@ -357,7 +367,7 @@ subjects:
This is the development/alternative dashboard which has TLS disabled and is easier to use.

```
$ kubectl apply -f https://raw.githubusercontent.com/kubernetes/dashboard/master/aio/deploy/alternative/kubernetes-dashboard-arm-head.yaml
$ kubectl apply -f kubectl apply -f https://raw.githubusercontent.com/kubernetes/dashboard/master/aio/deploy/alternative.yaml

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo.. duplicate kubectl

@davelosert
Copy link

@chrisbu : Just stumbled accross this PR when I was trying to install the dashboard.
You might also want to adjust the ClusterRoleBinding above to:

apiVersion: rbac.authorization.k8s.io/v1beta1
kind: ClusterRoleBinding
metadata:
  name: kubernetes-dashboard-admin
  labels:
    k8s-app: kubernetes-dashboard
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: cluster-admin
subjects:
- kind: ServiceAccount
  name: kubernetes-dashboard
  namespace: kubernetes-dashboard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants