8000 Auto-detect Kubernetes API URL for web UI by ccremer · Pull Request #195 · ccremer/clustercode · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Auto-detect Kubernetes API URL for web UI #195

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 6, 2022
Merged

Auto-detect Kubernetes API URL for web UI #195

merged 1 commit into from
Oct 6, 2022

Conversation

ccremer
Copy link
Owner
@ccremer ccremer commented Oct 5, 2022 •
edited
Loading

Summary

  • Updates CLI flags so that the API URL for the Proxy can be detected by the service account token.
  • The token contains a bunch of metadata, such as aud (audience) field, which is set to the K8s URL.

Checklist

  • Categorize the PR by setting a good title and adding one of the labels:
    kind:bug, kind:enhancement, kind:documentation, kind:change, kind:breaking, kind:dependency
    as they show up in the changelog
  • PR contains the label area:operator
  • Link this PR to related issues
  • I have not made any changes in the charts/ directory.

@ccremer ccremer changed the title Auto-detect Kubernetes API URL Auto-detect Kubernetes API URL for web UI Oct 5, 2022
@ccremer ccremer mentioned this pull request Oct 5, 2022
Merged
7 tasks
@ccremer ccremer marked this pull request as ready for review October 5, 2022 23:35
@ccremer ccremer merged commit 930cee6 into master Oct 6, 2022
@ccremer ccremer deleted the auto-detect-k8s branch October 6, 2022 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area:operator kind:enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ccremer
0