Skip to content

feat: make PXC backup job resources configurable - #953

Open
Harishrs2006 wants to merge 1 commit into
openeverest:mainfrom
Harishrs2006:fix/backup-job-configurable-resources
Open

feat: make PXC backup job resources configurable#953
Harishrs2006 wants to merge 1 commit into
openeverest:mainfrom
Harishrs2006:fix/backup-job-configurable-resources

Conversation

@Harishrs2006

@Harishrs2006 Harishrs2006 commented May 8, 2026

Copy link
Copy Markdown

Fixes: #1905

Large (1TB+) PXC backups fail after ~5 hours because the xtrabackup
container has a hardcoded 600m CPU limit that cannot be overridden by users.

Related pull requests
Will be linked from openeverest/openeverest once that PR is opened.

Cause:
genPXCStorageSpec() in applier.go hardcodes 600m CPU / 1G memory for
every BackupStorageSpec.Resources with no way to override it from user input.

Solution:
Added BackupJobResources *Resources field to DatabaseClusterBackupSpec.
When set, the values override the defaults in getStoragesSpec() before
the BackupStorageSpec is passed to the PXC operator.

Adds BackupJobResources to DatabaseClusterBackupSpec so users can
override the hardcoded 600m CPU / 1G memory limits that cause large
backups to fail.

Fixes: openeverest/openeverest#1905
Signed-off-by: Harish R S <harishrs21082006@gmail.com>
@Harishrs2006
Harishrs2006 requested a review from a team as a code owner May 8, 2026 18:36
@atharvamhaske

Copy link
Copy Markdown

again ai slop

@atharvamhaske

Copy link
Copy Markdown

Fixes: #1905

Large (1TB+) PXC backups fail after ~5 hours because the xtrabackup container has a hardcoded 600m CPU limit that cannot be overridden by users.

Related pull requests Will be linked from openeverest/openeverest once that PR is opened.

  • [link]

Cause: genPXCStorageSpec() in applier.go hardcodes 600m CPU / 1G memory for every BackupStorageSpec.Resources with no way to override it from user input.

Solution: Added BackupJobResources *Resources field to DatabaseClusterBackupSpec. When set, the values override the defaults in getStoragesSpec() before the BackupStorageSpec is passed to the PXC operator.

where is your link bro? are you even checking before raising issues or PR?

@Harishrs2006

Copy link
Copy Markdown
Author

@atharvamhaske
sorry, that was an unfilled placeholder i missed.
i have removed it from the description.

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