Attention is currently required from: pespin.

osmith has posted comments on this change by osmith. ( https://gerrit.osmocom.org/c/osmo-ci/+/40138?usp=email )

Change subject: ansible: build-hosts: add testenv-coredump-helper ......................................................................

Patch Set 2:

(2 comments)

File ansible/roles/testenv-coredump-helper/files/testenv-coredump-helper.py:

https://gerrit.osmocom.org/c/osmo-ci/+/40138/comment/cba2f35a_c80e1535?usp=e... : PS1, Line 4: # OS#6769: simple webserver to make Osmocom related coredumps available in LXCs

I'm really lacking a big chunk of descriptions here regarding what this script does, where it is spe […]

added a README.md answering those questions.

https://gerrit.osmocom.org/c/osmo-ci/+/40138/comment/33245c0f_236bf052?usp=e... : PS1, Line 66: # Put coredump into a temporary file and return it

this may be problematic if it ends up in tmpfs, a coredump can be veery large.

The temporary directory gets created in `/tmp`, e.g. `/tmp/tmptlrx_0jr`. The `/tmp` dir is not mounted as tmpfs on the build hosts. I was also curious about the max coredump size, it is set via `/etc/systemd/coredump.conf` and the default is 2GiB which should be reasonable. We can tweak it later if needed IMHO.