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

KIALI-2505 Fix error when fetching Grafana info with oauth #902

Merged
merged 1 commit into from Mar 15, 2019

Conversation

mwringe
Copy link
Contributor

@mwringe mwringe commented Mar 7, 2019

** Describe the change **

When fetching the Grafana service information, use the Kiali service account token instead of the user token.

This prevents issues when using OpenShift Oauth and the user doesn't have access to the Grafana service.

** Issue reference **

https://issues.jboss.org/browse/KIALI-2505

@rhqci
Copy link

rhqci commented Mar 7, 2019

Jenkins CI: kiali-core-pr-e2e-test #664

  • ✔️ run-kiali-e2e-tests #[1397]

Copy link
Contributor

@jotak jotak left a comment

Choose a reason for hiding this comment

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

LGTM

@mwringe
Copy link
Contributor Author

mwringe commented Mar 8, 2019

@gbaufake @mattmahoneyrh Can one of you review this pr?

@gbaufake gbaufake assigned gbaufake and unassigned gbaufake Mar 15, 2019
@gbaufake gbaufake self-requested a review March 15, 2019 11:49
@lucasponce lucasponce merged commit 23e4206 into kiali:master Mar 15, 2019
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

5 participants