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 test for gateway attached to AuthPolicy #295

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

averevki
Copy link
Contributor

@averevki averevki commented Nov 27, 2023

Test case

AuthPolicy attached straight to gateway

Changes

  • Change authorization targetRef parameter name from route to target
  • Add smoke test for AuthPolicy attached straight to gateway

Note

Also an automatization for the following issue: Kuadrant/kuadrant-operator#287

closes #269

@averevki averevki added the Test case New test case label Nov 27, 2023
@averevki averevki self-assigned this Nov 27, 2023
@averevki averevki marked this pull request as draft November 27, 2023 17:43
@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch from 48a22ff to fe86b94 Compare November 29, 2023 17:28
@averevki averevki marked this pull request as ready for review November 29, 2023 17:28
@averevki averevki requested a review from pehala November 30, 2023 12:36
@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch 2 times, most recently from 9fb2a8b to 40f1976 Compare December 4, 2023 12:39
@averevki
Copy link
Contributor Author

averevki commented Dec 4, 2023

^rebase

@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch from 40f1976 to 17db78d Compare December 5, 2023 16:03
@averevki averevki requested a review from pehala December 5, 2023 16:07
@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch from 17db78d to 97c8d2f Compare December 5, 2023 16:10
@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch from 97c8d2f to a00f1c7 Compare December 5, 2023 22:59
@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch 2 times, most recently from 74e518a to 91d3b35 Compare December 6, 2023 18:23
@averevki
Copy link
Contributor Author

averevki commented Dec 6, 2023

Dependent on #305

pehala
pehala previously approved these changes Jan 2, 2024
@averevki averevki requested review from jsmolar and ficap January 2, 2024 11:45
@averevki averevki force-pushed the test-authpolicy-attached-to-gateway branch from 91d3b35 to 91b8477 Compare January 2, 2024 14:57
@pehala pehala merged commit e92fa79 into Kuadrant:main Jan 3, 2024
2 checks passed
@averevki averevki deleted the test-authpolicy-attached-to-gateway branch November 11, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Test case New test case
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a simple test for AuthPolicy attached to a Gateway
4 participants