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

chore(wrapper/publish) serve uctest.json and the HTML in the virtual tree download/ from Apache instead of mirrors #813

Merged

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Oct 21, 2024

Related to jenkins-infra/helpdesk#4311 (comment) (second bullet).

Closes 810 (as not needed anymore)

This PR is a follow up of #812 for the new Update Center using Apache2 + Mirrorbits.

It updates the publication script to serve the following file from Apache instead of mirrors:

  • uctest.json => solves the HTTP/404 on azure.updates.jenkins.io/whatever?uctest
  • download/**/*.html (only these files are present: other are sent to get.jenkins.io` => should solve the broken links in these HTML pages

As #812 did add a condition on "the file must not exist in httpd documentroot" to redirect incoming requests to the UC mirrors, Apache will serve these files.

As discussed with @daniel-beck:

  • These files will count in the (billed) outbound bandwidth as served by Apache, but the amount of traffic is really low so it does not even count
  • These files won't benefit from the new "mirror proximity" feature. But we do not really care as it is a really low usage
  • We won't need to define an absolute URL (constraining us for years to a scheme and an hostname: not really flexible)

…l tree `download/` from Apache instead of mirrors

Signed-off-by: Damien Duportal <[email protected]>
@daniel-beck
Copy link
Contributor

Closes #810 (as not needed anymore)

Sure, but as we've discussed, it's a good idea anyway to link to the canonical URLs.

@dduportal
Copy link
Contributor Author

Closes #810 (as not needed anymore)

Sure, but as we've discussed, it's a good idea anyway to link to the canonical URLs.

Updated the body then, to avoid PR being closed

@dduportal
Copy link
Contributor Author

Merging as it's only scoped to the publish.sh script to solve the new UC remaining errors

@dduportal dduportal closed this Oct 22, 2024
@dduportal dduportal reopened this Oct 22, 2024
@dduportal dduportal merged commit bcb5398 into jenkins-infra:master Oct 22, 2024
2 checks passed
@dduportal dduportal deleted the chore/wrappers/publish/httpd-files branch October 22, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants