Project

Profile

Help

Story #4827

closed

Add artifact sha details to the content API

Added by sajha over 5 years ago. Updated about 5 years ago.

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

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Easy Fix, Katello-P1
Sprint:
Sprint 53
Quarter:

Description

It would be helpful to see artifact sha details on the content details/list results to avoid an additional call to artifacts API to fetch sha details.


Related issues

Related to File Support - Test #5274: Test - Read artifact sha details from the content APICLOSED - DUPLICATEActions
Actions #1

Updated by daviddavis over 5 years ago

  • Tags Katello-P1 added

Talked to Katello and they said this is a P1.

Actions #2

Updated by amacdona@redhat.com over 5 years ago

  • Tracker changed from Issue to Story
  • % Done set to 0
  • Sprint set to Sprint 53
Actions #3

Updated by daviddavis over 5 years ago

  • Tags Easy Fix added
Actions #4

Updated by daviddavis over 5 years ago

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

Updated by daviddavis over 5 years ago

Should the checksums just be fields on the serializer or should I do something like a checksum dictionary mapping checksum type to its value:

"checksums":  {
   "sha256": "abcdef..."
    ...
}
Actions #6

Updated by bmbouter over 5 years ago

+1 to the "checksum" key which contains keys of the algorithm name (as you gave an example of)

Actions #7

Updated by daviddavis over 5 years ago

After talking it over on IRC, we decided to just include the checksum fields on the file. For example:

        {
            "_artifact": "/pulp/api/v3/artifacts/f523664e-d291-427b-b260-198598c42ae0/",
            "_created": "2019-05-22T21:09:31.343339Z",
            "_href": "/pulp/api/v3/content/file/files/2b347487-c5f9-48bd-9a15-aaa79fae851a/",
            "_type": "file.file",
            "md5": "f4e4ee897387d9061e5d6a17d3991ff3",
            "relative_path": "test.iso",
            "sha1": "08e282883a0f20b24414e9e5e1f21aedba008ff2",
            "sha224": "f80f493561e579352ce24beeb9a2c71d1f1a8f7525782c9ba8e1f244",
            "sha256": "582f1729b0c15eacbb7d02a553c0a96a39f817135c15ac4190aef96e45c4b65d",
            "sha384": "ad3e4cb414ee360477c54d0db3369497fb2b8858f6328c792112f54327b394716c2e5a86caa3989960e4f27ec274a49f",
            "sha512": "4f3ca80d512fa86af762a1883a31b6a47da825368a42bfcadeb824e849f76202ebe9529e5be1a911ec4a92b5da3f4eecce95cd6a5d2b5cda3b5dd
ccf5aadf5c4"
        }

Added by daviddavis over 5 years ago

Revision 9728deec | View on GitHub

Adding checksum serializer for content

ref #4827 https://pulp.plan.io/issues/4827

Actions #9

Updated by daviddavis over 5 years ago

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

Updated by kersom over 5 years ago

  • Related to Test #5274: Test - Read artifact sha details from the content API added
Actions #11

Updated by bmbouter about 5 years ago

  • Sprint/Milestone set to 0.1.0
Actions #12

Updated by bmbouter about 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF