Project

Profile

Help

Task #5129

closed

drop MariaDB support

Added by dkliban@redhat.com over 5 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
No
Tags:
Sprint:
Sprint 59
Quarter:

Description

The reasoning for dropping MariaDB support was summarized on pulp-dev list[0].

[0] https://www.redhat.com/archives/pulp-dev/2019-July/msg00035.html


Related issues

Blocks Pulp - Task #5130: Remove maria/mysql from installerCLOSED - CURRENTRELEASECodeHeeler

Actions

Added by dkliban@redhat.com over 5 years ago

Revision 75abd0fb | View on GitHub

Problem: testing against mariadb is not needed

Solution: remove mariadb option from plugin-template

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

Actions #1

Updated by daviddavis over 5 years ago

  • Groomed changed from No to Yes
  • Sprint set to Sprint 56
Actions #2

Updated by amacdona@redhat.com over 5 years ago

  • Blocks Task #5130: Remove maria/mysql from installer added
Actions #3

Updated by dalley over 5 years ago

In Pulpcore and Plugins, we should change CharField back to TextField where applicable. The only reason we used CharField was because MySQL cannot have an index on an unbounded text field.

In PostgreSQL this is not a limitation. There's no reason to use a 'varchar' type instead of a 'text' type unless you want a limit on the length of the text. They have identical performance (except for the overhead of the length check on 'varchar')

We should decide whether having a length limit on the field provides any value and remove it if not.

I don't see a compelling reason to switch back to autoincrement PKs despite the fact that we only moved away from them to begin with due to MySQL support.

Actions #4

Updated by daviddavis over 5 years ago

It was also discussed that we should create a blog post for why we're dropping support. I think this is a good idea so I'm adding it to the checklist.

Added by dkliban@redhat.com over 5 years ago

Revision d3804e8f | View on GitHub

Problem: mariadb no long supported by pulp

Solution: stop testing with MariaDB on Travis

This patch also brings in the latest changes from plugin_template. This includes a template_config.yml in the root of the repository.

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

Actions #6

Updated by rchan over 5 years ago

  • Sprint changed from Sprint 56 to Sprint 57
Actions #7

Updated by rchan about 5 years ago

  • Sprint changed from Sprint 57 to Sprint 58
Actions #8

Updated by daviddavis about 5 years ago

  • Sprint/Milestone changed from 3.0.0 to 71
Actions #9

Updated by bmbouter about 5 years ago

  • Sprint/Milestone changed from 71 to 3.0.0
Actions #10

Updated by rchan about 5 years ago

  • Sprint changed from Sprint 58 to Sprint 59
Actions #11

Updated by daviddavis about 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis
Actions #12

Updated by daviddavis about 5 years ago

Added by daviddavis about 5 years ago

Revision 68ac3216 | View on GitHub

Drop reference to mysql/mariadb

ref #5129

Added by daviddavis about 5 years ago

Revision 68ac3216 | View on GitHub

Drop reference to mysql/mariadb

ref #5129

Actions #14

Updated by daviddavis about 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #15

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF