Closed
Bug 1328322
Opened 8 years ago
Closed 8 years ago
Remove unused custom Django management commands
Categories
(Tree Management :: Treeherder, defect, P3)
Tree Management
Treeherder
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
Assignee | ||
Comment 1•8 years ago
|
||
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 | ||
Updated•8 years ago
|
Assignee: nobody → emorley
Assignee | ||
Comment 2•8 years ago
|
||
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)
Comment 3•8 years ago
|
||
(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)
Comment 5•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Attachment #8837838 -
Flags: review?(james)
Updated•8 years ago
|
Flags: needinfo?(james)
Attachment #8837838 -
Flags: review?(james) → review+
Comment 6•8 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/88b1a91a9f24cc87a1c3357f07f1f0f437443883
Bug 1328322 - Remove obsolete commands/remove_duplicate_summaries.py
Assignee | ||
Updated•8 years ago
|
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.
Description
•