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

Add support for Azure Workload Identity #975

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EppO
Copy link
Contributor

@EppO EppO commented Jul 5, 2023

Add annotations on Kubernetes Runner service and deployment to support Workload Identity

  • Add documentation how to setup with Azure Managed Prometheus

@EppO
Copy link
Contributor Author

EppO commented Jul 5, 2023

failed test is about slack, not related to my changes

@RoiGlinik
Copy link
Contributor

Thanks @EppO , will test that asap.

@RoiGlinik RoiGlinik self-assigned this Jul 6, 2023
@EppO EppO force-pushed the azure-workload-identity branch 3 times, most recently from a30f2f1 to 8a2d61e Compare July 11, 2023 22:41
@EppO EppO force-pushed the azure-workload-identity branch 2 times, most recently from 0e15195 to 085a820 Compare August 1, 2023 02:04
@pavangudiwada pavangudiwada added the needs-review A community contribution for Robusta team to review label Nov 16, 2023
@pavangudiwada
Copy link
Contributor

Hi @EppO, I'm testing this PR and I have some issues with authentication. Is this PR still relevant? And how is it related to robusta-dev/prometrix#12?

@EppO
Copy link
Contributor Author

EppO commented Jan 30, 2024

Hi @EppO, I'm testing this PR and I have some issues with authentication. Is this PR still relevant? And how is it related to robusta-dev/prometrix#12?

robusta-dev/prometrix#12 implements the Azure Workload Id authentication along with the rest of managed prometheus service authentications. My plan is too rework this PR as soon as robusta-dev/prometrix#12 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-review A community contribution for Robusta team to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants