镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

ci: bump mariadb from 11.4 to 11.8 - #13045

Merged
kesselb merged 1 commit into
mainfrom
ci/update-mariadb-11.8
Jun 9, 2026
Merged

kesselb merged 1 commit into
mainfrom
ci/update-mariadb-11.8

Conversation

@ChristophWurst

Copy link
Copy Markdown
Member

11.8 is recommended: https://docs.nextcloud.com/server/latest/admin_manual/installation/system_requirements.html#server

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable5.9

Assisted-by: Claude:claude-sonnet-4-6
Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
@ChristophWurst
ChristophWurst force-pushed the ci/update-mariadb-11.8 branch from 54a425b to bd5250e Compare June 9, 2026 08:20
@coderabbitai

coderabbitai Bot commented Jun 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request updates the MariaDB container image version in the GitHub Actions integration test workflow. The mariadb-service image used by the integration-test job is bumped from version 11.4 to 11.8. No workflow logic or test steps are modified; only the container image tag is changed.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately and concisely summarizes the main change: updating MariaDB container image from version 11.4 to 11.8 in CI configuration.
Description check ✅ Passed The description is directly related to the changeset, providing context that version 11.8 is recommended per Nextcloud documentation and noting AI assistance.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e77a8e63-961a-467b-b9bc-6a9adc922e3c

📥 Commits

Reviewing files that changed from the base of the PR and between bcaa60c and bd5250e.

📒 Files selected for processing (1)
  • .github/workflows/test.yml

- 4190:4190
mariadb-service:
image: ghcr.io/nextcloud/continuous-integration-mariadb-11.4:latest
image: ghcr.io/nextcloud/continuous-integration-mariadb-11.8:latest

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Verify the MariaDB 11.8 container image exists in the GitHub Container Registry

# Check if the image exists and can be pulled
echo "Checking if MariaDB 11.8 image exists..."
docker manifest inspect ghcr.io/nextcloud/continuous-integration-mariadb-11.8:latest > /dev/null 2>&1
if [ $? -eq 0 ]; then
    echo "✓ Image exists and is accessible"
    docker manifest inspect ghcr.io/nextcloud/continuous-integration-mariadb-11.8:latest | jq -r '.manifests[0].platform'
else
    echo "✗ Image not found or not accessible"
    exit 1
fi

Repository: nextcloud/mail

Length of output: 135


🌐 Web query:

MariaDB 11.8 compatibility issues with Nextcloud stable32 stable33 stable34

💡 Result:

As of June 9, 2026, MariaDB 11.8 is officially listed as a recommended database version in the current Nextcloud documentation [1][2]. Nextcloud support for MariaDB 11.8 has been integrated into the platform, and the version is included in the latest system requirements [1][2]. While earlier iterations of Nextcloud (such as those preceding official 11.8 support) may have generated warnings when detecting MariaDB 11.8—due to factors like new reserved keywords (e.g., "vector") causing issues with specific apps like Recognize [3][4][5]—these have been addressed in the main server codebase [3]. Key points regarding compatibility: - Version Status: MariaDB 11.8 is now a recognized and recommended LTS version for use with current Nextcloud stable releases [1][2]. - Warnings: Users running older Nextcloud versions or instances that have not been fully updated may encounter warnings suggesting MariaDB versions up to 11.4 [4][5]. These warnings were precautionary and largely related to the validation of untested configurations [5]. - App Compatibility: While the core Nextcloud server is compatible, ensure that all installed third-party apps are up to date, as some older app versions may have encountered issues with new MariaDB reserved keywords introduced in 11.8 [3][4]. For the best experience with stable32, stable33, and stable34, ensure you are running the latest point releases of your specific Nextcloud version, as support and compatibility fixes are typically backported to these stable branches [3][5]. Always refer to the specific System Requirements page in your installed instance's Administration Manual for the most accurate information relative to your exact version [6].

Citations:


Fix GHCR MariaDB 11.8 image pullability before merging

  • .github/workflows/test.yml points to ghcr.io/nextcloud/continuous-integration-mariadb-11.8:latest, but docker manifest inspect fails (“not found or not accessible”), so the CI mariadb-service is likely to break when the workflow tries to pull it.
  • Nextcloud’s system requirements indicate MariaDB 11.8 is supported/recommended for the relevant stable lines, so compatibility should be OK once the correct/pullable GHCR image tag is used.

@kesselb
kesselb disabled auto-merge June 9, 2026 10:28
@kesselb
kesselb merged commit 93a5aa7 into main Jun 9, 2026
59 of 63 checks passed
@kesselb
kesselb deleted the ci/update-mariadb-11.8 branch June 9, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants