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

Update Readme - drop reference, update links, fix formatting #210

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

rwaffen
Copy link
Member

@rwaffen rwaffen commented Feb 23, 2024

  • drop reference part and link to reference.md
  • update format
  • Update ci link

@rwaffen rwaffen changed the title Update Readme: Update Readme Feb 23, 2024
@rwaffen rwaffen changed the title Update Readme Update Readme - drop reference, update links, fix formatting Feb 23, 2024
Copy link
Contributor

@zilchms zilchms left a comment

Choose a reason for hiding this comment

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

LGTM, OL7 changes seem unrelated. wo got those quite often on this module without actual changes :/

@rwaffen rwaffen merged commit e4d63c8 into voxpupuli:master Feb 23, 2024
6 of 7 checks passed
@rwaffen rwaffen deleted the update_readme branch February 23, 2024 10:58
@alexjfisher
Copy link
Member

I wonder if the test failures are because the service takes a long time to start due to a lack of entropy in the container environment when generating keys???

No SSL keys/certificates found in datadir (/var/lib/proxysql). Generating new keys/certificates

@zilchms
Copy link
Contributor

zilchms commented Feb 23, 2024

Might be, at the same time the OL7 stuff dies by non-root PID. Definitly something that should be investigated, but I am missing the experience/knowledge to do that

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.

3 participants