-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Support for permission segregation when using Azure AD Pod & Workload Identity. #3216
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a Go expert but think this looks OK.
I didn't see (m)any tests using separate identities though or did I look over them?
/run-e2e azure* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
Apart from @tomkerkhove comments, I left another inline
/run-e2e azure* |
b49d44c
to
e3ef2a8
Compare
/run-e2e azure* |
/run-e2e azure* Update: You can check the progress here |
/run-e2e workload* |
e3ef2a8
to
0c7025e
Compare
/run-e2e workload* |
0c7025e
to
897978f
Compare
/run-e2e user_assigned* |
/run-e2e user_assigned* |
/run-e2e user_assigned* |
/run-e2e azure* |
Final sanity check. |
/run-e2e azure* |
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
…Identity tokens. Signed-off-by: Vighnesh Shenoy <[email protected]>
…ity tokens. Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
Signed-off-by: Vighnesh Shenoy <[email protected]>
888b44b
to
d516b98
Compare
Fixed the conflicts in CHANGELOG. |
/run-e2e azure* |
Anything else to add here @JorTurFer ? Only thing changed since last e2e test run was fixing a merge conflict in the changelog. |
/run-e2e azure* |
Provide a description of what has been changed
Checklist
Relates to #2656
Relates to kedacore/keda-docs#790