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

After deleting HTTPRoute, AuthorizationPolicy is still present #124

Closed
pehala opened this issue Jan 4, 2023 · 1 comment · Fixed by Kuadrant/testsuite#182
Closed

After deleting HTTPRoute, AuthorizationPolicy is still present #124

pehala opened this issue Jan 4, 2023 · 1 comment · Fixed by Kuadrant/testsuite#182
Assignees
@azgabur
Labels
kind/bug Something isn't working

Comments

@pehala
Copy link
Contributor

pehala commented Jan 4, 2023

Current behaviour:
If you delete HTTPRoute and have bound AuthPolicy to it, it will still route those requests to the Authorino.

Expected behaviour:
On deletion of HTTPRoute, the AuthPolicy is also invalidated, and all resources created for it will disappear.
This was referenced Jan 4, 2023
Closed
Closed
@pehala pehala added the kind/bug Something isn't working label Jan 10, 2023
@guicassolato guicassolato mentioned this issue Jan 17, 2023
Merged
@guicassolato guicassolato self-assigned this Jan 19, 2023
@guicassolato guicassolato moved this from In Progress to Done in Kuadrant Service Protection Jan 26, 2023
@pehala pehala moved this from Done to To test in Kuadrant Service Protection Jan 30, 2023
@pehala pehala assigned pehala and unassigned guicassolato Jan 30, 2023
@pehala pehala removed their assignment Feb 8, 2023
@azgabur azgabur self-assigned this Feb 23, 2023
@azgabur azgabur mentioned this issue Mar 15, 2023
Merged
@azgabur
Copy link

azgabur commented Mar 15, 2023

Manually tested deletion of HTTPRoute. The AuthorizationPolicy gets deleted with it and requests stop routing to Authorino returning 404.

@azgabur azgabur closed this as completed Mar 16, 2023
@azgabur azgabur moved this from To test to Done in Kuadrant Service Protection Mar 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@azgabur azgabur

Labels
kind/bug Something isn't working
Projects
No open projects
Kuadrant Service Protection
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add tests for HTTPRoute delete reconciliation azgabur/testsuite
3 participants
@guicassolato @pehala @azgabur