Closed Bug 1328322 Opened 8 years ago Closed 8 years ago

Remove unused custom Django management commands

Categories

(Tree Management :: Treeherder, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

A few of our management commands are no longer needed. We should go through the list and remove any that are now superfluous: ~/treeherder $ grep 'class Command' -R -l . ./treeherder/autoclassify/management/commands/autoclassify.py ./treeherder/autoclassify/management/commands/detect_intermittents.py ./treeherder/autoclassify/management/commands/es_benchmark.py ./treeherder/autoclassify/management/commands/matchers.py ./treeherder/credentials/management/commands/create_credentials.py ./treeherder/etl/management/commands/ingest_push.py ./treeherder/etl/management/commands/publish_to_pulse.py ./treeherder/etl/management/commands/read_pulse_jobs.py ./treeherder/etl/management/commands/read_pulse_resultsets.py ./treeherder/etl/management/commands/update_bugscache.py ./treeherder/log_parser/management/commands/crossreference_error_lines.py ./treeherder/log_parser/management/commands/remove_duplicate_summaries.py ./treeherder/log_parser/management/commands/test_parse_log.py ./treeherder/model/management/commands/calculate_durations.py ./treeherder/model/management/commands/clear_cache.py ./treeherder/model/management/commands/cycle_data.py ./treeherder/model/management/commands/es_import_failure_lines.py ./treeherder/model/management/commands/import_reference_data.py ./treeherder/model/management/commands/init_datasources.py ./treeherder/model/management/commands/load_initial_data.py ./treeherder/model/management/commands/migrate_to_intermediate_jobs.py ./treeherder/model/management/commands/publish_result_set_to_pulse.py ./treeherder/model/management/commands/run_sql.py ./treeherder/perf/management/commands/create_test_perf_data.py ./treeherder/perf/management/commands/generate_alerts.py ./treeherder/perf/management/commands/import_perf_data.py ./treeherder/perf/management/commands/remove_duplicate_signatures.py ./treeherder/perf/management/commands/remove_perf_signatures.py ./treeherder/perf/management/commands/remove_single_test_summaries.py ./treeherder/perf/management/commands/test_analyze_perf.py
Bug 1328277 removed: ./treeherder/model/management/commands/migrate_to_intermediate_jobs.py ./treeherder/perf/management/commands/remove_duplicate_signatures.py ./treeherder/perf/management/commands/remove_single_test_summaries.py
Depends on: 1328277
Assignee: nobody → emorley
James, is this now unused? treeherder/log_parser/management/commands/remove_duplicate_summaries.py Will, how about this one? treeherder/perf/management/commands/migrate_perf_signature_extra_options.py
Flags: needinfo?(wlachance)
Flags: needinfo?(james)
(In reply to Ed Morley [:emorley] from comment #2) > Will, how about this one? > treeherder/perf/management/commands/migrate_perf_signature_extra_options.py This was added to do a one-off migration as part of bug 1310386 and can now be taken out. :rwood, can you take care of this? Maybe in the future we should lean towards doing those things as bonafide django migrations. The problem is that django migrations are kind of annoying to edit & test when you get something wrong.
Flags: needinfo?(wlachance) → needinfo?(rwood)
Depends on: 1337211
Yes certainly
Flags: needinfo?(rwood)
Attachment #8837838 - Flags: review?(james)
Flags: needinfo?(james)
Attachment #8837838 - Flags: review?(james) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: