pagure

#2318 Add multiple locking types
Merged by pingou. Opened by puiterwijk.
puiterwijk/pagure nonlockinglocking  into  master

Download 2318.patch

This allows us to add multiple types of locks, so you can lock a project only for the worker, and the frontend won't get locked with it.

rebased

6 new commits added

  • Make wait page take prev
  • Reverse commits in test
  • Make tests handle wait pages
  • Add explicit project-level locking with lock types
  • Use createdb and alembic during docker-composer
  • SIGKILL the worker and broker in the tests

1 new commit added

  • Accept results in race condition

7 new commits added

  • Accept results in race condition
  • Make wait page take prev
  • Reverse commits in test
  • Make tests handle wait pages
  • Add explicit project-level locking with lock types
  • Use createdb and alembic during docker-composer
  • SIGKILL the worker and broker in the tests

7 new commits added

  • Accept results in race condition
  • Make wait page take prev
  • Reverse commits in test
  • Make tests handle wait pages
  • Add explicit project-level locking with lock types
  • Use createdb and alembic during docker-composer
  • SIGKILL the worker and broker in the tests

7 new commits added

  • Accept results in race condition
  • Make wait page take prev
  • Reverse commits in test
  • Make tests handle wait pages
  • Add explicit project-level locking with lock types
  • Use createdb and alembic during docker-composer
  • SIGKILL the worker and broker in the tests

1 new commit added

  • Use a more reliably and compatible way to set test broker url

1 new commit added

  • After in-test self.setUp(), give worker time to start

1 new commit added

  • Close workerlog at the end

Tests are working locally and we've tested the locking and it works fine.

Thanks for all these @puiterwijk !!

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.