Skip to content

Update conditional logic xtrabackup.pp to properly... #1609

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update conditional logic xtrabackup.pp to properly... #1609

wants to merge 1 commit into from

Conversation

ndelic0
Copy link
Contributor

@ndelic0 ndelic0 commented Dec 8, 2023

Summary

Update conditional logic xtrabackup.pp to properly manage permissions of xtrabackup users for MariaDB 10.5.2

Additional Context

Check #1598

Related Issues (if any)

Mention any related issues or pull requests.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified. (For example puppet apply)

@ndelic0 ndelic0 requested review from alexjfisher, bastelfreak and a team as code owners December 8, 2023 12:06
@CLAassistant
Copy link

CLAassistant commented Dec 8, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

optiz0r
optiz0r approved these changes Jun 26, 2025
View reviewed changes
Copy link

@optiz0r optiz0r left a comment

Choose a reason for hiding this comment

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

I've been running this PR in production for >18 months. Forgot all about it, but had to come back and re-apply this to puppetlabs-mysql 16.2 after upgrading puppet, finding it still unmerged. Would be lovely if Perforce could merge and release this.

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

@alexjfisher alexjfisher Awaiting requested review from alexjfisher alexjfisher is a code owner

@bastelfreak bastelfreak Awaiting requested review from bastelfreak bastelfreak is a code owner

1 more reviewer

@optiz0r optiz0r optiz0r approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
community
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ndelic0 @CLAassistant @optiz0r @ia-content