Changeset 05a3fd9 in trunk

Timestamp:

2023-05-04T16:49:03Z (2 years ago)

Author:

Itamar Turner-Trauring <itamar@…>

Branches:

master

Children:

0b992c4

Parents:

3cc80ae2 (diff), ff1b7918 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge remote-tracking branch 'origin/master' into 4016-http-storage-content-type

Files:

• .github/workflows/ci.yml (modified) (1 diff)
• integration/conftest.py (modified) (1 diff)
• newsfragments/4022.minor (added)
• setup.py (modified) (1 diff)
• src/allmydata/test/common_system.py (modified) (3 diffs)
• src/allmydata/test/test_system.py (modified) (2 diffs)

.github/workflows/ci.yml ¶

@@ -170 +170 @@
             force-foolscap: false
           - os: windows-latest
-            python-version: "3.9"
+            python-version: "3.11"
             force-foolscap: false
           # 22.04 has some issue with Tor at the moment:
           # https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3943
           - os: ubuntu-20.04
-            python-version: "3.11"
+            python-version: "3.10"
             force-foolscap: false
     steps:

integration/conftest.py ¶

@@ -51 +51 @@
 from allmydata.node import read_config
 
-# No reason for HTTP requests to take longer than two minutes in the
+# No reason for HTTP requests to take longer than four minutes in the
 # integration tests. See allmydata/scripts/common_http.py for usage.
-os.environ["__TAHOE_CLI_HTTP_TIMEOUT"] = "120"
+os.environ["__TAHOE_CLI_HTTP_TIMEOUT"] = "240"
 
 # Make Foolscap logging go into Twisted logging, so that integration test logs

setup.py ¶

@@ -419 +419 @@
               "python-subunit==1.4.2",
               "junitxml==0.7",
-              "coverage ~= 5.0",
+              "coverage==7.2.5",
           ],
 

src/allmydata/test/common_system.py ¶

@@ -820 +820 @@
 
         self.helper_furl = helper_furl
-        if self.numclients >= 4:
-            with open(os.path.join(basedirs[3], 'tahoe.cfg'), 'a+') as f:
+        if self.numclients >= 2:
+            with open(os.path.join(basedirs[1], 'tahoe.cfg'), 'a+') as f:
                 f.write(
                     "[client]\n"
@@ -837 +837 @@
         # now find out where the web port was
         self.webish_url = self.clients[0].getServiceNamed("webish").getURL()
-        if self.numclients >=4:
+        if self.numclients >=2:
             # and the helper-using webport
-            self.helper_webish_url = self.clients[3].getServiceNamed("webish").getURL()
+            self.helper_webish_url = self.clients[1].getServiceNamed("webish").getURL()
 
     def _generate_config(self, which, basedir, force_foolscap=False):
@@ -855 +855 @@
 
             # client 0 runs a webserver and a helper
-            # client 3 runs a webserver but no helper
-            ("node", "web.port"): {0, 3},
+            # client 1 runs a webserver but no helper
+            ("node", "web.port"): {0, 1},
             ("node", "timeout.keepalive"): {0},
-            ("node", "timeout.disconnect"): {3},
+            ("node", "timeout.disconnect"): {1},
 
             ("helper", "enabled"): {0},

src/allmydata/test/test_system.py ¶

@@ -783 +783 @@
     def test_filesystem(self):
         self.data = LARGE_DATA
-        d = self.set_up_nodes()
+        d = self.set_up_nodes(2)
         def _new_happy_semantics(ign):
             for c in self.clients:
@@ -1747 +1747 @@
         return d
 
+    # In CI this test can be very slow, so give it a longer timeout:
+    test_filesystem.timeout = 360  # type: ignore[attr-defined]
+
+
     def test_filesystem_with_cli_in_subprocess(self):
         # We do this in a separate test so that test_filesystem doesn't skip if we can't run bin/tahoe.