Skip to content

Commit 8fec04e

Browse files
authored
Turn on LPID (#8976)
1 parent d213bcc commit 8fec04e

File tree

2 files changed

+15
-14
lines changed

2 files changed

+15
-14
lines changed

tests/log_parser/test_store_failure_lines.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,7 @@ def test_infra_no_intermittent(activate_responses, hundred_job_blobs, mock_parse
327327

328328
# this will parse and check for intermittents
329329
mock_full_log_parser(job_logs, mock_parser)
330-
# verify_classification_id(jobs, 1, 8)
330+
verify_classification_id(jobs, 1, 8)
331331

332332

333333
def test_infra_intermittent(activate_responses, hundred_job_blobs, mock_parser, create_jobs):
@@ -342,7 +342,7 @@ def test_infra_intermittent(activate_responses, hundred_job_blobs, mock_parser,
342342

343343
# this will parse and check for intermittents
344344
mock_full_log_parser(job_logs, mock_parser)
345-
# verify_classification_id(jobs, 8, 1)
345+
verify_classification_id(jobs, 8, 1)
346346

347347

348348
def test_multiple_jobs_intermittent(
@@ -360,7 +360,7 @@ def test_multiple_jobs_intermittent(
360360

361361
# this will parse and check for intermittents
362362
mock_full_log_parser(job_logs, mock_parser)
363-
# verify_classification_id(jobs, 8, 8)
363+
verify_classification_id(jobs, 8, 8)
364364

365365

366366
def test_confirm_failure_no_intermittent(
@@ -378,7 +378,7 @@ def test_confirm_failure_no_intermittent(
378378

379379
# this will parse and check for intermittents
380380
mock_full_log_parser(job_logs, mock_parser)
381-
# verify_classification_id(jobs, 1, 1)
381+
verify_classification_id(jobs, 1, 1)
382382

383383

384384
def test_confirm_failure_partial_intermittent(
@@ -395,7 +395,7 @@ def test_confirm_failure_partial_intermittent(
395395

396396
# this will parse and check for intermittents
397397
mock_full_log_parser(job_logs, mock_parser)
398-
# verify_classification_id(jobs, 1, 1)
398+
verify_classification_id(jobs, 1, 1)
399399

400400

401401
def test_confirm_failure_pass_intermittent(
@@ -414,7 +414,7 @@ def test_confirm_failure_pass_intermittent(
414414

415415
# this will parse and check for intermittents
416416
mock_full_log_parser(job_logs, mock_parser)
417-
# verify_classification_id(jobs, 8, 1)
417+
verify_classification_id(jobs, 8, 1)
418418

419419

420420
def test_retrigger_no_intermittent(activate_responses, hundred_job_blobs, mock_parser, create_jobs):
@@ -429,7 +429,7 @@ def test_retrigger_no_intermittent(activate_responses, hundred_job_blobs, mock_p
429429

430430
# this will parse and check for intermittents
431431
mock_full_log_parser(job_logs, mock_parser)
432-
# verify_classification_id(jobs, 1, 1)
432+
verify_classification_id(jobs, 1, 1)
433433

434434

435435
def test_retrigger_intermittent(activate_responses, hundred_job_blobs, mock_parser, create_jobs):
@@ -444,7 +444,7 @@ def test_retrigger_intermittent(activate_responses, hundred_job_blobs, mock_pars
444444

445445
# this will parse and check for intermittents
446446
mock_full_log_parser(job_logs, mock_parser)
447-
# verify_classification_id(jobs, 8, 8)
447+
verify_classification_id(jobs, 8, 8)
448448

449449

450450
def test_reclassify_group_intermittent(
@@ -462,7 +462,7 @@ def test_reclassify_group_intermittent(
462462

463463
# this will parse and check for intermittents
464464
mock_full_log_parser(job_logs, mock_parser)
465-
# verify_classification_id(jobs, 1, 8)
465+
verify_classification_id(jobs, 1, 8)
466466

467467

468468
def test_reclassify_infra_intermittent(
@@ -480,4 +480,4 @@ def test_reclassify_infra_intermittent(
480480

481481
# this will parse and check for intermittents
482482
mock_full_log_parser(job_logs, mock_parser)
483-
# verify_classification_id(jobs, 1, 1)
483+
verify_classification_id(jobs, 1, 1)

treeherder/log_parser/tasks.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import newrelic.agent
44
import simplejson as json
55
from celery.exceptions import SoftTimeLimitExceeded
6+
from django.conf import settings
67
from requests.exceptions import HTTPError
78

89
from treeherder.etl.artifact import serialize_artifact_json_blobs, store_job_artifacts
@@ -13,7 +14,7 @@
1314
from treeherder.model.models import Job, JobLog
1415
from treeherder.workers.task import retryable_task
1516

16-
from . import failureline
17+
from . import failureline, intermittents
1718

1819
logger = logging.getLogger(__name__)
1920

@@ -84,9 +85,9 @@ def store_failure_lines(job_log):
8485
logger.info("Running store_failure_lines for job %s", job_log.job.id)
8586
failureline.store_failure_lines(job_log)
8687
# NOTE: do not run on production
87-
# if settings.SITE_HOSTNAME != "treeherder.mozilla.org":
88-
# logger.info("Running check_and_mark_intermittent for job %s", job_log.job.id)
89-
# intermittents.check_and_mark_intermittent(job_log.job.id)
88+
if settings.SITE_HOSTNAME != "treeherder.mozilla.org":
89+
logger.info("Running check_and_mark_intermittent for job %s", job_log.job.id)
90+
intermittents.check_and_mark_intermittent(job_log.job.id)
9091

9192

9293
def post_log_artifacts(job_log):

0 commit comments

Comments
 (0)