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

Use proxysql from packetfence repo for proxysql container #8267

Merged
merged 12 commits into from
Sep 10, 2024
Merged

Conversation

JeGoi
Copy link
Contributor

@JeGoi JeGoi commented Aug 30, 2024

Description

Use official proxysql docker version

Impacts

Use official proxysql docker version

Issue

fixes #8180

Delete branch after merge

YES

NOTE

Only tested in CI

@JeGoi JeGoi requested a review from fdurand August 30, 2024 12:21
@JeGoi JeGoi added this to the +1 (patch release) milestone Aug 30, 2024
@JeGoi
Copy link
Contributor Author

JeGoi commented Sep 4, 2024

Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 10: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 11: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 12: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 13: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 16: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 17: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 20: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 21: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: /proxysql-read-only-handler.sh: line 22: jq: command not found
Sep 04 13:52:45 pf1 proxysql-docker-wrapper[2102127]: ERROR 2005 (HY000): Unknown server host '-P' (-2)

Need to add jq

@JeGoi JeGoi changed the title Use official proxysql docker version Use proxysql from packetfence repo for proxysql container Sep 6, 2024
@JeGoi
Copy link
Contributor Author

JeGoi commented Sep 6, 2024

Since official container does not support systemd.
We create deb file for proxysql and add it in our repo.

@fdurand fdurand merged commit 3e8d9ad into devel Sep 10, 2024
13 of 14 checks passed
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.

Remove build for proxysql container and use an existing one
2 participants