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

fix #1788 lock dashboard version to 1.6.3 #1805

Merged
merged 1 commit into from
Oct 17, 2017
Merged

fix #1788 lock dashboard version to 1.6.3 #1805

merged 1 commit into from
Oct 17, 2017

Conversation

Atoms
Copy link
Member

@Atoms Atoms commented Oct 16, 2017

partially fix #1788 lock dashboard version to 1.6.3 while we get 1.7.x working

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 16, 2017
dashboard_image_repo: kubernetesdashboarddev/kubernetes-dashboard-amd64
dashboard_image_tag: head
dashboard_image_repo: gcr.io/google_containers/kubernetes-dashboard-amd64
dashboard_image_tag: 1.6.3
Copy link
Contributor

Choose a reason for hiding this comment

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

Please change to v1.6.3 (the whole tag)

@@ -62,7 +62,7 @@ spec:
spec:
containers:
- name: kubernetes-dashboard
image: {{ dashboard_image_repo }}:{{ dashboard_image_tag }}
image: {{ dashboard_image_repo }}:v{{ dashboard_image_tag }}
Copy link
Contributor

Choose a reason for hiding this comment

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

remove 'v'

@mattymo
Copy link
Contributor

mattymo commented Oct 16, 2017

ci check this

@mattymo
Copy link
Contributor

mattymo commented Oct 16, 2017

Tested locally and it works.

@mattymo mattymo merged commit 688e589 into kubernetes-sigs:master Oct 17, 2017
@shadycuz
Copy link

@atom
@mattymo

This doesn't fix auth correct? Only a change upstream to how kublet mounts kubernetes folder will correct?

@mattymo
Copy link
Contributor

mattymo commented Oct 17, 2017

@shadycuz I don't understand your question. Dashboard with basic auth works with this PR.

@shadycuz
Copy link

@mattymo When I asked you in slack you said what broke basic auth was how kubelet mounted the kubernetes folder? So looking at the PR I wasn't sure this actually fixed it and you closed my issues so I wanted to Double check =). My basic AUTH issue that you closed has nothing to do with dashboard and I tried a different 1.6.3 dashboard and it still didn't work. So I don't see how changing the dashboard fixes my auth problem?

@shadycuz
Copy link

@mattymo Sorry, misread my emails, my issue is still open for auth problems.

#1803

@mattymo
Copy link
Contributor

mattymo commented Oct 17, 2017

@shadycuz that's kubeadm mode. Dashboard auth was broken for both kubeadm and non-kubeadm mode.

@shadycuz
Copy link

shadycuz commented Oct 17, 2017

Yeah but my basic auth is just broken completely. If you look at my issue you will see using basic auth I'm not even able to list nodes through http post #1803 , I used all defaults when standing up cluster. So if I rerun my playbook basic auth will be fixed or should I stand up a new cluster? @mattymo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kubernetes dashboard 1.7.x (not working)
4 participants