Ecosyste.ms: Timeline

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

Kuadrant/kuadrant-operator

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

View on GitHub

KevFan created a review comment on a pull request on Kuadrant/kuadrant-operator
Yeah, this outputting happened since moving to the policy-machinery (previous to this, it worked as it should by outputting logs only when a test fails). So this looks good to me until we resolv...

View on GitHub

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

View on GitHub

guicassolato created a review comment on a pull request on Kuadrant/kuadrant-operator
Being able to optionally `log.WriteTo(GinkgoWriter)` (default to `false`) would be nice, I guess. The otherwise noisy output sometimes helps us spot silent issues in advance, even when the tests su...

View on GitHub

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

View on GitHub

KevFan created a review on a pull request on Kuadrant/kuadrant-operator
Looks good to me πŸ‘

View on GitHub

eguzki created a review comment on a pull request on Kuadrant/kuadrant-operator
> Problem is the output without -v is rubbish as it doesn't even tell you what tests it's running I do not see it as a problem. What is the problem there? We can make it optional, something that...

View on GitHub

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

View on GitHub

guicassolato created a review comment on a pull request on Kuadrant/kuadrant-operator
> What i would like is the output you get with -v, without the logs unless you get an error +1

View on GitHub

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

View on GitHub

david-martin closed a pull request on Kuadrant/kuadrant-operator
Update steps for setting up metrics on openshift, focusing on single …
…cluster - combine service monitors for operators for easier deploy - update openshift install steps to reference this file and explain the setup better - remove reference to multi cluster tha...
david-martin opened a pull request on Kuadrant/kuadrant-operator
Add observability setup to Kubernetes install guide
Based on [readme](https://github.com/Kuadrant/kuadrant-operator/tree/main/config/observability#observability-stack-guide). Just for Istio (not Envoy Gateway) as there is extensive use of istio s...
david-martin created a branch on Kuadrant/kuadrant-operator

add-obs-guide-k8s - The Operator to install and manage the lifecycle of the Kuadrant components deployments.

mikenairn created a review comment on a pull request on Kuadrant/kuadrant-operator
I think this is an issue with ginkgo personally, or maybe i just don't know how to use it. You are correct, anything written to GinkgoWriter shouldn't appear unless it fails, that is unless you ...

View on GitHub

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

View on GitHub

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

View on GitHub

guicassolato created a review comment on a pull request on Kuadrant/kuadrant-operator
Yes, `HTTPQueryParamMatch`. Until we get CEL in, I think I'm gonna keep building redundant AuthConfig top-level conditions from the route matches then. At least this case of the query params wil...

View on GitHub

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

View on GitHub

eguzki created a review comment on a pull request on Kuadrant/kuadrant-operator
Confused here. Writting to GinkgoWriter should be helpful as it only shows errors when the test fails and not when the test passes. That is desirable, isn't it?? Or maybe you want to have logs d...

View on GitHub

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

View on GitHub

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

View on GitHub

eguzki created a review comment on a pull request on Kuadrant/kuadrant-operator
what about adding `VERBOSE` makefile variable to opt-in (by default false) ? Same as unit tests.

View on GitHub

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

View on GitHub

mikenairn created a comment on a pull request on Kuadrant/kuadrant-operator
Tests are likely going to fail because of a a race in the authorino tests: ``` WARNING: DATA RACE Read at 0x00c00077eb40 by goroutine 542: runtime.mapaccess1_faststr() /home/mnairn...

View on GitHub

david-martin pushed 1 commit to update-obs-on-openshift-doc Kuadrant/kuadrant-operator

View on GitHub

mikenairn opened a pull request on Kuadrant/kuadrant-operator
Test updates
**tests: Disable all output capture in ginkgo tests** This creates a lot of extra output during the test runs which is really distracting, and makes the output virtually useless. Apart from ...
guicassolato pushed 1 commit to sotw/auth Kuadrant/kuadrant-operator

View on GitHub

david-martin created a review comment on a pull request on Kuadrant/kuadrant-operator
Sure, i'll mention it here.

View on GitHub

david-martin created a review on a pull request on Kuadrant/kuadrant-operator

View on GitHub

mikenairn pushed 3 commits to sotw_dnspolicy Kuadrant/kuadrant-operator
  • tests: Improve test run output * Add verbose option to test runs so you can see the tests being executed. * Stop w... 35fbc85
  • dev: Add `--race` option to `make run` task Enables the race detector on locally running instances https://go.dev/do... 4d0cc64
  • fix: go mod update bump policy-machinery module https://github.com/Kuadrant/policy-machinery/pull/45 Signed-off-by:... 5f923fa

View on GitHub

Load more