Merge branch 'main' into firestore-change-headers

Author: bshaffer

.github/run-package-tests.sh

@@ -13,6 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+set -e
+
 DIRS=$(find * -maxdepth 0 -type d -name '[A-Z]*')
 PREFER_LOWEST=""
 if [ "$#" -eq 1 ]; then
@@ -42,45 +44,57 @@ fi
 export COMPOSER=composer-local.json
 
 FAILED_FILE=$(mktemp -d)/failed
-for DIR in ${DIRS}; do {
-    cp ${DIR}/composer.json ${DIR}/composer-local.json
+for DIR in ${DIRS}; do
+    echo "--- Processing ${DIR} ---"
+    cp "${DIR}/composer.json" "${DIR}/composer-local.json"
     # Update composer to use local packages
-    for i in CommonProtos,common-protos,4.100 BigQuery,cloud-bigquery Core,cloud-core Logging,cloud-logging PubSub,cloud-pubsub Storage,cloud-storage ShoppingCommonProtos,shopping-common-protos GeoCommonProtos,geo-common-protos,0.1; do
-        IFS=","; set -- $i;
-        if grep -q "\"google/$2\":" ${DIR}/composer.json; then
+    PACKAGE_DEPENDENCIES=(
+        "CommonProtos,common-protos,4.100"
+        "BigQuery,cloud-bigquery"
+        "Core,cloud-core"
+        "Logging,cloud-logging"
+        "PubSub,cloud-pubsub"
+        "Storage,cloud-storage"
+        "ShoppingCommonProtos,shopping-common-protos"
+        "GeoCommonProtos,geo-common-protos,0.1"
+    )
+    for i in "${PACKAGE_DEPENDENCIES[@]}"; do
+        IFS="," read -r PKG_DIR PKG_NAME PKG_VERSION <<< "$i"
+        if grep -q "\"google/${PKG_NAME}\":" "${DIR}/composer.json"; then
             # determine local package version
-            if [ "$STRICT" = "true" ]; then VERSION=$(cat $1/VERSION); elif [ -z "$3" ]; then VERSION="1.100"; else VERSION=$3; fi
-            echo "Use local package $1 as google/$2:$VERSION in $DIR"
+            if [ "${STRICT}" = "true" ]; then
+                VERSION=$(cat "${PKG_DIR}/VERSION")
+            elif [ -z "${PKG_VERSION}" ]; then
+                VERSION="1.100"
+            else
+                VERSION=${PKG_VERSION}
+            fi
+            echo "Use local package ${PKG_DIR} as google/${PKG_NAME}:${VERSION} in ${DIR}"
             # "canonical: false" ensures composer will try to install from packagist when the "--prefer-lowest" flag is set.
-            composer config repositories.$2 -d ${DIR} "{\"type\": \"path\", \"url\": \"../$1\", \"options\":{\"versions\":{\"google/$2\":\"$VERSION\"}},\"canonical\":false}"
+            JSON_CONFIG=$(printf '{"type":"path","url":"../%s","options":{"versions":{"google/%s":"%s"}},"canonical":false}' "${PKG_DIR}" "${PKG_NAME}" "${VERSION}")
+            composer config "repositories.${PKG_NAME}" -d "${DIR}" "${JSON_CONFIG}"
         fi
     done
 
-    echo -n "Installing composer in $DIR"
-    if [ "$PREFER_LOWEST" != "" ]; then
-        echo -n " (with $PREFER_LOWEST)"
+    echo -n "Installing composer in ${DIR}"
+    if [ -n "${PREFER_LOWEST}" ]; then
+        echo -n " (with ${PREFER_LOWEST})"
     fi
     echo ""
-    composer -q --no-interaction --no-ansi --no-progress $PREFER_LOWEST update -d ${DIR};
-    if [ $? != 0 ]; then
-        echo "$DIR: composer install failed" >> "${FAILED_FILE}"
+    composer -q --no-interaction --no-ansi --no-progress ${PREFER_LOWEST} update -d "${DIR}" || {
+        echo "${DIR}: composer install failed" >> "${FAILED_FILE}"
         # run again but without "-q" so we can see the error
-        composer --no-interaction --no-ansi --no-progress $PREFER_LOWEST update -d ${DIR};
+        composer --no-interaction --no-ansi --no-progress ${PREFER_LOWEST} update -d "${DIR}"
         continue
-    fi
-    echo "Running $DIR Unit Tests"
-    ${DIR}/vendor/bin/phpunit -c ${DIR}/phpunit.xml.dist;
-    if [ $? != 0 ]; then
-        echo "$DIR: failed" >> "${FAILED_FILE}"
-    fi
+    }
+    echo "Running ${DIR} Unit Tests"
+    "${DIR}/vendor/bin/phpunit" -c "${DIR}/phpunit.xml.dist" || echo "${DIR}: failed" >> "${FAILED_FILE}"
+
     if [ -f "${DIR}/phpunit-snippets.xml.dist" ]; then
-        echo "Running $DIR Snippet Tests"
-        ${DIR}/vendor/bin/phpunit -c ${DIR}/phpunit-snippets.xml.dist;
-        if [ $? != 0 ]; then
-            echo "$DIR (snippets): failed" >> "${FAILED_FILE}"
-        fi
+        echo "Running ${DIR} Snippet Tests"
+        "${DIR}/vendor/bin/phpunit" -c "${DIR}/phpunit-snippets.xml.dist" || echo "${DIR} (snippets): failed" >> "${FAILED_FILE}"
     fi
-}; done
+done
 
 if [ -f "${FAILED_FILE}" ]; then
     echo "--------- Failed tests --------------"

