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 mirrors check to test-suite #302

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Add mirrors check to test-suite #302

merged 1 commit into from
Nov 7, 2024

Conversation

rgaudin
Copy link
Member

@rgaudin rgaudin commented Nov 6, 2024

test-suite now gets a new module (test_mirrors) that checks a few things regarding mirrorbrain and out mirrors

  • mirrors.html is there and enough populated
  • ZIM permalinks are working
  • release (APK) permalink are working
  • Load balancer is working (redirecting)
  • ZIM file is present on all mirrors
  • APK is present on all mirrors mirroring releases
  • ZIM files are sent with a Content-Type header
  • APK files are sent with a Content-Type header
  • ZIM files are served with application/octet-stream
  • APK files are served with application/vnd.android.package-archive

Workflows tested:

I did not create a ticket for the problem as I needed the solution for #301. I think there's enough info in PR/commit

@rgaudin rgaudin requested a review from benoit74 November 6, 2024 14:16
@rgaudin rgaudin self-assigned this Nov 6, 2024
Copy link
Collaborator

@benoit74 benoit74 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the significant work done (and no need for an issue if work is already done, my point about opening an issue was more that we might not do the work right now ... but if it is done, then fine ^^)

test-suite now gets a new module (test_mirrors) that checks a few things
regarding mirrorbrain and out mirrors

- mirrors.html is there and enough populated
- ZIM permalinks are working
- release (APK) permalink are working
- Load balancer is working (redirecting)
- ZIM file is present on all mirrors
- APK is present on all mirrors mirroring releases
- ZIM files are sent with a Content-Type header
- APK files are sent with a Content-Type header
- ZIM files are served with `application/octet-stream`
- APK files are served with `application/vnd.android.package-archive`
@rgaudin rgaudin merged commit 95ec3b3 into main Nov 7, 2024
1 check passed
@rgaudin rgaudin deleted the test-mirrors branch November 7, 2024 09:07
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