Skip to content

test: enable filecache partitioning for one mysql test - #62491

Merged
provokateurin merged 1 commit into
masterfrom
partitioning-ci
Oct 5, 2026
Merged

provokateurin merged 1 commit into
masterfrom
partitioning-ci

Conversation

@icewind1991

@icewind1991 icewind1991 commented Jul 24, 2026 •

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

Test db level filecache partitioning in CI.

Instead of adding yet-another CI job, this enabled partitioning in one of the existing mysql jobs

Checklist

AI (if applicable)

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

@icewind1991 icewind1991 added this to the Nextcloud 35 milestone Jul 24, 2026
@icewind1991
icewind1991 requested a review from a team as a code owner July 24, 2026 18:39
@icewind1991
icewind1991 removed the request for review from a team July 24, 2026 18:39
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Jul 24, 2026
@icewind1991
icewind1991 marked this pull request as draft July 24, 2026 20:12
@icewind1991
icewind1991 force-pushed the partitioning-ci branch 6 times, most recently from fb55457 to 3461df7 Compare July 28, 2026 11:56
@blizzz blizzz mentioned this pull request Aug 27, 2026
@icewind1991
icewind1991 force-pushed the partitioning-ci branch 4 times, most recently from c216329 to ae714da Compare September 10, 2026 17:15
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 changed the title test: add ci test for mariadb with filecache partitioning test: enable filecache partitioning for one mysql test Oct 2, 2026
@icewind1991
icewind1991 marked this pull request as ready for review October 2, 2026 18:01
Comment on lines +94 to +95
echo "alter table oc_filecache drop primary key, add primary key (fileid, storage);" | mysql -h 127.0.0.1 -P 4444 -u root -prootpassword nextcloud
echo "alter table oc_filecache partition by hash(storage) partitions 5;" | mysql -h 127.0.0.1 -P 4444 -u root -prootpassword nextcloud

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So there is no built-in way for Nextcloud to enable partitioning? This means admins always have to do it manually like this?

@provokateurin
provokateurin merged commit 2cf165d into master Oct 5, 2026
166 of 168 checks passed
@provokateurin
provokateurin deleted the partitioning-ci branch October 5, 2026 10:59
@provokateurin

Copy link
Copy Markdown
Member

Backport to stable35?

@susnux

susnux commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

/backport to stable35

@susnux

susnux commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

/backport to stable34

@susnux

susnux commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

/backport to stable33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants