Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update django-q with django-q2 #739

Draft
wants to merge 107 commits into
base: master
Choose a base branch
from
Draft

Update django-q with django-q2 #739

wants to merge 107 commits into from

Conversation

GDay
Copy link
Collaborator

@GDay GDay commented Mar 5, 2024

Still a work in progress. What needs to be done:

  • Update docs to use django-q (instead of django-q2) everywhere. PR: Docs updates django-q2/django-q2#163
  • Update workflow on django-q2 to push both packages to PyPI. (disable push when on the django-q repo)

After merge:

  • Go through all issues and PRs and close the ones that are not relevant anymore (have already been fixed)

LJlkdskdjflsa and others added 30 commits September 1, 2022 12:08
* refactor: show function name as it runs

* feat: log func name when processed

Co-authored-by: Noortheen Raja <jnoortheen@gmail.com>
Migrate codecov to coveralls
* Fix readthedocs file

* remove trigger from test workflow
* Update django docs links to 4.0
* Update Django Q to Django Q2
* Use different theme to make difference more clear between the original and this forked one
* Added info notice about this fork
* Some small layout/typo fixes
Co-authored-by: ethemguner
Co-authored-by: Marc Sabatier <marc@sabatier.online>
Co-authored-by: Marc Sabatier <marc@sabatier.online>
* Add the group column to OrmQ admin page

* Add group field to tasks list and search_field

* Add search by name to Schedule admin page

* Limit succesful task history per schedule with SAVE_LIMIT_GROUP config

* Improve ScheduleAdmin list view performance

Co-authored-by: Marc Sabatier <marc@sabatier.online>
* feat: option to save tasks per group/func/name

* test: update tests for save limit

* fix: convert func when save-limit checking

when passed as function it needs to be converted to work

* Add warning if option is not valid

Co-authored-by: Noortheen Raja <jnoortheen@gmail.com>
Instead of checking if the django-q logger has any handlers directly,
use the hasHandlers [1] method, which will check if any parent loggers
have handlers as well.

This fixes a bug where django-q configures it's own logging even though
the root logger has a handler set

 1. https://docs.python.org/3/library/logging.html#logging.Logger.hasHandlers

Co-authored-by: Wesley Ellis <wellis@benchsci.com>
Co-authored-by: Rajeev KL <rajeev@labmoneta.io>
* fix django q2 install command and remove space

* remove warnings for very old versions
having build-system config helps installing  this without setup.py built
GDay and others added 30 commits May 14, 2023 03:04
* make processes identifiable

* update docs
* Move logic that resubmits a task and then deletes the original into an action function that can be used for both failure and success resubmission.

* Use the resubmit_task action in the FailAdmin.

* Add the resubmit_task action to the TaskAdmin class.

* Update resubmit failure test to reflect using the new resubmit action.

* Add a test for resubmitting successful tasks.

---------

Co-authored-by: Scott Pashley
List entry points using importlib.metadata (Python >=3.10) or
importlib-metadata>=3.6 (Python < 3.10)
* Fix not picking up result from falsy result

* add test and docs

* typo
* Correct health check import in examples.rst

* Update monitor.rst

---------

Co-authored-by: Stan Triepels <1939656+GDay@users.noreply.github.com>
Otherwise, trying to build the project with a tool like gpep517 or build
results in using the setuptools legacy backend instead
* Comment out decrement of s.repeats in scheduler to fix negative repeat count issue

* Fix repeat count handling in scheduler

- Reset `repeats` to -1 for values less than -1
- Decrement `repeats` by 1 if it's greater than 0
* Update release dependencies

* Update test.yml
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.

None yet