Skip to content

Upload support#3

Merged
bowlofeggs merged 1 commit intopulp:masterfrom
bowlofeggs:upload
Oct 24, 2014
Merged

Upload support#3
bowlofeggs merged 1 commit intopulp:masterfrom
bowlofeggs:upload

Conversation

@bowlofeggs
Copy link

This pull request introduces upload support. It does not come with documentation, as that will be part of a later pull request. If you wish to test uploading an archive: find any Python package, and run it's setup.py with the sdist argument. It will output a tarball, and you should be able to upload that to Pulp. To verify that it's been uploaded, you should be able to see it in Mongo like this:

> db.units_python_package.find()

Also, you should find the package you uploaded stored in /var/lib/pulp/content/python_package/.

@bmbouter bmbouter self-assigned this Oct 22, 2014
@bmbouter
Copy link
Member

All tests pass for me on FC20

@bmbouter
Copy link
Member

The upload worked! Here is the metadata I see in the DB:

{
    "_id": "f0a9d909-a548-423d-94b9-b49e6bca475e",
    "_storage_path": "/var/lib/pulp/content/python_package/mongoengine-0.8.7.tar.gz",
    "description": "===========",
    "license": "MIT",
    "author": "Ross Lawley",
    "home_page": "http://mongoengine.org/",
    "_filename": "mongoengine-0.8.7.tar.gz",
    "_last_updated": 1414007686,
    "summary": "MongoEngine is a Python Object-Document Mapper for working with MongoDB.",
    "author_email": "ross.lawley@{nospam}gmail.com",
    "version": "0.8.7",
    "platform": "any",
    "_content_type_id": "python_package",
    "_ns": "units_python_package",
    "name": "mongoengine"
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I expected this to be for 'filename' not 'args'. Also the filename field should be documented.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was supposed to say filename - I'll fix it.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@bmbouter
Copy link
Member

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants