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: Adds missing kustomize for k8s-dashboard #602

Merged
merged 2 commits into from
Jan 18, 2023

Conversation

axeII
Copy link
Contributor

@axeII axeII commented Jan 16, 2023

Hi Devin,

when I tried to deploy kubernetes-dashboard I was following your template. However the dashboard was not deployed - I found there is a missing Kustomization file.

This PR also adds step base64 -d to the comment for getting secret token for login to dashboard in the file rbac.yaml. Reason for that is that when I tried the original secret returned by k8s I wasn't able to login and had to decoded the secret as base64. After that I was able to login via the secret token.

Let me know what do you think about these changes.
Cheers.

also adds base64 decode to comment in rbac
@onedr0p
Copy link
Owner

onedr0p commented Jan 18, 2023

@axeII there's no need for a kustomization file there since the ks.yaml in being applied in the parent directory.

I'm not sure why that wouldn't deploy for you but I don't think that was the issue.

I've deleted that kustomization and kept the comment update. Thanks!

@onedr0p onedr0p merged commit c35de07 into onedr0p:main Jan 18, 2023
@axeII
Copy link
Contributor Author

axeII commented Jan 19, 2023

@onedr0p

Yeah you are right! I missed that one since in my repo I didn't apply the ks.yaml directly but the root directory of ks.yaml which is why kustomize didn't apply whole dashboard. Well TIL you can do that with kustomize.
Cheers.

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

Successfully merging this pull request may close these issues.

None yet

2 participants