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

Spec the status reporting behaviour #96

Closed
1 of 4 tasks
alexsnaps opened this issue Sep 15, 2022 · 1 comment
Closed
1 of 4 tasks

Spec the status reporting behaviour #96

alexsnaps opened this issue Sep 15, 2022 · 1 comment
Assignees
Labels
area/api Changes user facing APIs

Comments

@alexsnaps
Copy link
Member

alexsnaps commented Sep 15, 2022

Using limitador server as an example to illustrate the scope of this issue:

When a new RateLimitPolicy is added or modified it will ultimately be reflected as a change to a ConfigMap mounted on the Limitador pod. While things can go "wrong" along the way to the CM, all actors are "k8s aware" and can reflect any status changes so to report to the users. It is necessary for us to decide how to report these back up the chain to the user. But when things make it to Limitador, the server as no way to report back to k8s. So we need to decide how Limitador could for instance report a syntax error in a Condition back to the matching RateLimitPolicy (possibly translating to "what" actually triggered the condition to be "generated" in this "invalid" form).

@alexsnaps alexsnaps added kind/investigation Spike into investigating something size/small target/current labels Sep 15, 2022
@alexsnaps alexsnaps changed the title Spec the error reporting behavior Spec the status reporting behaviour Nov 3, 2022
@didierofrivia didierofrivia transferred this issue from Kuadrant/kuadrant-controller Nov 8, 2022
@alexsnaps alexsnaps moved this from To do to Needs refinement in Kuadrant Service Protection Jan 19, 2023
@alexsnaps alexsnaps added the area/api Changes user facing APIs label Jan 19, 2023
@alexsnaps alexsnaps moved this from Needs refinement to To do in Kuadrant Service Protection Jan 19, 2023
@didierofrivia didierofrivia self-assigned this Feb 3, 2023
@didierofrivia didierofrivia moved this from To do to In Progress in Kuadrant Service Protection Feb 8, 2023
@didierofrivia didierofrivia moved this from In Progress to To do in Kuadrant Service Protection Feb 8, 2023
@didierofrivia didierofrivia moved this from To do to In Progress in Kuadrant Service Protection Feb 9, 2023
@alexsnaps alexsnaps moved this from In Progress to To do in Kuadrant Service Protection Jun 15, 2023
@alexsnaps
Copy link
Member Author

Dup // see Kuadrant/architecture#9

@github-project-automation github-project-automation bot moved this from To do to To test in Kuadrant Service Protection Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Changes user facing APIs
Projects
No open projects
Status: To test
Development

No branches or pull requests

2 participants