Project

Profile

Help

Issue #4544

Relative paths in pulp are limited to 255 characters

Added by daviddavis 7 months ago. Updated 8 days ago.

Status:
MODIFIED
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:
Sprint 60

Description

The problem is that we use relative paths as part of indexes (see FileContent) and in mariadb/mysql cannot index text fields. Users might desire relative paths that are longer than 255 chars. We should think of a solution or at the very least document this limitation.

Associated revisions

Revision 5502688f View on GitHub
Added by Fabricio Aguiar 9 days ago

Change relative_path from CharField to TextField

ref #4544
https://pulp.plan.io/issues/4544
related to https://github.com/pulp/pulpcore/pull/336

Revision ef8fc136 View on GitHub
Added by Fabricio Aguiar 8 days ago

Change relative_path from CharField to TextField

closes #4544
https://pulp.plan.io/issues/4544

Revision b7e668b4 View on GitHub
Added by Fabricio Aguiar 8 days ago

Change relative_path from CharField to TextField

ref #4544
https://pulp.plan.io/issues/4544
related to https://github.com/pulp/pulpcore/pull/336

History

#1 Updated by CodeHeeler 7 months ago

  • Triaged changed from No to Yes

#2 Updated by bmbouter 6 months ago

  • Tags deleted (Pulp 3)

#3 Updated by daviddavis about 1 month ago

  • Sprint/Milestone deleted (3.0)

#4 Updated by daviddavis about 1 month ago

  • Sprint/Milestone set to 71

#5 Updated by bmbouter about 1 month ago

  • Sprint/Milestone changed from 71 to 3.0

#6 Updated by bmbouter 13 days ago

How long would we set them to?

#7 Updated by daviddavis 13 days ago

How about making them TextFields? They were TextFields before we started supporting mariadb/mysql (mariadb/mysql cannot index text fields).

#8 Updated by bmbouter 13 days ago

daviddavis wrote:

How about making them TextFields? They were TextFields before we started supporting mariadb/mysql (mariadb/mysql cannot index text fields).

+1 to TextField

#9 Updated by bmbouter 13 days ago

  • Sprint set to Sprint 60

#10 Updated by fabricio.aguiar 12 days ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to fabricio.aguiar

#12 Updated by Anonymous 8 days ago

  • Status changed from POST to MODIFIED

Please register to edit this issue

Also available in: Atom PDF