Skip to content

Commit 6ff02c0

Browse files
authored
ci: ignore errors on log collection (#777)
Ignore errors on log collection to ensure we capture as many log files as possible. Avoid problems such as what happened in https://github.com/oxidecomputer/terraform-provider-oxide/actions/runs/27942817765/job/82680160718, where we can't really tell what went wrong.
1 parent 4b64ec0 commit 6ff02c0

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

.github/workflows/acceptance-sim.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -114,12 +114,13 @@ jobs:
114114
if: always()
115115
continue-on-error: true
116116
working-directory: acctest
117+
# Ignore command errors to collect as many logs as possible.
117118
run: |
118-
docker compose exec -T omicron-dev sh -c 'tar -cf - /tmp/omicron-dev*.log' | tar -xf -
119-
docker compose exec -T mitmproxy sh -c 'tar -cf - /tmp/mitmproxy.log' | tar -xf -
120-
docker compose logs omicron-dev > ./tmp/omicron-dev.log
121-
find .. -name terraform.log -exec cat {} \; > ./tmp/terraform.log
122-
cp ./acc-tests.xml ./tmp
119+
docker compose exec -T omicron-dev sh -c 'tar -cf - /tmp/omicron-dev*.log' | tar -xf - || true
120+
docker compose exec -T mitmproxy sh -c 'tar -cf - /tmp/mitmproxy.log' | tar -xf - || true
121+
docker compose logs omicron-dev > ./tmp/omicron-dev.log || true
122+
find .. -name terraform.log -exec cat {} \; > ./tmp/terraform.log || true
123+
cp ./acc-tests.xml ./tmp || true
123124
- name: rerun warn
124125
if: always()
125126
continue-on-error: true

0 commit comments

Comments
 (0)