fix: Pinecone define pod, pod_type

[x110]

Related Issues

• Fixes Pinecone define pod, pod_type and index_config #5964

Proposed Changes:

• Added support for the pods and pod_type parameters in the initialization of PineconeDocumentStore.

How did you test it?

• I've written a test using Haystack APIs to create a Pinecone index. The test involves querying the index properties through the Pinecone API and then verifying that the retrieved values align with the predefined configuration.
• In Pinecone starter plan, all create_index calls ignore the pod_type parameter and limits pods to 1. So I was not able to conduct the test.

Notes for the Reviewer

• To evaluate these changes, you can execute the test with a Pinecone subscription.
• The index_config parameter has been excluded as it is deprecated in the Pinecone library.

Checklist

• I have read the contributors guidelines and the code of conduct
• I have updated the related issue with new insights and changes
• I added unit tests and updated the docstrings
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.
• I documented my code
• I ran pre-commit hooks and fixed any issues

[anakin87]

Hello, @x110 and thanks for this contribution.

Some suggestions:

• the new tests should be added to the existing Pinecone test file
• some tests are failing. Please look at the errors and try to fix them. If you need help, feel free to ask...

[coveralls]

Pull Request Test Coverage Report for Build 6977366512

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 896 unchanged lines in 18 files lost coverage.
• Overall coverage decreased (-0.2%) to 40.113%

---

Files with Coverage Reduction	New Missed Lines	%
nodes/file_converter/init.py	1	88.89%
utils/context_matching.py	1	95.7%
preview/components/builders/init.py	2	0.0%
preview/components/generators/init.py	2	0.0%
preview/components/routers/init.py	2	0.0%
nodes/preprocessor/base.py	4	81.82%
preview/components/builders/prompt_builder.py	12	0.0%
preview/document_stores/protocols.py	15	0.0%
preview/components/audio/whisper_remote.py	24	0.0%
preview/utils/filters.py	25	0.0%

Totals
Change from base Build 6929759681:	-0.2%
Covered Lines:	10689
Relevant Lines:	26647

---

💛 - Coveralls

[anakin87]

Could you please add an integration test similar to that deleted in eb72798?

For the API key, let's do something similar to

haystack/test/document_stores/test_pinecone.py

Line 43 in b34c35d

api_key=os.environ.get("PINECONE_API_KEY") or "fake-pinecone-test-key",

[x110]

Hi @anakin87 , Thank you for your feedback. I have added an integration test. let me know if everything looks good now.

[anakin87]

Hey, @x110!

I am sorry to have kept you waiting. I will be testing your PR in depth soon...
Please bear with me.

[anakin87]

I finally spent some time on this PR and made some changes.

• I set default values for pod and pod_type (otherwise Pinecone API raises errors)

• I manually tested your changes with Starter and Standard plans. They work well!

• I removed the integration test, since I discovered in our CI we run tests using the Starter Plan

Thanks for your contribution 💙

(Don't miss Haystack 2.0 Beta)