Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

Commit

Permalink
tradefed: do not log error on missing tradefed file
Browse files Browse the repository at this point in the history
The CTS/VTS plugin will only work on jobs that have the tradefed.yml file
specified in definition, otherwise, skip it.

Logging as error causes error monitoring systems, such as Sentry, to
think it's an actual error.

Signed-off-by: Charles Oliveira <[email protected]>
  • Loading branch information
chaws committed Apr 5, 2024
1 parent 162490e commit f9bf726
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tradefed/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -561,7 +561,7 @@ def postprocess_testjob(self, testjob):

tradefed_files = self._extract_tradefed_from_job_definition(testjob)
if len(tradefed_files) != 1:
logger.error(f"Job {testjob.id} has {len(tradefed_files)} tradefed files in the definition, it should have 1")
logger.info(f"Job {testjob.id} has {len(tradefed_files)} tradefed files in the definition, it should have 1, aborting")
update_testjob_status.delay(testjob.id, self.extra_args.get("job_status"))
logger.info("Finishing CTS/VTS plugin for test run: %s" % testjob)
return
Expand Down

0 comments on commit f9bf726

Please sign in to comment.