From 27a8b2ea67572aa9b516fa59bda8b7683e804a7c Mon Sep 17 00:00:00 2001 From: ktyagiapphelix2u Date: Thu, 26 Mar 2026 12:39:37 +0000 Subject: [PATCH 1/5] fix: add managemement command to filter hook --- manage.py | 28 +++++++++++++++++++++++++++- 1 file changed, 27 insertions(+), 1 deletion(-) diff --git a/manage.py b/manage.py index 9b8518937d48..fbbd40d0360a 100755 --- a/manage.py +++ b/manage.py @@ -23,6 +23,14 @@ import sys from argparse import ArgumentParser +from openedx_filters import OpenEdxPublicFilter + + +class ManagementCommandExecutionRequested(OpenEdxPublicFilter): + """Filter triggered before a management command is executed.""" + + FILTER_TYPE = 'org.openedx.platform.management.command.execute.requested.v1' + def parse_args(): """Parse edx specific arguments to manage.py""" @@ -96,4 +104,22 @@ def parse_args(): django_args.append('--help') from django.core.management import execute_from_command_line - execute_from_command_line([sys.argv[0]] + django_args) + + def command_runner(): + return execute_from_command_line([sys.argv[0]] + django_args) + + command_name = 'help' + if django_args and not django_args[0].startswith('-'): + command_name = django_args[0] + + try: + pipeline_output = ManagementCommandExecutionRequested.run_filter( + command_name=command_name, + service_variant=os.environ.get("SERVICE_VARIANT", edx_args.service_variant), + command_runner=command_runner, + ) + runner = pipeline_output.get('command_runner', command_runner) + except Exception: # pylint: disable=broad-except + runner = command_runner + + runner() From e0628c690ab8a12cf00902884891ff5b667b8139 Mon Sep 17 00:00:00 2001 From: ktyagiapphelix2u Date: Wed, 17 Jun 2026 06:19:44 +0000 Subject: [PATCH 2/5] fix: add managemement command to filter hook --- manage.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/manage.py b/manage.py index fbbd40d0360a..1dfbbfa54fe3 100755 --- a/manage.py +++ b/manage.py @@ -23,13 +23,13 @@ import sys from argparse import ArgumentParser -from openedx_filters import OpenEdxPublicFilter +from openedx_filters.tooling import OpenEdxPublicFilter class ManagementCommandExecutionRequested(OpenEdxPublicFilter): """Filter triggered before a management command is executed.""" - FILTER_TYPE = 'org.openedx.platform.management.command.execute.requested.v1' + filter_type = 'org.openedx.platform.management.command.execute.requested.v1' def parse_args(): From 8e3ba1deab52f28466228f6dc80e282da29d6651 Mon Sep 17 00:00:00 2001 From: ktyagiapphelix2u Date: Wed, 17 Jun 2026 07:12:40 +0000 Subject: [PATCH 3/5] fix: add managemement command to filter hook --- manage.py | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/manage.py b/manage.py index 1dfbbfa54fe3..b7cf924ee6af 100755 --- a/manage.py +++ b/manage.py @@ -31,6 +31,15 @@ class ManagementCommandExecutionRequested(OpenEdxPublicFilter): filter_type = 'org.openedx.platform.management.command.execute.requested.v1' + @classmethod + def run_filter(cls, command_name, service_variant, command_runner): + """Run the management command execution pipeline.""" + return cls.run_pipeline( + command_name=command_name, + service_variant=service_variant, + command_runner=command_runner, + ) + def parse_args(): """Parse edx specific arguments to manage.py""" From 7503829b475bcf2b19fda2cf3be4b960ce521ce6 Mon Sep 17 00:00:00 2001 From: ktyagiapphelix2u Date: Mon, 20 Jul 2026 09:30:47 +0000 Subject: [PATCH 4/5] fix: add managemement command to filter hook --- manage.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/manage.py b/manage.py index b7cf924ee6af..0ba4e77772d7 100755 --- a/manage.py +++ b/manage.py @@ -27,13 +27,17 @@ class ManagementCommandExecutionRequested(OpenEdxPublicFilter): - """Filter triggered before a management command is executed.""" + """ + Filter triggered before a management command is executed. + """ filter_type = 'org.openedx.platform.management.command.execute.requested.v1' @classmethod def run_filter(cls, command_name, service_variant, command_runner): - """Run the management command execution pipeline.""" + """ + Run the management command execution pipeline. + """ return cls.run_pipeline( command_name=command_name, service_variant=service_variant, @@ -42,7 +46,9 @@ def run_filter(cls, command_name, service_variant, command_runner): def parse_args(): - """Parse edx specific arguments to manage.py""" + """ + Parse edx specific arguments to manage.py + """ parser = ArgumentParser() subparsers = parser.add_subparsers(title='system', description='edX service to run') From 381a748e4d4f153e9fbd7a5f4d40464d945173df Mon Sep 17 00:00:00 2001 From: ktyagiapphelix2u Date: Thu, 23 Jul 2026 05:13:19 +0000 Subject: [PATCH 5/5] fix: add managemement command to filter hook --- manage.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/manage.py b/manage.py index 0ba4e77772d7..b27a04ca41af 100755 --- a/manage.py +++ b/manage.py @@ -123,18 +123,18 @@ def parse_args(): def command_runner(): return execute_from_command_line([sys.argv[0]] + django_args) - command_name = 'help' - if django_args and not django_args[0].startswith('-'): - command_name = django_args[0] + command_name = next((arg for arg in django_args if not arg.startswith('-')), 'help') - try: - pipeline_output = ManagementCommandExecutionRequested.run_filter( - command_name=command_name, - service_variant=os.environ.get("SERVICE_VARIANT", edx_args.service_variant), - command_runner=command_runner, - ) + pipeline_output = ManagementCommandExecutionRequested.run_filter( + command_name=command_name, + service_variant=os.environ.get("SERVICE_VARIANT", edx_args.service_variant), + command_runner=command_runner, + ) + + runner = command_runner + if isinstance(pipeline_output, dict): runner = pipeline_output.get('command_runner', command_runner) - except Exception: # pylint: disable=broad-except + if not callable(runner): runner = command_runner runner()