Project

Profile

Help

Refactor #4567

Convert data between createrepo_c format and pulp format in a unified way

Added by ttereshc over 2 years ago. Updated almost 2 years ago.

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

100%

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

Description

There are few issues which inspired that refactor task:
- null value can come from creatrepo_c while in TextField it's recommended to use empty string in such cases https://pulp.plan.io/issues/4502
- more complicated structures which Pulp stores in a TextField requires serialization/deserialization (or maybe remodelling to make them simpler) https://github.com/pulp/pulp_rpm/pull/1298#discussion_r265235083
- there are two ways to get content into Pulp: via sync and via upload. And RPM plugin processes data differently in those cases, e.g. at upload time there is some data JSON serialization is happening, no serialization happens at sync time.

In general, if it's possible to isolate createrepo_c specifics (not only data conversion), code would be much better organized and readable.

Associated revisions

Revision e871d21a View on GitHub
Added by ppicka over 2 years ago

Refactor naming for pakcage attr

to better readability code package method to convert data to createrepo_c using PULP namespace.

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

Signed-off-by: Pavel Picka

History

#1 Updated by ttereshc over 2 years ago

  • Description updated (diff)

#2 Updated by dalley over 2 years ago

  • Groomed changed from No to Yes

#3 Updated by ttereshc over 2 years ago

  • Sprint set to Sprint 51

#4 Updated by daviddavis over 2 years ago

  • Sprint changed from Sprint 51 to Sprint 52

#5 Updated by bmbouter over 2 years ago

  • Tags deleted (Pulp 3)

#6 Updated by rchan over 2 years ago

  • Sprint changed from Sprint 52 to Sprint 53

#7 Updated by ppicka over 2 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ppicka

#8 Updated by amacdona@redhat.com over 2 years ago

  • Sprint changed from Sprint 53 to Sprint 54

#9 Updated by Anonymous over 2 years ago

  • File clipboard-201906201006-ofmgl.png added

#10 Updated by Anonymous over 2 years ago

  • File clipboard-201906201006-fi9lh.png added
  • File clipboard-201906201006-9jece.png added

#11 Updated by ppicka over 2 years ago

  • File deleted (clipboard-201906201006-ofmgl.png)

#12 Updated by ppicka over 2 years ago

  • File deleted (clipboard-201906201006-fi9lh.png)

#13 Updated by ppicka over 2 years ago

  • File deleted (clipboard-201906201006-9jece.png)

#14 Updated by ttereshc over 2 years ago

  • Sprint changed from Sprint 54 to Sprint 55

#15 Updated by ppicka over 2 years ago

  • Status changed from ASSIGNED to POST

#16 Updated by ppicka over 2 years ago

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

#17 Updated by ttereshc almost 2 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF