Bug #84299 | mtr: mem is pure flag / add /run/shm (contribution) | ||
---|---|---|---|
Submitted: | 21 Dec 2016 13:29 | Modified: | 21 Dec 2016 13:47 |
Reporter: | OCA Admin (OCA) | Email Updates: | |
Status: | Verified | Impact on me: | |
Category: | Tools: MTR / mysql-test-run | Severity: | S3 (Non-critical) |
Version: | 5.7 | OS: | Any |
Assigned to: | CPU Architecture: | Any | |
Tags: | Contribution |
[21 Dec 2016 13:29]
OCA Admin
[21 Dec 2016 13:29]
OCA Admin
Contribution submitted via Github - mtr: mem is pure flag / add /run/shm (*) Contribution by Daniel Black (Github grooverdan, mysql-server/pull/116#issuecomment-268423030): I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: git_patch_98864585.txt (text/plain), 2.37 KiB.
[21 Dec 2016 13:47]
MySQL Verification Team
Hello Daniel, Thank you for the report and contribution. Thanks, Umesh
[21 Dec 2016 14:13]
Bjørn Munch
See my comment to the pull request. I think this needs to be extended with a check that the chosen mem directory is not a softlink.
[22 Dec 2016 13:16]
OCA Admin
Contribution submitted via Github - bug 84299: Symlinks for mtr mem directory not supported (*) Contribution by Daniel Black (Github grooverdan, mysql-server/pull/117#issuecomment-268670322): I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: git_patch_99022504.txt (text/plain), 744 bytes.
[7 Apr 2017 0:22]
Daniel Black
mtr.patch
Attachment: mtr.patch (text/x-patch), 861 bytes.
[7 Apr 2017 0:24]
Daniel Black
Attached is a new version. It was pointed out that ./mtr --mem /tmp cannot differentiate between /tmp being a a tmp directory or a test suite to run. As such the revise patch enabled MTR_MEM=/xxx ./mtr --mem to take the MTR_MEM environment variable as the preferences location. Additionally it checks the default tmpfs locations aren't a symlink.