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

Prevent import-examples to fail #2874

Merged
merged 1 commit into from
Dec 7, 2024
Merged

Prevent import-examples to fail #2874

merged 1 commit into from
Dec 7, 2024

Conversation

vmcj
Copy link
Member

@vmcj vmcj commented Dec 5, 2024

The mariadb images seems to now allow a smaller maximum packet by default. This resulted in problems for the boolfind problem which we didn't detect as all checks would still pass correctly.

We now also capture the SQL logs to detect this easier and increase the maximum packetsize (1G).

When this is accepted I'll move the installation of the docker.io to the runner image and remove that line for a small speedup.

The mariadb images seems to now allow a smaller maximum packet by
default. This resulted in problems for the boolfind problem which we
didn't detect as all checks would still pass correctly.

We now also capture the SQL logs to detect this easier and increase the maximum packetsize (1G).
@vmcj vmcj enabled auto-merge December 6, 2024 18:02
@vmcj vmcj added this pull request to the merge queue Dec 6, 2024
@vmcj vmcj removed this pull request from the merge queue due to a manual request Dec 6, 2024
@vmcj
Copy link
Member Author

vmcj commented Dec 7, 2024

The other errors will be fixed in #2876

@vmcj vmcj enabled auto-merge December 7, 2024 10:33
@vmcj vmcj added this pull request to the merge queue Dec 7, 2024
Merged via the queue into DOMjudge:main with commit e596a3b Dec 7, 2024
58 of 65 checks passed
@vmcj vmcj deleted the find_broken branch December 7, 2024 10:53
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