Prevent _get_api_instance from modifying OpenAPI config

[austin-denoble]

Problem

Previous revision we added some code to apply a custom host value to control plane operations: #218

This change caused 404s when calling dataplane operations. The Configuration._base_path value was being overwritten by the custom host that was provided due to the Configuration.host setter:

  @host.setter
  def host(self, value):
      """Fix base path."""
      self._base_path = value
      self.server_index = None

pinecone-python-client/pinecone/core/client/configuration.py

Line 493 in 7ff4ed8

def host(self, value):

Solution

The change I made was actually modifying the reference to Config.OPENAPI_CONFIG which overwrote things across the board.

Instead of operating on the singleton's instance of the config, we can use copy.deepcopy(Config.OPENAPI_CONFIG) and use that to construct things before handing it off to ApiClient

Type of Change

• Bug fix (non-breaking change which fixes an issue)

Test Plan

Pull this branch down, use the client locally and make sure that dataplane calls work properly without a host or with one in the case of supplying a custom control plane URL.

[austin-denoble]

This is what we're doing in index.py, I think it's probably prudent to do that here as well rather than possibly changing fields on the singleton.

pinecone-python-client/pinecone/index.py

Line 92 in 7ff4ed8

openapi_client_config = copy.deepcopy(Config.OPENAPI_CONFIG)