Task #4270: Add support for MariaDB and MySQL - Pulp

Actions

Send by e-mail Copy link

Task #4270

closed

Add support for MariaDB and MySQL

Added by bmbouter over 5 years ago. Updated over 4 years ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

Category:

-

Sprint/Milestone:

Start date:

Due date:

% Done:

100%

Estimated time:

Platform Release:

Groomed:

No

Sprint Candidate:

No

Tags:

Sprint:

Sprint 50

Quarter:

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

Actions

#1

Updated by bmbouter over 5 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.

Actions

#2

Updated by daviddavis over 5 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.

Actions

#3

Updated by bmbouter over 5 years ago

MariaDB sounds fine to me.

Actions

#4

Updated by bmbouter over 5 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

Actions

#5

Updated by daviddavis over 5 years ago

Blocked by Issue #4288: _mysql_exceptions.OperationalError: (1170, "BLOB/TEXT column 'name' used in key specification without a key length") added

Actions

#6

Updated by daviddavis over 5 years ago

Status changed from NEW to POST

https://github.com/pulp/pulpcore/pull/21

Actions

#7

Updated by bmbouter over 5 years ago

Sprint set to Sprint 47

Actions

#8

Updated by rchan over 5 years ago

In the future, please bring up the question of adding something to the sprint if work starts on it.

Actions

#9

Updated by rchan about 5 years ago

Sprint changed from Sprint 47 to Sprint 48

Actions

#10

Updated by ipanova@redhat.com about 5 years ago

Assignee set to daviddavis

Actions

#11

Updated by rchan about 5 years ago

Sprint changed from Sprint 48 to Sprint 49

Actions

#12

Updated by daviddavis about 5 years ago

Subject changed from Enable MariaDB testing in Travis for pulp/pulp and pulp/pulpcore-plugin to Add support for MariaDB and MySQL

Actions

#13

Updated by daviddavis about 5 years ago

Blocked by Task #4290: Investigate performance of Pulp with UUID PKs added

Actions

#14

Updated by daviddavis about 5 years ago

Blocks Task #4507: Remove job_id field on Tasks added

Added by dalley about 5 years ago

Revision 55b43da4 | View on GitHub

Use UUID primary keys

Required PR: https://github.com/pulp/pulpcore-plugin/pull/69

re: #4270 https://pulp.plan.io/issues/4270

Actions

#15

Updated by daviddavis about 5 years ago

Related to Story #4517: As a user, I can install pulp with mysql or mariadb added

Added by daviddavis about 5 years ago

Revision 1be95b7d | View on GitHub

Adding support for mariadb/mysql

fixes #4270 https://pulp.plan.io/issues/4270

Actions

#16

Updated by daviddavis about 5 years ago

Status changed from POST to MODIFIED
% Done changed from 0 to 100

Applied in changeset pulpcore|1be95b7d8a454ae7066b597bfb275a5f79357a34.

Actions

#17

Updated by daviddavis about 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 about 5 years ago

Revision 39c8147d | View on GitHub

Install pulpcore with extra postgres packages

Added by David Davis about 5 years ago

Revision 225e1c41 | View on GitHub

Install pulpcore with postgres extra packages

Added by David Davis about 5 years ago

Revision 5ea25c9d | View on GitHub

Install postgres extra packages

Added by David Davis about 5 years ago

Revision 62a2628f | View on GitHub

Update instructions to install db extras

Actions

#18

Updated by rchan about 5 years ago

Sprint changed from Sprint 49 to Sprint 50

Added by daviddavis about 5 years ago

Revision 246abd41 | View on GitHub

Install the postgresql extra packages in Travis

Added by daviddavis about 5 years ago

Revision 85a6f969 | View on GitHub

Install the psycopg package if using postgres

Added by daviddavis about 5 years ago

Revision 85a6f969 | View on GitHub

Install the psycopg package if using postgres

Actions

#19

Updated by daviddavis about 5 years ago

Status changed from POST to MODIFIED

Added by David Davis about 5 years ago

Revision 1a7a1e5a | View on GitHub

Install pulpcore with postgresql extra packages

Added by David Davis about 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 about 5 years ago

Revision 1a7a1e5a | View on GitHub

Install pulpcore with postgresql extra packages

Added by David Davis about 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 about 5 years ago

Revision 1a7a1e5a | View on GitHub

Install pulpcore with postgresql extra packages

Added by David Davis about 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 about 5 years ago

Revision 1a7a1e5a | View on GitHub

Install pulpcore with postgresql extra packages

Added by David Davis about 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 about 5 years ago

Revision c31d1db6 | View on GitHub

Install the postgresql extra packages in Travis

Actions

#20

Updated by daviddavis about 5 years ago

Sprint/Milestone set to 3.0.0

Actions

#21

Updated by bmbouter almost 5 years ago

Tags deleted (~~Pulp 3, Pulp 3 RC Blocker~~)

Added by daviddavis almost 5 years ago

Revision e6832c74 | View on GitHub

Removing file publisher stuff

refs #4270 https://pulp.plan.io/issues/4720

Actions

#22

Updated by bmbouter over 4 years ago

Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Actions

Send by e-mail Copy link

Also available in: Atom PDF