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

Trim down dependencies and settings #5

Closed
atodorov opened this Issue May 26, 2017 · 0 comments

Comments

Projects
None yet
1 participant
@atodorov
Member

atodorov commented May 26, 2017

Some dependencies are not required. For example most installation don't use Kerberos, smaller installations may not need Celery.

KTP needs to remove them from the default installation and provide an easy way to configure them if they are required.

atodorov added a commit that referenced this issue Jun 6, 2017

Remove dependency on Celery and django-celery. Relates to #5
- refactored the email backend to use threading instead of Celery
- dropped Celery based Bugzilla integration. Fixes #1
- dropped the following configuration settings:
  EMAILS_FOR_DEBUG (replaced by ADMINS), ENABLE_ASYNC_EMAIL,
  BROKER_URL, CELERY_TASK_RESULT_EXPIRES, CELERY_RESULT_BACKEND,
  CELERYD_TIMER_PRECISION, CELERY_IGNORE_RESULT,
  CELERY_MAX_CACHED_RESULTS, CELERY_DEFAULT_RATE_LIMIT

atodorov added a commit that referenced this issue Jun 6, 2017

Remove dependency on Celery and django-celery. Relates to #5
- refactored the email backend to use threading instead of Celery
- dropped Celery based Bugzilla integration. Fixes #1
- dropped the following configuration settings:
  EMAILS_FOR_DEBUG (replaced by ADMINS), ENABLE_ASYNC_EMAIL,
  BROKER_URL, CELERY_TASK_RESULT_EXPIRES, CELERY_RESULT_BACKEND,
  CELERYD_TIMER_PRECISION, CELERY_IGNORE_RESULT,
  CELERY_MAX_CACHED_RESULTS, CELERY_DEFAULT_RATE_LIMIT

atodorov added a commit that referenced this issue Jun 6, 2017

Remove kerberos dependency. Relates to #5
kerberos is not used by everyone so drop it from the required
dependencies. However add a documentation section how to configure
it in case someone wants to use it!

atodorov added a commit that referenced this issue Jun 6, 2017

Remove kerberos dependency. Relates to #5
kerberos is not used by everyone so drop it from the required
dependencies. However add a documentation section how to configure
it in case someone wants to use it!

atodorov added a commit that referenced this issue Jun 6, 2017

Drop dependency on Kobo. Fix #5
- copy XML-RPC related code from Kobo
- drop the dependency

Note: Kobo appears to be an old piece of code which isn't actively
maintained with latest Django versions and appears to include
bunch of random integrations from Red Hat's Release Engineering.
I've taken only the bare minimum code which is relevant to us
and copied it here so we can drop the dependency.

@atodorov atodorov closed this in 1d3b269 Jun 6, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment