Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add download option to jenkins module #1073

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Apr 19, 2023

This allows forcing HTTP 1.1 per official suggestion, since HTTP 2 regularly fails when downloading.

@evgeni evgeni force-pushed the download-options branch 2 times, most recently from 4ac3c88 to fa06d8c Compare April 23, 2024 18:11
evgeni
evgeni previously approved these changes Apr 23, 2024
@evgeni
Copy link
Member

evgeni commented Apr 23, 2024

@ekohl @CyberLine bug or enhancement? I'm leaning bug as it fixes unstable downloads?

@evgeni
Copy link
Member

evgeni commented Apr 23, 2024

Welp, EL7's curl doesn't know --http1.1:

# curl --http1.1 https://…
curl: option --http1.1: is unknown
curl: try 'curl --help' or 'curl --manual' for more information

@evgeni evgeni dismissed their stale review April 23, 2024 18:42

needs el7 fixed

@evgeni evgeni force-pushed the download-options branch from fa06d8c to 39b2ec0 Compare April 24, 2024 09:44
manifests/plugin.pp Outdated Show resolved Hide resolved
This allows forcing HTTP 1.1 per official suggestion[1], since HTTP 2
regularly fails when downloading.

cURL on RHEL7 doesn't support HTTP 2 and thus does not have a --http1.1
switch.

[1]: jenkins-infra/helpdesk#3078 (comment)
@evgeni evgeni force-pushed the download-options branch from 39b2ec0 to cb975d4 Compare April 24, 2024 09:47
@evgeni evgeni merged commit 8d58576 into voxpupuli:master Apr 24, 2024
15 checks passed
@evgeni evgeni added the enhancement New feature or request label May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants