add that back and just check explicitly

pull/24414/head
Shane Smiskol 3 years ago
parent efc73ab4c1
commit 9c27d297dc
  1. 2
      .github/workflows/selfdrive_tests.yaml
  2. 3
      selfdrive/test/process_replay/test_processes.py

@ -314,7 +314,7 @@ jobs:
- name: Run replay
run: |
${{ env.RUN }} "scons -j$(nproc) && \
CI=1 coverage run selfdrive/test/process_replay/test_processes.py && \
FILEREADER_CACHE=1 CI=1 coverage run selfdrive/test/process_replay/test_processes.py && \
coverage xml"
- name: Upload reference logs
if: ${{ failure() && github.event_name == 'pull_request' && github.repository == 'commaai/openpilot' }}

@ -63,7 +63,8 @@ def test_process(cfg, lr, cmp_log_fn, ignore_fields=None, ignore_msgs=None):
cmp_log_path = cmp_log_fn if os.path.exists(cmp_log_fn) else BASE_URL + os.path.basename(cmp_log_fn)
cmp_log_msgs = list(LogReader(cmp_log_path))
print(os.environ)
if not len(cmp_log_msgs):
raise Exception("Skipping process replay, no log messages for ref commit")
log_msgs = replay_process(cfg, lr)

Loading…
Cancel
Save