Skip to content

feat: rip kafka_consumer out of edx-platform #30018

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.

Sign up for GitHub

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

Closed
rgraber wants to merge 3 commits into from
Closed

feat: rip kafka_consumer out of edx-platform #30018

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
901016e
feat: rm kafka_consumer
Feb 25, 2022
e71d68b
feat: rm unused libraries
Feb 25, 2022
415e3c2
fix: reqs
Mar 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions lms/envs/common.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3213,9 +3213,6 @@ def _make_locale_paths(settings): # pylint: disable=missing-function-docstring
# For save for later
'lms.djangoapps.save_for_later',

# TODO (EventBus): Make Kafka/event-bus optional
'openedx.core.djangoapps.kafka_consumer',

# Course Live App
'openedx.core.djangoapps.course_live',
]
Expand Down
17 changes: 0 additions & 17 deletions openedx/core/djangoapps/kafka_consumer/README
View file
Open in desktop

This file was deleted.

Empty file removed openedx/core/djangoapps/kafka_consumer/__init__.py
View file
Open in desktop
Empty file.
7 changes: 0 additions & 7 deletions openedx/core/djangoapps/kafka_consumer/apps.py
View file
Open in desktop

This file was deleted.

99 changes: 0 additions & 99 deletions openedx/core/djangoapps/kafka_consumer/consumers.py
View file
Open in desktop

This file was deleted.

30 changes: 0 additions & 30 deletions openedx/core/djangoapps/kafka_consumer/docs/0001_kafka_consumer_command.rst
View file
Open in desktop

This file was deleted.

Empty file removed openedx/core/djangoapps/kafka_consumer/management/__init__.py
View file
Open in desktop
Empty file.
Empty file removed openedx/core/djangoapps/kafka_consumer/management/commands/__init__.py
View file
Open in desktop
Empty file.
118 changes: 0 additions & 118 deletions openedx/core/djangoapps/kafka_consumer/management/commands/consume_events.py
View file
Open in desktop

This file was deleted.

5 changes: 5 additions & 0 deletions requirements/common_constraints.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,11 @@
# See BOM-2721 for more details.
# Below is the copied and edited version of common_constraints

# This is a temporary solution to override the real common_constraints.txt
# In edx-lint, until the pyjwt constraint in edx-lint has been removed.
# See BOM-2721 for more details.
# Below is the copied and edited version of common_constraints

# A central location for most common version constraints
# (across edx repos) for pip-installation.
#
Expand Down
2 changes: 0 additions & 2 deletions requirements/edx/base.in
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ botocore==1.8.17 # via boto3, s3transfer
bridgekeeper # Used for determining permissions for courseware.
celery # Asynchronous task execution library
chem # A helper library for chemistry calculations
confluent_kafka
contextlib2 # We need contextlib2.ExitStack so we can stop using contextlib.nested which doesn't exist in python 3
crowdsourcehinter-xblock
cryptography # Implementations of assorted cryptography algorithms
Expand Down Expand Up @@ -97,7 +96,6 @@ edx-user-state-client
edx-when
edxval
event-tracking
fastavro
fs==2.0.18
fs-s3fs==0.1.8
geoip2 # Python API for the GeoIP web services and databases
Expand Down
6 changes: 1 addition & 5 deletions requirements/edx/base.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -129,8 +129,6 @@ code-annotations==1.3.0
# via
# edx-enterprise
# edx-toggles
confluent-kafka==1.8.2
# via -r requirements/edx/base.in
contextlib2==21.6.0
# via -r requirements/edx/base.in
coreapi==2.3.3
Expand Down Expand Up @@ -526,9 +524,7 @@ event-tracking==2.0.0
# edx-proctoring
# edx-search
fastavro==1.4.9
# via
# -r requirements/edx/base.in
# openedx-events
# via openedx-events
frozenlist==1.3.0
# via
# aiohttp
Expand Down
2 changes: 0 additions & 2 deletions requirements/edx/development.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,8 +183,6 @@ code-annotations==1.3.0
# edx-enterprise
# edx-lint
# edx-toggles
confluent-kafka==1.8.2
# via -r requirements/edx/testing.txt
contextlib2==21.6.0
# via -r requirements/edx/testing.txt
coreapi==2.3.3
Expand Down
3 changes: 0 additions & 3 deletions requirements/edx/testing.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@ attrs==21.4.0
# aiohttp
# edx-ace
# openedx-events
# outcome
# pytest
babel==2.9.1
# via
Expand Down Expand Up @@ -174,8 +173,6 @@ code-annotations==1.3.0
# edx-enterprise
# edx-lint
# edx-toggles
confluent-kafka==1.8.2
# via -r requirements/edx/base.txt
contextlib2==21.6.0
# via -r requirements/edx/base.txt
coreapi==2.3.3
Expand Down