.github/workflows/backwards-compatibility-checks.yaml

@@ -0,0 +1,58 @@
+# Copyright 2025 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+# Github action job to test core java library features on
+# downstream client libraries before they are released.
+on:
+  push:
+    branches:
+      - main
+    paths:
+      - 'Bigtable/**'
+      - '.github/workflows/conformance-tests-bigtable-proxy.yaml'
+  pull_request:
+    paths:
+      - 'Bigtable/**'
+      - '.github/workflows/conformance-tests-bigtable-proxy.yaml'
+  workflow_dispatch:
+name: Run Bigtable Conformance Tests With Proxy
+jobs:
+  conformance:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v5
+
+      - name: Setup PHP
+        uses: shivammathur/setup-php@v2
+        with:
+          php-version: '8.1'
+          extensions: grpc
+
+      - name: Checkout Bigtable conformance tests
+        uses: actions/checkout@v5
+        with:
+          repository: googleapis/cloud-bigtable-clients-test
+          ref: v0.0.3
+          path: cloud-bigtable-clients-test
+
+      - uses: actions/setup-go@v5
+        with:
+          go-version: '>=1.20.2'
+
+      - name: Install Road Runner PHP
+        run: |
+          wget https://github.com/roadrunner-server/roadrunner/releases/download/v2024.3.1/roadrunner-2024.3.1-linux-amd64.deb
+          sudo dpkg -i roadrunner-2024.3.1-linux-amd64.deb
+
+      - run: bash Bigtable/tests/Conformance/proxy/conformance.sh

.github/workflows/conformance-tests-bigtable-emulator.yaml

@@ -16,13 +16,13 @@ jobs:
 
     services:
       emulator:
-        image: gcr.io/cloud-devrel-public-resources/storage-testbench:v0.53.0
+        image: gcr.io/cloud-devrel-public-resources/storage-testbench:v0.55.0
         ports:
           - 9000:9000
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2

.github/workflows/conformance-tests-bigtable-proxy.yaml

@@ -12,7 +12,7 @@ jobs:
     env:
       PHPDOC_ENV: prod
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v5
     - name: Setup Python
       uses: actions/setup-python@v5
       with: