Fix Docker Client crash #792

jpraynaud · 2023-03-09T11:11:30Z

Content

This PR includes:

A fix to the Docker Client image that crashes because of missing home directory for the appuser
An update to the documentation with Docker images usage in:
- Mithril Client Node developers doc
- Bootstrap a Cardano Node user guide

The Mithril Client multi-platform test GitHub action will be updated to also test the behavior of the Docker Client image.

Branch
- Commit sequence broadly makes sense
- Key commits have useful messages
PR
- No clippy warnings in the CI
- Self-reviewed the diff
- Useful pull request description
- Reviewer requested
Documentation
- Update documentation website (if relevant)

Closes #791

Fix home 'appuser' directory not existing but needed for snapshot digest cache.

github-actions · 2023-03-09T11:24:09Z

    3 files ±0   30 suites ±0 5m 19s ⏱️ -41s
446 tests ±0 446 ✔️ ±0 0 💤 ±0 0 ❌ ±0
516 runs ±0 516 ✔️ ±0 0 💤 ±0 0 ❌ ±0

Results for commit 61e7785. ± Comparison against base commit 8fd902c.

Fix Docker CI client image

68abea7

Fix home 'appuser' directory not existing but needed for snapshot digest cache.

jpraynaud requested a review from ghubertpalo March 9, 2023 11:11

Add Docker Mithril Client usage in docs

61e7785

jpraynaud force-pushed the jpraynaud/791-fix-docker-client-crash branch from a715dda to 61e7785 Compare March 9, 2023 11:13

jpraynaud temporarily deployed to testing-preview March 9, 2023 11:24 — with GitHub Actions Inactive

ghubertpalo approved these changes Mar 9, 2023

View reviewed changes

jpraynaud merged commit 7a871b2 into main Mar 9, 2023

jpraynaud deleted the jpraynaud/791-fix-docker-client-crash branch March 9, 2023 13:27

jpraynaud self-assigned this Mar 9, 2023