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

Fixing local docker setup configuration for easier e2e dev testing #9

Closed
wants to merge 1 commit into from

Conversation

zarna1parekh
Copy link
Collaborator

Summary

  • Fixing the local setup configuration to help with dev testing.
  • Added a flag (S3_FORCE_PATH_STYLE) for S3Async client to indicate how to read the buckets.

Had a relevant Astra PR but it is waiting on other object store access changes.

Requirements

@autata
Copy link
Collaborator

autata commented Aug 28, 2024

i think we should close this one for now. it looks like the refactor landed slackhq#1030

and we probably want to see if the fake s3 is still broken with the new implementation. another thought is setting INDEXER_MAX_TIME_PER_CHUNK_SECONDS=0 in the docker compose file since 1.5 hours is really fast for local dev and we can just turn it off.

@zarna1parekh zarna1parekh force-pushed the airbnb-main branch 2 times, most recently from 65bc1ba to 83d2138 Compare September 5, 2024 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants