Task #4270
closedAdd support for MariaDB and MySQL
100%
Description
Given the recent discovery that we are benefitting from PostgreSQL-only Django features, we need to test against another SQL database to ensure we are compatible.
Related issues
Updated by bmbouter almost 6 years ago
I think the best thing would be to do this with each PR's run in Travis. Another option is to do it in the nightly Travis run, but by then the PR author is long gone. Since it would run in parallel with PostgreSQL I don't think it will add significant runtime to the PR runner.
Updated by daviddavis almost 6 years ago
I am +1 to this. This might be my personal bias against Oracle but what about using MariaDB instead of MySQL?
Also, I suggest we keep the number of jobs to < 5. Travis limits the number of concurrent builds per project to about 4-5 (depending on their overall load) and having 5+ jobs per build makes builds take a long time.
Updated by bmbouter almost 6 years ago
- Subject changed from Enable MySQL testing in Travis for pulp/pulp and pulp/pulpcore-plugin to Enable MariaDB testing in Travis for pulp/pulp and pulp/pulpcore-plugin
Travis has support for this: https://docs.travis-ci.com/user/database-setup/#mariadb
Updated by daviddavis almost 6 years ago
- Blocked by Issue #4288: _mysql_exceptions.OperationalError: (1170, "BLOB/TEXT column 'name' used in key specification without a key length") added
Updated by daviddavis almost 6 years ago
- Status changed from NEW to POST
Updated by rchan almost 6 years ago
In the future, please bring up the question of adding something to the sprint if work starts on it.
Updated by daviddavis almost 6 years ago
- Subject changed from Enable MariaDB testing in Travis for pulp/pulp and pulp/pulpcore-plugin to Add support for MariaDB and MySQL
Updated by daviddavis almost 6 years ago
- Blocked by Task #4290: Investigate performance of Pulp with UUID PKs added
Updated by daviddavis over 5 years ago
- Blocks Task #4507: Remove job_id field on Tasks added
Added by dalley over 5 years ago
Updated by daviddavis over 5 years ago
- Related to Story #4517: As a user, I can install pulp with mysql or mariadb added
Added by daviddavis over 5 years ago
Revision 1be95b7d | View on GitHub
Adding support for mariadb/mysql
Updated by daviddavis over 5 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulpcore|1be95b7d8a454ae7066b597bfb275a5f79357a34.
Updated by daviddavis over 5 years ago
- Status changed from MODIFIED to POST
Reopening issue. I need to update the docs and notify plugins of this change.
Added by David Davis over 5 years ago
Revision 39c8147d | View on GitHub
Install pulpcore with extra postgres packages
ref #4270
Added by David Davis over 5 years ago
Revision 225e1c41 | View on GitHub
Install pulpcore with postgres extra packages
ref #4270
Added by David Davis over 5 years ago
Revision 5ea25c9d | View on GitHub
Install postgres extra packages
ref #4270
Added by David Davis over 5 years ago
Revision 62a2628f | View on GitHub
Update instructions to install db extras
ref #4270
Added by daviddavis over 5 years ago
Revision 246abd41 | View on GitHub
Install the postgresql extra packages in Travis
ref #4270
Added by daviddavis over 5 years ago
Revision 85a6f969 | View on GitHub
Install the psycopg package if using postgres
ref #4270
Added by daviddavis over 5 years ago
Revision 85a6f969 | View on GitHub
Install the psycopg package if using postgres
ref #4270
Added by David Davis over 5 years ago
Revision 1a7a1e5a | View on GitHub
Install pulpcore with postgresql extra packages
ref #4270
Added by David Davis over 5 years ago
Revision d32db5b4 | View on GitHub
Fixing conditions check pk for None
With UUIDs, pks are never None. Instead check _state.
refs #4270
Added by David Davis over 5 years ago
Revision 1a7a1e5a | View on GitHub
Install pulpcore with postgresql extra packages
ref #4270
Added by David Davis over 5 years ago
Revision d32db5b4 | View on GitHub
Fixing conditions check pk for None
With UUIDs, pks are never None. Instead check _state.
refs #4270
Added by David Davis over 5 years ago
Revision 1a7a1e5a | View on GitHub
Install pulpcore with postgresql extra packages
ref #4270
Added by David Davis over 5 years ago
Revision d32db5b4 | View on GitHub
Fixing conditions check pk for None
With UUIDs, pks are never None. Instead check _state.
refs #4270
Added by David Davis over 5 years ago
Revision 1a7a1e5a | View on GitHub
Install pulpcore with postgresql extra packages
ref #4270
Added by David Davis over 5 years ago
Revision d32db5b4 | View on GitHub
Fixing conditions check pk for None
With UUIDs, pks are never None. Instead check _state.
refs #4270
Added by mdellweg over 5 years ago
Revision c31d1db6 | View on GitHub
Install the postgresql extra packages in Travis
ref #4270
Added by daviddavis over 5 years ago
Revision e6832c74 | View on GitHub
Removing file publisher stuff
Updated by bmbouter almost 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Use UUID primary keys
Required PR: https://github.com/pulp/pulpcore-plugin/pull/69
re: #4270 https://pulp.plan.io/issues/4270