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

fix: serve all static files previously accessible from stats.jenkins.io #168

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

lemeurherveCB
Copy link
Contributor

@lemeurherveCB lemeurherveCB commented Sep 13, 2024

This PR allows to continue serving all static files that were previously available from stats.jenkins.io and consumed from many places.

I've kept it KISS, copying over files (.gitignored) into the /public folder so they're serve without any more change.
This can be refined later if deemed important, the goal here is to restore broken URLs.

Tested locally.

Ref:

cc @dduportal @krisstern @timja

@lemeurherveCB
Copy link
Contributor Author

lemeurherveCB commented Sep 13, 2024

(This account doesn't have permission to merge the PR, feel free to do so)

@timja timja merged commit 936d4fd into jenkins-infra:main Sep 13, 2024
3 checks passed
@lemeurherveCB lemeurherveCB deleted the fix-serve-static-files branch September 13, 2024 20:11
@lemeurherveCB
Copy link
Contributor Author

@dduportal
Copy link
Contributor

FYI This PR broke the production deployment: jenkins-infra/helpdesk#4132 (comment)

@dduportal
Copy link
Contributor

Azcopy logs (needed to run a replay and gather the log in the post {} block) shows there are 2 files, with the same name, from different directories, to be copied:

2024/09/14 06:11:11 WARN: [P#1-T#1064] Block size 8388608 larger than maximum file chunk size, 4 MB chunk size used
2024/09/14 06:11:11 INFO: [P#1-T#1064] Starting transfer: Source "/home/jenkins/workspace/ction-jobs_stats.jenkins.io_main/dist/plugin-installation-trend/perfpublisher.stats.json" Destination "https://statsjenkinsio.file.core.windows.net/stats-jenkins-io/plugin-installation-trend/perfpublisher.stats.json". Specified chunk size 4194304
2024/09/14 06:11:11 ERR: [P#1-T#1064] UPLOADFAILED: /home/jenkins/workspace/ction-jobs_stats.jenkins.io_main/dist/plugin-installation-trend/perfpublisher.stats.json : 000 : cannot simultaneously send two files to same destination name
   Dst: https://statsjenkinsio.file.core.windows.net/stats-jenkins-io/plugin-installation-trend/perfpublisher.stats.json


2024/09/14 06:11:11 WARN: [P#1-T#1066] Block size 8388608 larger than maximum file chunk size, 4 MB chunk size used
2024/09/14 06:11:11 INFO: [P#1-T#1066] Starting transfer: Source "/home/jenkins/workspace/ction-jobs_stats.jenkins.io_main/dist/pluginversions/perfpublisher.html" Destination "https://statsjenkinsio.file.core.windows.net/stats-jenkins-io/pluginversions/perfpublisher.html". Specified chunk size 4194304
2024/09/14 06:11:11 ERR: [P#1-T#1066] UPLOADFAILED: /home/jenkins/workspace/ction-jobs_stats.jenkins.io_main/dist/pluginversions/perfpublisher.html : 000 : cannot simultaneously send two files to same destination name
   Dst: https://statsjenkinsio.file.core.windows.net/stats-jenkins-io/pluginversions/perfpublisher.html

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.

3 participants