Project

Profile

Help

Story #948

Author field in metadata.json not parsed

Added by bkearney@redhat.com over 4 years ago. Updated 9 months ago.

Status:
CLOSED - WONTFIX
Priority:
High
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Sprint:

Description

When a puppet module is uploaded into Pulp, the author of the module is not set to the "Author" as specified in metadata.json.

This is consistently reproducable

Steps to Reproduce:
1. Create a repo
2. Create module and ensure it has "Author" in metadata.json
3. Upload module to the repo
4. List the metadata for the module

Actual results: Module author is set to package name, not "Author"

Expected results: The author should be set based on the "Author" field in metadata.json.


Related issues

Blocks Puppet Support - Story #984: As a user, all metadata.json fields are stored as Pulp first-class metadata CLOSED - WONTFIX Actions

History

#1 Updated by bmbouter over 4 years ago

This needs to apply cleanly to a 2.6.x release of Pulp

#2 Updated by bmbouter over 4 years ago

  • Description updated (diff)

See some notes about doing similar commands from bug report #890.

#3 Updated by dkliban@redhat.com over 4 years ago

After some discussion, here is the proposed plan:

0. Write a migration that updates each puppet unit by opening the archive representing it and copying the content of metadata.json into the unit record in the database.
1. Update behavior of the puppet importer to save the contents of metadata.json as part of the unit record in the database.
2. Update the API to return the new metadata blob when requesting `/pulp/api/v2/repositories/<repo_id>/search/units/` for a puppet repository.
3. Add flag to `pulp-admin puppet repo modules --repo-id repo1` to display metadata.json blob when displaying details of module.
4. Add documentation for this to the pulp_puppet docs.
5. Add a release note for this change.

#4 Updated by bmbouter over 4 years ago

  • Related to Story #984: As a user, all metadata.json fields are stored as Pulp first-class metadata added

#5 Updated by bmbouter over 4 years ago

  • Related to deleted (Story #984: As a user, all metadata.json fields are stored as Pulp first-class metadata)

#6 Updated by bmbouter over 4 years ago

  • Blocks Story #984: As a user, all metadata.json fields are stored as Pulp first-class metadata added

#7 Updated by jortel@redhat.com over 4 years ago

  • Triaged changed from No to Yes

#8 Updated by jortel@redhat.com over 4 years ago

  • Platform Release set to 2.7.0

#9 Updated by dkliban@redhat.com over 4 years ago

  • Platform Release deleted (2.7.0)

#10 Updated by mhrivnak over 4 years ago

  • Platform Release set to 2.8.0

#11 Updated by dkliban@redhat.com about 4 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dkliban@redhat.com

#12 Updated by jortel@redhat.com almost 4 years ago

  • Tracker changed from Issue to Story
  • Priority changed from Normal to High
  • Platform Release deleted (2.8.0)
  • Groomed set to No
  • Sprint Candidate set to No

#13 Updated by dkliban@redhat.com almost 4 years ago

  • Status changed from ASSIGNED to NEW
  • Assignee deleted (dkliban@redhat.com)

#16 Updated by bmbouter 9 months ago

  • Status changed from NEW to CLOSED - WONTFIX

#17 Updated by bmbouter 9 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF