Oliver Newland

Results 13 comments of Oliver Newland

> i would add some sort of exponential `sleep` backoff here, but generally not sure if this is the sanest way to deal with those errors > > does the...

flag.Parse is eating my app's command-line arguments. I've figured out a way to patch redis.go to fail but not bring flag's parsing operation with it, but there's no simple way...

> @lynnagara should this PR be closed or are you planning to take up again? This is still valid, it fixes the issue that caused INC-1165

To repro locally, run: ``` docker run --rm -p 127.0.0.1:1218:1218 --network devservices --name snuba-snuba-1 -e DEFAULT_BROKERS=kafka-kafka-1:9092 -e REDIS_HOST=redis-redis-1 -e SNUBA_SETTINGS=docker -e CLICKHOUSE_HOST=clickhouse c22b79361144 # this hash is the result of...

https://github.com/getsentry/snuba/pull/7278 should make these changes unnecessary

This has a PR in review: https://github.com/getsentry/arroyo/pull/371 (posting to take it out of our support queue)

> same issue here with sentry [2.4.5](https://github.com/getsentry/sentry/releases/tag/24.5.0) version. The workaround works fine, but why it is not fixed in the code repository to avoid need to manually change this to...

the answer for this is probably to use basicSetup to give codemirror reasonable defaults: https://codemirror.net/docs/ref/#codemirror.basicSetup currently: https://github.com/getsentry/snuba/blob/master/snuba/admin/static/common/components/sql_editor.tsx#L31-L37 just want to check with @gggritso that there's nothing special in our logic...

basicSetup and minimalSetup seem to do something that conflicts with the DOM so maybe we just add in a custom history handler and call it a day: ``` hook.js:608 Warning:...

Hi @plemelin, Can you share the output of `docker compose logs clickhouse`? It sounds like your server isn't starting.