Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

Kuadrant/kuadrant-operator

maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
```suggestion existingRecordObj, recordExists := lo.Find(topology.Objects().Children(listener), func(o machinery.Object) bool { ```

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

eguzki created a comment on a pull request on Kuadrant/kuadrant-operator
Closing in favor of https://github.com/Kuadrant/kuadrant-operator/pull/893 I leave the branch undeleted because there are some integration tests that can be backported

View on GitHub

eguzki closed a draft pull request on Kuadrant/kuadrant-operator
Rate limiting limits controller
### What Custom controller for Limitador limits reading from DAG Gateway API topology and RateLimitPolicies Limitador cluster envoy filter controller reading from DAG Gateway API topology and...
maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
minor thing

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
yeah I think we could break here once found?

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

guicassolato pushed 1 commit to sotw/auth Kuadrant/kuadrant-operator

View on GitHub

codecov[bot] created a comment on a pull request on Kuadrant/kuadrant-operator
## [Codecov](https://app.codecov.io/gh/Kuadrant/kuadrant-operator/pull/952?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term...

View on GitHub

guicassolato opened a draft pull request on Kuadrant/kuadrant-operator
sotw: auth
State-of-the-world reconciler – Auth workflow - [x] `spec.targetRef.sectionName` - [x] `spec.(defaults|overrides).strategy` - [x] Defaults & Overrides merge strategies ([RFC-0009](https://docs.kua...
guicassolato created a branch on Kuadrant/kuadrant-operator

sotw/auth - The Operator to install and manage the lifecycle of the Kuadrant components deployments.

maksymvavilov created a review comment on a pull request on Kuadrant/kuadrant-operator
We use this ID to ensure records in the DNSProvider don't conflict with each other in the multicluster context. I'm wondering if we can make topologies unique when creating them (we do client reque...

View on GitHub

maksymvavilov created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

Boomatang created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maksymvavilov created a review comment on a pull request on Kuadrant/kuadrant-operator
What `NegativePolirity` meats for the record condition?

View on GitHub

maksymvavilov created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maksymvavilov created a review comment on a pull request on Kuadrant/kuadrant-operator
how do we end up only with policies that target a DNS record under reconciliation? and should there be more than one policy?

View on GitHub

maksymvavilov created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
does this remove it from the topology and then it gets re-added when we update it?

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maksymvavilov created a review comment on a pull request on Kuadrant/kuadrant-operator
why not `meta.IsStatusConditionTrue()?`

View on GitHub

maksymvavilov created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
doesn't seem to be much left in here, do we want to get rid entirely?

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
nit: any reason not to just pass the function straight instead of assigning it here?

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maleck13 created a review comment on a pull request on Kuadrant/kuadrant-operator
Do we also want events for DNSRecord? As DNSPolicy owns DNSRecord and if it is changed manually, we would want to set it back. Or is DNSPolicy enough to trigger that behaviour?

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

maleck13 created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

Load more