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.

Sign up for GitHub

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

Jump to bottom

Add ingress test for opaAuthorizeRequest filter #8575

Open
wants to merge 21 commits into
base: dev
Choose a base branch
from
Open

Add ingress test for opaAuthorizeRequest filter #8575

wants to merge 21 commits into from
+160 −5

Conversation

wisinghe
Copy link
Collaborator

@wisinghe wisinghe commented Dec 5, 2024 •
edited
Loading

Changes

  • Skipper rbac configuration is ordered to be applied before deployment to make sure the service account is created before the deployment.
  • Cluster configuration in teapot-e2e is changed to enable opa filters for the skipper ingress
  • opaAuthorizeRequest is initialized and verified in tests

@wisinghe wisinghe added minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call. do-not-merge labels Dec 5, 2024
@wisinghe wisinghe added e2e and removed do-not-merge labels Dec 6, 2024
@MustafaSaber MustafaSaber removed the request for review from RomanZavodskikh December 18, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikkeloscar mikkeloscar Awaiting requested review from mikkeloscar

@mjungsbluth mjungsbluth Awaiting requested review from mjungsbluth

@AlexanderYastrebov AlexanderYastrebov Awaiting requested review from AlexanderYastrebov

@MustafaSaber MustafaSaber Awaiting requested review from MustafaSaber

@demonCoder95 demonCoder95 Awaiting requested review from demonCoder95

Assignees
No one assigned
Labels
e2e minor Minor changes, e.g. low risk config updates, changes that do not introduce a new API call.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wisinghe @mikkeloscar @mjungsbluth