Override output dir for preorpcessor

[gmechali]

No description provided.

[gemini-code-assist]

Code Review

This pull request updates the ingestion preprocessing job configuration to append /datacommons to both the ingestion artifacts path and the OUTPUT_DIR environment variable. The review feedback suggests using trimsuffix to sanitize these paths, preventing potential double slashes if the base paths already end with a trailing slash.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

To prevent potential double slashes (e.g., //datacommons) in the GCS path if env.value already has a trailing slash, consider using trimsuffix to sanitize the path.

  preprocessing_env_variables = [
    for env in local.cloud_run_shared_env_variables :
    env.name == "OUTPUT_DIR" ? {
      name  = "OUTPUT_DIR"
      value = "${trimsuffix(env.value, "/")}/datacommons"
    } : env
  ]

[gemini-code-assist]

To prevent potential double slashes (e.g., //datacommons) in the GCS path if var.ingestion_config.ingestion_artifacts_path already has a trailing slash, consider using trimsuffix to sanitize the path.

  ingestion_artifacts_path = "${trimsuffix(var.ingestion_config.ingestion_artifacts_path, "/")}/datacommons"