search

Unstructured-IO / unstructured

Open source libraries and APIs to build custom preprocessing pipelines for labeling, training, or production machine learning pipelines.
https://www.unstructured.io/
Apache License 2.0
7.44k stars 580 forks source link

FIX: Use proper Astra env vars name to match internal docs #3179

Closed erichare closed 1 week ago

erichare commented 3 weeks ago

Updates env var names to match Astra internal docs. Keeping as draft temporarily due to a 500 error which may or may not be related.

erichare commented 3 weeks ago

@potter-potter I opened this pr first, before the refactored source connector PR (which i have ready) because this standardizes the environment variables to match what our other integrations do, as well as what our documentation lists as the astra variables, for the endpoint and token.

I hope i caught all the instances where this needs to change - let me know where i can open a corresponding PR for the docs if you'd like me too.

Sorry about the confusion on this, just thought it might be easier to address it now rather than later - i left a TODO item as a comment which is that, if desired, we could update the github secret to match the env var name - but (hopefully) this PR should work without that change as is.

potter-potter commented 3 weeks ago

@erichare This seems fine. And thanks for adjusting the docs in our new Docs repo.

I'll add this to the Astra Source connector PR when it is created. Just to keep things simple.

erichare commented 3 weeks ago

@erichare This seems fine. And thanks for adjusting the docs in our new Docs repo.

I'll add this to the Astra Source connector PR when it is created. Just to keep things simple.

Thanks so much. I’ll try to have that opened up today. Appreciate it.