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

MON-3346: add kube-rbac-proxy #59

Merged
merged 1 commit into from
Apr 29, 2024
Merged

MON-3346: add kube-rbac-proxy #59

merged 1 commit into from
Apr 29, 2024

Conversation

rexagod
Copy link
Member

@rexagod rexagod commented Sep 5, 2023

adds info about kube-rbac-proxy's high-level workflow

@rexagod rexagod added the documentation Improvements or additions to documentation label Sep 5, 2023
Copy link
Contributor

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

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

Regarding examples, I'd take a very concrete approach and detail how monitoring components leverage kube-rbac-proxy:

  • /metric authn & authz
  • API frontend

content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
@rexagod
Copy link
Member Author

rexagod commented Sep 5, 2023

I pushed some of my own changes without noticing the review, I'll incorporate the suggested changes.

@rexagod
Copy link
Member Author

rexagod commented Sep 6, 2023

@simonpasquier WRT #59 (review), I think the first point is covered now, however, I'm not sure what the second one refers to. Is it the KRP configuration as the (KRP) API frontend that we want to document further (I've added a couple of resource and non-resource examples based on the config present in CMO)? PLMK if I'm missing anything here. :)

@rexagod rexagod marked this pull request as ready for review September 6, 2023 14:06
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
@rexagod rexagod mentioned this pull request Jan 16, 2024
@rexagod rexagod force-pushed the add-krp branch 3 times, most recently from 12d7e44 to 775428c Compare January 25, 2024 00:28
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
content/Projects/Observability/kube-rbac-proxy.md Outdated Show resolved Hide resolved
@rexagod
Copy link
Member Author

rexagod commented Mar 11, 2024

@simonpasquier Addressed the review comments, I believe this is good for another round of review (or merging).

adds info about kube-rbac-proxy's high-level workflow
@simonpasquier simonpasquier merged commit b4ea79b into rhobs:main Apr 29, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants