lp:~bac/charmworld/make-queue-a-job

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:charmworld at revision 469

Juju Gui Bot: Approve (continuous-integration) on 2014-01-02

Charmworld Developers: Pending requested 2013-12-24

Diff: 448 lines (+177/-45)

6 files modified

charmworld/jobs/lp.py (+16/-6)
charmworld/jobs/tests/test_lp.py (+47/-0)
charmworld/jobs/tests/test_worker.py (+58/-26)
charmworld/jobs/worker.py (+26/-11)
charmworld/testing/__init__.py (+29/-0)
docs/hacking.rst (+1/-2)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1130732: The ingest process is easily broken on staging

Low

Fix Released

Link a bug report

Related blueprints

470. By Brad Crittenden on 2014-01-02

Fixes from review. Allow lp-main to be called with no args since that's what bin/enqueue does. Add tests. Make existing tests more readable and comment intent.

469. By Brad Crittenden on 2013-12-24

Updated hacking.rst to remove bin/enqueue.

468. By Brad Crittenden on 2013-12-24

Run queue and ingest from one worker. Add missing tests.

467. By Brad Crittenden on 2013-11-20

Add download stats to bundle API call.

https://codereview.appspot.com/29850043/

R=benji.

Approved by Juju Gui Bot.

466. By j.c.sackett on 2013-11-20

Remove buggy jspark and create better sparklines.

Approved by Juju Gui Bot, Aaron Bentley.

465. By Benji York on 2013-11-19

Add a script to remove a charm

https://codereview.appspot.com/29210043/

R=bac.

Approved by Juju Gui Bot, Benji York.

464. By j.c.sackett on 2013-11-18

Handle some failure modes with better logging.

Approved by Juju Gui Bot, Aaron Bentley.

463. By Brad Crittenden on 2013-11-15

Handle api_searchs with ~user

Treat them as a search "owner: user" as we do in the in-site search.
Also don't puke if special characters in the user-entered search
cause an ElasticHttpError.

https://codereview.appspot.com/23330046/

R=gary.

Approved by Juju Gui Bot.

462. By Benji York on 2013-11-15

Factor the server start time script out.

This is a refactor in anticipation of adding another script.

https://codereview.appspot.com/22190049/

R=gary.

Approved by Juju Gui Bot, Benji York.

461. By Abel Deuring on 2013-11-14

heartbeat page:
  - show a warning if charmworld's crontab file is missing
  - show the time when the last ingest job finished
  - show the time when the charmworld app server(s) were started.

Approved by Juju Gui Bot, Abel Deuring, Benji York.