Project

Profile

Help

Task #4270

Add support for MariaDB and MySQL

Added by bmbouter about 2 years ago. Updated about 1 year 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

Related to Pulp - Story #4517: As a user, I can install pulp with mysql or mariadbCLOSED - WONTFIX

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Blocked by Pulp - Issue #4288: _mysql_exceptions.OperationalError: (1170, "BLOB/TEXT column 'name' used in key specification without a key length")CLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Blocked by Pulp - Task #4290: Investigate performance of Pulp with UUID PKsCLOSED - COMPLETE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Blocks Pulp - Task #4507: Remove job_id field on TasksCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 39c8147d View on GitHub
Added by David Davis almost 2 years ago

Install pulpcore with extra postgres packages

ref #4270

Revision 225e1c41 View on GitHub
Added by David Davis almost 2 years ago

Install pulpcore with postgres extra packages

ref #4270

Revision 5ea25c9d View on GitHub
Added by David Davis almost 2 years ago

Install postgres extra packages

ref #4270

Revision 62a2628f View on GitHub
Added by David Davis almost 2 years ago

Update instructions to install db extras

ref #4270

Revision 246abd41 View on GitHub
Added by daviddavis almost 2 years ago

Install the postgresql extra packages in Travis

ref #4270

Revision 85a6f969 View on GitHub
Added by daviddavis almost 2 years ago

Install the psycopg package if using postgres

ref #4270

Revision 85a6f969 View on GitHub
Added by daviddavis almost 2 years ago

Install the psycopg package if using postgres

ref #4270

Revision 1a7a1e5a View on GitHub
Added by David Davis almost 2 years ago

Install pulpcore with postgresql extra packages

ref #4270

Revision d32db5b4 View on GitHub
Added by David Davis almost 2 years ago

Fixing conditions check pk for None

With UUIDs, pks are never None. Instead check _state.

refs #4270

Revision 1a7a1e5a View on GitHub
Added by David Davis almost 2 years ago

Install pulpcore with postgresql extra packages

ref #4270

Revision d32db5b4 View on GitHub
Added by David Davis almost 2 years ago

Fixing conditions check pk for None

With UUIDs, pks are never None. Instead check _state.

refs #4270

Revision 1a7a1e5a View on GitHub
Added by David Davis almost 2 years ago

Install pulpcore with postgresql extra packages

ref #4270

Revision d32db5b4 View on GitHub
Added by David Davis almost 2 years ago

Fixing conditions check pk for None

With UUIDs, pks are never None. Instead check _state.

refs #4270

Revision 1a7a1e5a View on GitHub
Added by David Davis almost 2 years ago

Install pulpcore with postgresql extra packages

ref #4270

Revision d32db5b4 View on GitHub
Added by David Davis almost 2 years ago

Fixing conditions check pk for None

With UUIDs, pks are never None. Instead check _state.

refs #4270

Revision c31d1db6 View on GitHub
Added by mdellweg almost 2 years ago

Install the postgresql extra packages in Travis

ref #4270

History

#1 Updated by bmbouter about 2 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.

#2 Updated by daviddavis about 2 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.

#3 Updated by bmbouter about 2 years ago

MariaDB sounds fine to me.

#4 Updated by bmbouter about 2 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

#5 Updated by daviddavis about 2 years ago

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

#6 Updated by daviddavis about 2 years ago

  • Status changed from NEW to POST

#7 Updated by bmbouter about 2 years ago

  • Sprint set to Sprint 47

#8 Updated by rchan about 2 years ago

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

#9 Updated by rchan about 2 years ago

  • Sprint changed from Sprint 47 to Sprint 48

#10 Updated by ipanova@redhat.com almost 2 years ago

  • Assignee set to daviddavis

#11 Updated by rchan almost 2 years ago

  • Sprint changed from Sprint 48 to Sprint 49

#12 Updated by daviddavis almost 2 years ago

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

#13 Updated by daviddavis almost 2 years ago

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

#14 Updated by daviddavis almost 2 years ago

  • Blocks Task #4507: Remove job_id field on Tasks added

#15 Updated by daviddavis almost 2 years ago

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

#16 Updated by daviddavis almost 2 years ago

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

#17 Updated by daviddavis almost 2 years ago

  • Status changed from MODIFIED to POST

Reopening issue. I need to update the docs and notify plugins of this change.

#18 Updated by rchan almost 2 years ago

  • Sprint changed from Sprint 49 to Sprint 50

#19 Updated by daviddavis almost 2 years ago

  • Status changed from POST to MODIFIED

#20 Updated by daviddavis almost 2 years ago

  • Sprint/Milestone set to 3.0.0

#21 Updated by bmbouter almost 2 years ago

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

#22 Updated by bmbouter about 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF