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

Support Debian Buster and MariaDB #3405

Open
wants to merge 15 commits into
base: ng
Choose a base branch
from
Open

Conversation

CaptainSifff
Copy link

Curerntly the debian scripts prevent a build on Debian Buster.
Among the notable changes are:

  • Debian Buster ships mariadb instead of mysql, hence some package names have changed
  • Python-3.7 is the default.

allow the use of mariadb
Debian Buster ships python-3.7 hence we need to fix the debian rules.
@coveralls
Copy link

coveralls commented Feb 7, 2019

Coverage Status

Coverage increased (+0.07%) to 84.549% when pulling 1b8a520 on CaptainSifff:ng into 398e470 on allegro:ng.

@romcheg
Copy link
Contributor

romcheg commented Feb 8, 2019

@CaptainSifff Ralph is not tested against python-3.7 and MariaDB and most likely the tests will fail. Could you please verify this?

@CaptainSifff
Copy link
Author

CaptainSifff commented Feb 8, 2019

Sure, I can check. does ralphctl expose some kind of testsuite?
I had to fiddle a bit with the package version in requirements.txt (rq >= 0.12 and django-rq >= 1.2.0)
But so far, I found that I can populate the database with the demo data and play with
that dataset through the browser without noticing any issues so far.

@CaptainSifff
Copy link
Author

I tested some of the ralphctl commands:
ralphctl demodata: seems to work - database is populated and looks like I think it is expected.
ralphctl sqlflush - lots of output, no error
ralphctl ldap_sync - fails, currently not built with ldap(comes next)
ralphctl push_graphs_to_statd - no output, no error
ralphctl check - System check identified no issues (0 silenced).
ralphctl compilemessages - no error
ralphctl createcachetable - no error, no output
ralphctl dbshell - Mariadb shell preset to the ralph_ng database
ralphctl shell - Python3.7 shell
ralphctl test - Creating test database for alias 'default'...
Got an error creating the test database: (1044, "Access denied for user 'ralph_ng'@'localhost' to database 'test_ralph_ng'")
Which is expected that In the spirit of the installation docs I only gave permission to acces the database ralph_ng
ralphctl rqstats - Connection refused, somehow expected since I didn't set up redis

Anything else that you are interested in?

@CaptainSifff
Copy link
Author

I also got ldap and openldap sufficiently working for my needs by following the steps from the doc:
https://ralph-ng.readthedocs.io/en/latest/installation/configuration/
I also needed the patch from #3080 for patching ldap.py and the documentation given in
#3077 for patching prod.py.

@romcheg
Copy link
Contributor

romcheg commented Feb 28, 2019

@CaptainSifff thank you for the research you have made. Before we start shipping the package for debian, we should enable automatic tests for 3.7 and mariadb. A spare pair of hands (you :) ) would be helpful for that atm. Let's discuss details in our Gitter channel.

Copy link
Contributor

@romcheg romcheg left a comment

Choose a reason for hiding this comment

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

Left a few comments inline. Could you also please change mysql to mysql/mariadb in the debconf? You may want to test that after introducing the changes -- to do that build the package using make build-snapshot-package command and then install the package somewhere.

requirements/base.txt Outdated Show resolved Hide resolved
requirements/base.txt Outdated Show resolved Hide resolved
@romcheg romcheg requested a review from szymi- March 1, 2019 11:21
szymi-
szymi- previously approved these changes Mar 1, 2019
@CaptainSifff
Copy link
Author

I also modified now the template as pointed out by @romcheg so that mariadb should be a possibility.

debian/ralph-core.templates Show resolved Hide resolved
@romcheg romcheg changed the title Debian Scripts prevent build on Debian Buster Support Debian Buster and MariaDB Mar 4, 2019
@vi4m
Copy link
Contributor

vi4m commented Mar 11, 2020

@romcheg Do we wait for something here? is it RTM?

@romcheg
Copy link
Contributor

romcheg commented Mar 11, 2020

@vi4m the changes do not allow to build the package for Buster within the existing build scripts, so we cannot merge this.

@CaptainSifff
Copy link
Author

Can somebody remind me what I would have to check...? The pipeline still goes through or did I miss a point?

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.

None yet

5 participants