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 permissions issue on ServiceDetails under RBAC #3433

Merged
merged 1 commit into from
Jul 20, 2021
Merged

Conversation

Nokel81
Copy link
Collaborator

@Nokel81 Nokel81 commented Jul 19, 2021

  • Only watch the namespace that the service is in instead of all of them

Signed-off-by: Sebastian Malton sebastian@malton.name

fixes #3402

- Only watch the namespace that the service is in instead of all of them

Signed-off-by: Sebastian Malton <sebastian@malton.name>
@Nokel81 Nokel81 added the bug Something isn't working label Jul 19, 2021
@Nokel81 Nokel81 added this to the 5.1.3 milestone Jul 19, 2021
@Nokel81 Nokel81 self-assigned this Jul 19, 2021
@Nokel81 Nokel81 requested a review from a team as a code owner July 19, 2021 19:32
@Nokel81 Nokel81 requested review from jakolehm and Nachasic and removed request for a team July 19, 2021 19:32
@Nokel81
Copy link
Collaborator Author

Nokel81 commented Jul 19, 2021

@devodev does this fix your issue? I think it should.

@devodev
Copy link
Contributor

devodev commented Jul 19, 2021

@devodev does this fix your issue? I think it should.

At first glance, it should yes.

@jakolehm jakolehm merged commit c590bcc into master Jul 20, 2021
@jakolehm jakolehm deleted the issue-3402 branch July 20, 2021 04:34
@Nokel81 Nokel81 mentioned this pull request Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Service details pane tries loading endpoints from default namespace
3 participants