Bug 1114771

Summary:	unable to upload docker tar file when using latest docker package.
Product:	[Retired] Pulp	Reporter:	Chris Duryee <cduryee>
Component:	docker-support	Assignee:	Michael Hrivnak <mhrivnak>
Status:	CLOSED UPSTREAM	QA Contact:	pulp-qe-list
Severity:	unspecified	Docs Contact:
Priority:	medium
Version:	Master	CC:	mhrivnak
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-28 22:11:42 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Chris Duryee 2014-06-30 23:23:06 UTC

Description of problem: When docker-io-1.0.0-3 is installed, I'm unable to upload saved tar files.

# docker pull busybox
# docker save busybox > busybox.tar
# pulp-admin docker repo uploads upload --repo-id=busybox -f busybox.tar


Version-Release number of selected component (if applicable): latest master (hash f3146f5)


How reproducible: every time


Actual results:

Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR: Error from the importer while importing uploaded unit to repository [busybox]
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR: Traceback (most recent call last):
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR:   File "/usr/lib/python2.6/site-packages/pulp/server/managers/content/upload.py", line 223, in import_uploaded_unit
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR:     unit_metadata, file_path, conduit, call_config)
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR:   File "/usr/lib/python2.6/site-packages/pulp_docker/plugins/importers/importer.py", line 81, in upload_unit
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR:     metadata = tarutils.get_metadata(file_path)
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR:   File "/usr/lib/python2.6/site-packages/pulp_docker/common/tarutils.py", line 34, in get_metadata
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR:     'size': image_data['Size']
Jun 30 18:20:05 pulp24-clonebase pulp: pulp.server.managers.content.upload:ERROR: KeyError: 'id'

Comment 1 Michael Hrivnak 2014-07-01 15:13:40 UTC

It looks like docker is now generating metadata with those attribute names capitalized.

Comment 2 Michael Hrivnak 2014-07-01 15:33:05 UTC

https://github.com/pulp/pulp_docker/pull/27

Comment 3 Chris Duryee 2014-09-30 00:06:46 UTC

This has been in POST but looks to be merged. OK to change to MODIFIED?

Comment 4 Brian Bouterse 2015-02-28 22:11:42 UTC

Moved to https://pulp.plan.io/issues/462