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

feat(publick8s) fine tune ingress rule for updates.jenkins.io #4672

Merged

Conversation

dduportal
Copy link
Contributor

Related to jenkins-infra/helpdesk#2649 (comment)

This PR fine tune the ingress rule of update.jenkins.io to only send requests ending with .json, .html or .txt extensions to mirrorbits:

  • The https://updates.jenkins.io/ URL will still be sent to Apache (which has all the files) so it can serve the index.html file as the default file before trying listing diretctories
  • All other html and txt files are copied to the mirror along the json files (both UC and crawler)

Signed-off-by: Damien Duportal <damien.duportal@gmail.com>
Copy link
Contributor Author

@dduportal dduportal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

helmfil diff says:

updates-jenkins-io, updates-jenkins-io-mirrorbits-parent, Ingress (networking.k8s.io) has changed:
...
                  port:
                    number: 80
-           - path: /.*[.](deb|hpi|war|rpm|msi|pkg|sha256|md5sum|zip|gz|pdf|json|svg|sh|jpeg|ico|png|html|txt)$
+           - path: /.*[.](json|html|txt)$
              pathType: ImplementationSpecific
              backend:
...

let's roll

@dduportal dduportal merged commit 88e7c45 into jenkins-infra:main Nov 15, 2023
6 checks passed
@dduportal dduportal deleted the feat/publick8s/tune-UC-ingress branch November 15, 2023 09:30
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