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

OS and Version update #184

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

OS and Version update #184

wants to merge 16 commits into from

Conversation

altima
Copy link

@altima altima commented Dec 21, 2021

Pull Request (PR) description

  • Added Debian Bullseye support
  • Added ProxySQL 2.1/2.2/2.3 support

This Pull Request (PR) fixes the following issues

@altima
Copy link
Author

altima commented Dec 21, 2021

Also tests are running without F

@smortex smortex changed the title OS and Vesrion update OS and Version update Dec 21, 2021
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I just merged #181 but this adds acceptance tests. Would you mind rebasing and addressing my comments?

Dockerfile Outdated Show resolved Hide resolved
manifests/install.pp Outdated Show resolved Hide resolved
manifests/params.pp Outdated Show resolved Hide resolved
@vox-pupuli-tasks
Copy link

Dear @altima, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@saz
Copy link
Sponsor Contributor

saz commented Jul 11, 2022

@altima Can you take a look at the comments, please?

@altima altima requested a review from ekohl July 12, 2022 20:12
@altima
Copy link
Author

altima commented Jul 12, 2022

Changes are done. Feel free to check again :)

@saz
Copy link
Sponsor Contributor

saz commented Jul 21, 2022

FYI: proxysql 2.4 is available and might be included in this PR

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.

4 participants