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

[Bug]: Enumerate all required components #3081

Open
DarkNeedle opened this issue Dec 19, 2024 · 1 comment
Open

[Bug]: Enumerate all required components #3081

DarkNeedle opened this issue Dec 19, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@DarkNeedle
Copy link

DarkNeedle commented Dec 19, 2024

What happened?

A simple local run using the pre-reqs as in README.md yields.

helicone-jawn | Error processing request 6d2df17d-7bbe-49a3-97e0-5876d52e4184 for batch : Error fetching content from S3: {"error":"Error fetching content from S3: Forbidden, 403"}
helicone-jawn | Required Kafka environment variables are not set, KafkaProducer will not be initialized.

I did not see in the documentation where Kafka was a requirement. I did not see where this is spun up either.

You also have clickhouse errors

clickhouse-migration-runner | Traceback (most recent call last):
clickhouse-migration-runner | File "ch_hcone.py", line 8, in
clickhouse-migration-runner | import tabulate
clickhouse-migration-runner | ModuleNotFoundError: No module named 'tabulate'
clickhouse-migration-runner exited with code 1

I have seen this a lot in repos where code is committed and breaks the main branch. My suggestion is the same, exercise a clean build or run in a VM following the instructions that have been outlined in the repo. Many times developers have existing packages/libraries installed where as a fresh install does not. Equally, keeping packages somewhat up to date along with n-1 on node will help.

All of this may work in your cloud offering, but it provides no confidence when the core cannot be run with following the README.md. Calls in to question if the source code is current or if the cloud offering is stagnant or put together with bubble gum and shoe string.

@DarkNeedle DarkNeedle added the bug Something isn't working label Dec 19, 2024
@connortbot
Copy link

Clickhouse migration succeeds(?) if tabulate is installed, there's a temp soln in #2965
Kafka is not supposed to be needed for local hosting but even with ^^^ I am getting the same result w.r.t KafkaProducer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants