chore: Replace ValueError with logger.warning for missing GOOGLE_APPLICATION_CREDENTIALS environment variable

[hironow]

Overview

According to the Google Auth library, authentication should be possible without the GOOGLE_APPLICATION_CREDENTIALS environment variable by using commands like gcloud auth application-default login and gcloud config set project . However, the current implementation does not support this, so this pull request addresses and fixes that issue by logging a warning instead of raising a ValueError when the environment variable is not set. This change ensures consistency with the existing implementation and allows the use of alternative Google authentication methods.

Changes

• Replace the ValueError with a logger.warning

Related Information

By modifying the implementation to log a warning instead of raising an error, this pull request aligns the behavior with the intended flexibility of the Google Auth library (below). This allows applications to rely on other authentication methods seamlessly, enhancing compatibility with various deployment environments and configurations.

• Existing implementation's warning message:
  

  agents/livekit-plugins/livekit-plugins-openai/livekit/plugins/openai/llm.py

  Lines 224 to 228 in 489f786

  	_gac = os.environ.get("GOOGLE_APPLICATION_CREDENTIALS")
  	if _gac is None:
  	logger.warning(
  	"`GOOGLE_APPLICATION_CREDENTIALS` environment variable is not set. please set it to the path of the service account key file. Otherwise, use any of the other Google Cloud auth methods."
  	)

• Google Auth library's Credentials flexibility:
  https://github.com/googleapis/google-auth-library-python/blob/main/google/auth/_default.py#L577-L607

[changeset-bot]

🦋 Changeset detected

Latest commit: a57698b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
livekit-plugins-google	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[CLAassistant]

All committers have signed the CLA.

[theomonnom]

Thanks!