particle-os/debian-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

sources(curl): disable curl --parallel by default

Browse source 
Disable `curl --parallel` by default until the failure in
https://github.com/osbuild/osbuild-composer/pull/4247

is fully understood. It can be enabled via the environment:
```
OSBUILD_SOURCES_CURL_USE_PARALLEL=1
```
in the osbuild-composer test.
This commit is contained in:
Michael Vogt 2024-07-08 12:48:23 +02:00 • committed by Achilleas Koutsou
parent 4697a3fb84
commit 9bef57d5a6
2 changed files with 17 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

13
sources/test/test_curl_source.py
View file

@ -160,7 +160,9 @@ def test_curl_download_many_chksum_validate(tmp_path, curl_parallel):
@pytest.mark.parametrize("curl_parallel", [True, False], indirect=["curl_parallel"])
def test_curl_download_many_retries(tmp_path, curl_parallel):
def test_curl_download_many_retries(tmp_path, monkeypatch, curl_parallel):
monkeypatch.setenv("OSBUILD_SOURCES_CURL_USE_PARALLEL", "1")
fake_httpd_root = tmp_path / "fake-httpd-root"
with http_serve_directory(fake_httpd_root) as httpd:
@ -291,7 +293,14 @@ Features: AsynchDNS IPv6 Largefile GSS-API Kerberos SPNEGO NTLM NTLM_WB SSL libz
@patch("subprocess.check_output")
def test_curl_has_parallel_download(mocked_check_output, sources_module):
def test_curl_has_parallel_download(mocked_check_output, monkeypatch, sources_module):
# by default, --parallel is disabled
mocked_check_output.return_value = NEW_CURL_OUTPUT
assert not sources_module.curl_has_parallel_downloads()
# unless this environemnt is set
monkeypatch.setenv("OSBUILD_SOURCES_CURL_USE_PARALLEL", "1")
mocked_check_output.return_value = NEW_CURL_OUTPUT
assert sources_module.curl_has_parallel_downloads()