Commit 46765230 authored by Ramin Halavati's avatar Ramin Halavati Committed by Commit Bot

Revert "Add debug data for Traffic Annotation FYI bot."

This reverts commit 5bea452c.

Reason for revert: This CL was for collecting debug data on FYI bot.
Data is now collected.

Original change's description:
> Add debug data for Traffic Annotation FYI bot.
> 
> Traffic annotations FYI bot cannot access some files in build_path/gen.
> Some Debug data is added to find out why?
> 
> TBR=msramek
> 
> Bug: 690323
> Change-Id: I4fb716d8b843704bf48cfbb62d5b09a3791eb732
> Reviewed-on: https://chromium-review.googlesource.com/966303
> Reviewed-by: Ramin Halavati <rhalavati@chromium.org>
> Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#543702}

TBR=msramek@chromium.org,rhalavati@chromium.org

Change-Id: Ia3c53a938c1f19122e4b82ca77b357502feaeb89
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 690323
Reviewed-on: https://chromium-review.googlesource.com/966762Reviewed-by: default avatarRamin Halavati <rhalavati@chromium.org>
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543741}
parent bc83c776
...@@ -9,14 +9,13 @@ ...@@ -9,14 +9,13 @@
import os import os
import argparse import argparse
import sys import sys
import subprocess
from annotation_tools import NetworkTrafficAnnotationTools from annotation_tools import NetworkTrafficAnnotationTools
# If this test starts failing, please set TEST_IS_ENABLED to "False" and file a # If this test starts failing, please set TEST_IS_ENABLED to "False" and file a
# bug to get this reenabled, and cc the people listed in # bug to get this reenabled, and cc the people listed in
# //tools/traffic_annotation/OWNERS. # //tools/traffic_annotation/OWNERS.
TEST_IS_ENABLED = True TEST_IS_ENABLED = False
class TrafficAnnotationTestsChecker(): class TrafficAnnotationTestsChecker():
...@@ -44,13 +43,6 @@ class TrafficAnnotationTestsChecker(): ...@@ -44,13 +43,6 @@ class TrafficAnnotationTestsChecker():
print(stderr_text) print(stderr_text)
return return_code return return_code
# TODO(rhalavati): Remove this code.
def RunDiagnostics(build_path):
subprocess.call(["ls", os.path.join(build_path, "gen")])
print("---")
subprocess.call(["ls", os.path.join(build_path,
"gen/net/data/ssl/certificates")])
def main(): def main():
if not TEST_IS_ENABLED: if not TEST_IS_ENABLED:
...@@ -65,9 +57,6 @@ def main(): ...@@ -65,9 +57,6 @@ def main():
'found.') 'found.')
args = parser.parse_args() args = parser.parse_args()
RunDiagnostics(args.build_path)
return 0
checker = TrafficAnnotationTestsChecker(args.build_path) checker = TrafficAnnotationTestsChecker(args.build_path)
return checker.RunAllTests() return checker.RunAllTests()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment