Attention is currently required from: dexter. laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/38254?usp=email )
Change subject: tests: sanitize all cards before running tests ...................................................................... Patch Set 1: (2 comments) Patchset: PS1: Do we have some information how much time the restore of all cards takes? File tests/card_sanitizer/restore.script: https://gerrit.osmocom.org/c/pysim/+/38254/comment/70dea27c_d511b702?usp=email : PS1, Line 3: ru I don't like this approach with a global file in /var/tmp. What if for some reason there's two jobs running by accident? what if somebody else can write to that file? If you must use a temp file, I would suggest it to be created with mkstem(3) or mktemp(1) or some python equivalent thereof. But wouldn't it be much simpler to introduce something like a @-e@ command line argument to pySim-shell, which could be used multiple times to specify multiple commands? Then you could do `pySim-shell -e verify_adm -e run_script /my/path/to/full.script > /dev/null` See https://gerrit.osmocom.org/c/pysim/+/38255 -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38254?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I42eaf61280968518164f2280245136fd30a603ce Gerrit-Change-Number: 38254 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: laforge <[email protected]> Gerrit-Attention: dexter <[email protected]> Gerrit-Comment-Date: Mon, 23 Sep 2024 09:55:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
