Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
guicassolato created a comment on an issue on Kuadrant/kuadrant-operator
Being implemented for Rate Limit in #893 according to https://github.com/Kuadrant/wasm-shim/pull/110
guicassolato created a comment on an issue on Kuadrant/kuadrant-operator
This makes sense if we can automate enabling EnvoyPatchPolicies in Envoy Gateway, otherwise it can be closed I guess. cc @eguzki
guicassolato created a comment on an issue on Kuadrant/kuadrant-operator
I think we're not doing this anymore. Closing.
guicassolato closed an issue on Kuadrant/kuadrant-operator
[state-of-the-world reconciler] Gateway assignment
Annotate gateways in the scope of a Kuadrant instance.
guicassolato created a comment on an issue on Kuadrant/kuadrant-operator
This is not addressed by #893. I've removed the link.
codecov[bot] created a comment on a pull request on Kuadrant/kuadrant-operator
## [Codecov](https://app.codecov.io/gh/Kuadrant/kuadrant-operator/pull/945?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term...
guicassolato created a branch on Kuadrant/kuadrant-operator
disable-authorino-webhooks - The Operator to install and manage the lifecycle of the Kuadrant components deployments.
eguzki closed an issue on Kuadrant/kuadrant-operator
[state-of-the-world reconciler] Wasm module
Configure the wasm module integration for supported Gateway API providers.
eguzki created a comment on an issue on Kuadrant/kuadrant-operator
I think we can close this issue. The integration with Istio and EnvoyGateway using a wasm module is already done. **How** we use this integration is covered in https://github.com/Kuadrant/kuadr...
guicassolato created a comment on an issue on Kuadrant/kuadrant-operator
> @guicassolato plural targetRefs are in the scope of v1? > > Is this issue in the scope of #893 ? Yep. Let's push this to v1.1 for now.
guicassolato pushed 4 commits to sotw/rlp Kuadrant/kuadrant-operator
- set owner reference directly when building the desired objects Signed-off-by: Guilherme Cassolato <guicassolato@gmai... 46fb400
- only validate policies on create and update events Signed-off-by: Guilherme Cassolato <[email protected]> fe42854
- update policy status on all kinds of policy-releated event (deleting a policy may also affect the state of the resour... 4aa8229
- use labels on the internal resources created to watch and select them from topology Signed-off-by: Guilherme Cassola... a329d0e
eguzki closed an issue on Kuadrant/kuadrant-operator
[state-of-the-world reconciler] Istio config
Top-level Istio mesh configs to enable Kuadrant integration.
eguzki created a comment on an issue on Kuadrant/kuadrant-operator
This issue no longer seems valid. It should be closed, as integration with istio/OSSM regarding external authorization service will be covered by https://github.com/Kuadrant/kuadrant-operator/issue...
mikenairn created a review comment on a pull request on Kuadrant/kuadrant-operator
Related to https://github.com/Kuadrant/kuadrant-operator/pull/937#discussion_r1805111455, can keep the conversation about this there.
mikenairn created a review comment on a pull request on Kuadrant/kuadrant-operator
Yes, so i actually wasn't aware until you commented and i looked more closely at the controller code that there was a way to filter out the objects that get added to the topology. This is definitel...
mikenairn created a review comment on a pull request on Kuadrant/kuadrant-operator
Added this predicate based on what had been added for TLS but I'm unsure of its usefulness when we are using a label filter. The label filter gets added as a predicate internally anyway. I'm thinki...
guicassolato created a comment on an issue on Kuadrant/kuadrant-operator
> this is already happening, isn't it? @guicassolato For the Rate Limit part, yes.
eguzki created a comment on an issue on Kuadrant/kuadrant-operator
@guicassolato plural targetRefs are in the scope of v1? Is this issue in the scope of https://github.com/Kuadrant/kuadrant-operator/pull/893 ?
eguzki created a comment on an issue on Kuadrant/kuadrant-operator
this is already happening, isn't it? @guicassolato
mikenairn created a review comment on a pull request on Kuadrant/kuadrant-operator
Copied the layout and labels from https://github.com/Kuadrant/kuadrant-operator/blob/main/pkg/openshift/consoleplugin/common.go
Boomatang created a review on a pull request on Kuadrant/kuadrant-operator
I have removed so of my previous comments. After looking at this a second time last night, I had a misunderstanding of what was going on here. There is something going on here that seems out of pla...
KevFan created a review comment on a pull request on Kuadrant/kuadrant-operator
Yeah, I think it compares the pointers between the two policies
KevFan created a review comment on a pull request on Kuadrant/kuadrant-operator
Hmm, that's true. If you feel strongly on this I can update, but using the owner refernence looks a lot more readable unless filtering the list of policies should be avoided in general? 🤔