Bug 845465 - Review Request: python-django-celery - Django Celery Integration
Review Request: python-django-celery - Django Celery Integration
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Bohuslav "Slavek" Kabrda
Fedora Extras Quality Assurance
:
Depends On:
Blocks: 736776 840359
  Show dependency treegraph
 
Reported: 2012-08-03 03:20 EDT by Matthias Runge
Modified: 2012-08-06 01:44 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2012-08-06 01:44:31 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
bkabrda: fedora‑review+
limburgher: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Matthias Runge 2012-08-03 03:20:20 EDT
Spec URL: http://www.matthias-runge.de/fedora/python-django-celery.spec
SRPM URL: http://www.matthias-runge.de/fedora/python-django-celery-3.0.4-1.fc17.src.rpm
Description: django-celery provides Celery integration for Django; Using the Django ORM and
cache backend for storing results, autodiscovery of task modules for
applications listed in INSTALLED_APPS, and more.

Celery is a task queue/job queue based on distributed message passing. It is
focused on real-time operation, but supports scheduling as well.

The execution units, called tasks, are executed concurrently on a single or more
worker servers. Tasks can execute asynchronously (in the background) or
synchronously (wait until ready).

Fedora Account System Username: mrunge


please note, this is a review required for package rename.



[mrunge@mrungexp SPECS]$ rpmlint ./python-django-celery.spec ../SRPMS/python-django-celery-3.0.4-1.fc17.src.rpm ../RPMS/noarch/python-django-celery-3.0.4-1.fc17.noarch.rpm 
python-django-celery.src: W: spelling-error %description -l en_US backend -> backed, back end, back-end
python-django-celery.src: W: spelling-error %description -l en_US autodiscovery -> auto discovery, auto-discovery, discovery
python-django-celery.noarch: W: spelling-error %description -l en_US backend -> backed, back end, back-end
python-django-celery.noarch: W: spelling-error %description -l en_US autodiscovery -> auto discovery, auto-discovery, discovery
python-django-celery.noarch: W: no-manual-page-for-binary djcelerymon
2 packages and 1 specfiles checked; 0 errors, 5 warnings.
Comment 1 Bohuslav "Slavek" Kabrda 2012-08-03 04:11:36 EDT
I'll take it for a review.
Comment 2 Bohuslav "Slavek" Kabrda 2012-08-03 04:34:08 EDT
- Your changelog mentions updating to version 3.0.5, but the actual version is 3.0.4, please fix this.
- The package includes some tests, have you tried running them? It seems that after some playing with them, they may be runnable via nosetests, although I'm not sure it's doable without django_nose (which we don't have in Fedora).
Comment 3 Matthias Runge 2012-08-03 05:28:52 EDT
I'm honestly sorry!
I had put those files on my webspace and ran rpmlint later. I just have forgotten to replace those files the corrected versions. It's done now.

I must say, didn't try the checks yet, good news is, we already have django-nose:
https://admin.fedoraproject.org/pkgdb/acls/name/python-django-nose
Sadly, running those tests is not that easy as I thought.
Comment 4 Bohuslav "Slavek" Kabrda 2012-08-03 06:00:19 EDT
Ok, here is how to run the tests:
1) install all the requirements (django, django-nose, celery, mock)
2) patch the tests/settings.py to leave out the coverage stuff (starting line 37)
3) run python setup.py test
That does the trick for me.
Comment 5 Matthias Runge 2012-08-03 06:18:41 EDT
yepp, here too, without patching. 

You need latest billiard, celery and kombu from rawhide. Sadly, the builders don't have it (yet).

SRPM: http://www.matthias-runge.de/fedora/python-django-celery-3.0.4-2.fc17.src.rpm
SPEC: http://www.matthias-runge.de/fedora/python-django-celery.spec
Comment 6 Bohuslav "Slavek" Kabrda 2012-08-03 06:32:39 EDT
Oh, sure, you also install nose-cover3 so you don't need to patch the tests.

I tried a scratch build and it seems that koji already has the newest versions, but you're missing BR: python-mock [1]. Would you mind adding it and putting the spec + srpm one last time in here, so that I can check before finally approving?

Thanks!

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=4354079
Comment 7 Matthias Runge 2012-08-03 06:40:35 EDT
OK, that's it. [1]

I would have sworn to have python-mock included also, but you're right, it was missing!


SPEC: http://www.matthias-runge.de/fedora/python-django-celery.spec
SRPM: http://www.matthias-runge.de/fedora/python-django-celery-3.0.4-3.fc17.src.rpm


[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=4354083
Comment 8 Bohuslav "Slavek" Kabrda 2012-08-03 06:43:58 EDT
Great. I don't have any other objections. This package is APPROVED.
Comment 9 Matthias Runge 2012-08-03 06:45:54 EDT
Thank you very much!

New Package SCM Request
=======================
Package Name: python-django-celery
Short Description: Django Celery Integration
Owners: mrunge
Branches: devel
Comment 10 Gwyn Ciesla 2012-08-03 16:03:41 EDT
Git done (by process-git-requests).
Comment 11 Matthias Runge 2012-08-06 01:44:31 EDT
package imported and built. thanks.

Note You need to log in before you can comment on or make changes to this bug.