Skip to content

Add docker container logs to artefacts on test failures#74

Merged
jannebe merged 2 commits into
mainfrom
serverlogs
Mar 6, 2026
Merged

Add docker container logs to artefacts on test failures#74
jannebe merged 2 commits into
mainfrom
serverlogs

Conversation

@jannebe

@jannebe jannebe commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable

@jannebe jannebe requested a review from Huulivoide March 5, 2026 12:06
@github-actions

github-actions Bot commented Mar 5, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@Huulivoide Huulivoide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Huulivoide reviewed 1 file and made 1 comment.
Reviewable status: all files reviewed (commit messages unreviewed), 1 unresolved discussion (waiting on jannebe).


.github/workflows/shared-run-e2e.yml line 286 at r1 (raw file):

      - name: Collect Docker container logs
        id: collect_docker_logs
        if: always() && steps.run_tests.outcome == 'failure'

Mää laittasin ihan vaan suoraan aina, vaikka testit onnistuiskin. Ei tarvi sitten koskaan lähtä muutteleen, jos vaikka jostain tarviikin, tai vaikka haluu esim verrata onnistuneen ajon lokeja epäonnistuneeseen.

@jannebe jannebe force-pushed the serverlogs branch 8 times, most recently from a67be58 to 2e169ce Compare March 6, 2026 12:50

@culka culka left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@culka reviewed 2 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on jannebe).


.github/workflows/shared-run-e2e.yml line 311 at r3 (raw file):

          if [[ "${test_server_logs_exist}" == "true" ]]
          then
            find ${{ github.workspace }}/docker-logs -type f

Hetken ihmettelin mikä tämä on, mutta ilmeisesti tulostetaan lokiin vain mille palveluille on lokit? Jos haluaa, niin echo "Logs for services:" maininta aiemmin ei haittaisi.

@Huulivoide Huulivoide left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Huulivoide reviewed 2 files and all commit messages, and resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on jannebe).

@jannebe jannebe merged commit b12429a into main Mar 6, 2026
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants