#2296 Add workers and move repo editing tasks to them
Merged by pingou. Opened by puiterwijk.
puiterwijk/pagure rearchitect  into  master

Download 2296.patch

This is a huge change, that will make Pagure require redis or rabbitmq and a running worker.

..............................................
GIT PERFORMANCE TOTALS:
Walks: 217
Steps: 535


Ran 553 tests in 1212.239s

OK (SKIP=2)

Missing in the doc?

is vs can be?

This call does not have any of its parameters mentioned in the doc.

Right :)

One way to run the full setup for manual tests or deployments:

docker-compose up

This will then make the web frontend available on localhost:5000.

1 new commit added

  • Use the most compatible way of arguments for celery

3 new commits added

  • Make the wait page a little prettier
  • Make the test suites quiet again
  • Fix task ready check

Pull-Request has been merged by pingou

I think this value should be the value of:

{'message': 'Project "%s" created' % project.fullname}

Just a thought, perhaps this should check to make sure that config value is set, and if not, throw an exception rather than just get a KeyError.

It's always set: in default_config it's set to 0.0.0.0. So you'd just get an "cannot connect to redis" error, which is just fine.

Metadata