You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opendal.apache.org by xu...@apache.org on 2026/04/14 06:33:25 UTC

(opendal) 01/01: ci: temporarily disable gdrive behavior tests

This is an automated email from the ASF dual-hosted git repository.

xuanwo pushed a commit to branch xuanwo/disable-gdrive-behavior-tests
in repository https://gitbox.apache.org/repos/asf/opendal.git

commit 0b6aab72884b7286a4ba0fc59d856d326bfbeab5
Author: Xuanwo <gi...@xuanwo.io>
AuthorDate: Tue Apr 14 14:33:09 2026 +0800

    ci: temporarily disable gdrive behavior tests
---
 .github/scripts/test_behavior/plan.py      | 12 ++++++++++++
 .github/scripts/test_behavior/test_plan.py | 18 ++++++++++++++++++
 2 files changed, 30 insertions(+)

diff --git a/.github/scripts/test_behavior/plan.py b/.github/scripts/test_behavior/plan.py
index ddb21080b..6c5e8d087 100755
--- a/.github/scripts/test_behavior/plan.py
+++ b/.github/scripts/test_behavior/plan.py
@@ -35,6 +35,14 @@ LANGUAGE_BINDING = ["java", "python", "nodejs", "go", "c", "cpp", "dotnet"]
 
 INTEGRATIONS = ["object_store"]
 
+TEMPORARILY_DISABLED_CORE_SERVICES = {
+    # FIXME: Re-enable gdrive behavior tests after #7384 is fixed.
+    # Gdrive is currently not reliable enough for CI: #6684 tracks missing
+    # freshly-created entries in listings, and existing behavior tests already
+    # document redirect loops caused by Google Drive abuse detection.
+    "gdrive",
+}
+
 
 def provided_cases() -> list[dict[str, str]]:
     root_dir = f"{GITHUB_DIR}/services"
@@ -59,6 +67,10 @@ def provided_cases() -> list[dict[str, str]]:
     if not os.getenv("GITHUB_HAS_SECRETS") == "true":
         cases[:] = [v for v in cases if "op://services" not in v["content"]]
 
+    cases[:] = [
+        v for v in cases if v["service"] not in TEMPORARILY_DISABLED_CORE_SERVICES
+    ]
+
     # Remove content from cases.
     cases = [
         {
diff --git a/.github/scripts/test_behavior/test_plan.py b/.github/scripts/test_behavior/test_plan.py
index 02a76bc71..fce380f37 100644
--- a/.github/scripts/test_behavior/test_plan.py
+++ b/.github/scripts/test_behavior/test_plan.py
@@ -16,6 +16,7 @@
 # under the License.
 
 import unittest
+from unittest.mock import patch
 
 from plan import plan
 
@@ -74,6 +75,23 @@ class BehaviorTestPlan(unittest.TestCase):
         # Should contain fs
         self.assertTrue("fs" in cases)
 
+    @patch.dict("os.environ", {"GITHUB_HAS_SECRETS": "true"}, clear=False)
+    def test_gdrive_is_temporarily_disabled(self):
+        result = plan([".github/workflows/test_behavior.yml"])
+
+        self.assertTrue(result["components"]["core"])
+        core_cases = [v["service"] for target in result["core"] for v in target["cases"]]
+        self.assertFalse("gdrive" in core_cases)
+
+        for language in ["java", "python", "nodejs", "go", "c", "cpp", "dotnet"]:
+            self.assertTrue(result["components"][f"binding_{language}"])
+            binding_cases = [
+                v["service"]
+                for target in result[f"binding_{language}"]
+                for v in target["cases"]
+            ]
+            self.assertFalse("gdrive" in binding_cases)
+
 
 if __name__ == "__main__":
     unittest.main()