-
Notifications
You must be signed in to change notification settings - Fork 286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: fix wsl2 lsmem: This system does not support memory blocks #556
Labels
bug
Something isn't working
good-first-issue
Good for newcomers
Hacktoberfest
Hacktoberfest participants are welcome to take a stab at issues marked with this label.
Milestone
Comments
petermetz
added
bug
Something isn't working
good-first-issue
Good for newcomers
Hacktoberfest
Hacktoberfest participants are welcome to take a stab at issues marked with this label.
labels
Feb 8, 2021
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
Mar 24, 2021
…er-cacti#556 Added a fallback || true suffix to the lsmem, lscpu and smem commands which are used by the ci.sh script to dump diagnostic information about the hardware of the CI runner prior to actually proceeding with the execution of the full build+test suite. The existing guards examining the presence of the lsmem binary were not enough because the lsmem binary is indeed present on WSL 2 Ubuntu, it just does not work as intended (exit code is non-zero which crashes the ci.sh script by design) Fixes hyperledger-cacti#556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
Mar 25, 2021
…er-cacti#556 Added a fallback || true suffix to the lsmem, lscpu and smem commands which are used by the ci.sh script to dump diagnostic information about the hardware of the CI runner prior to actually proceeding with the execution of the full build+test suite. The existing guards examining the presence of the lsmem binary were not enough because the lsmem binary is indeed present on WSL 2 Ubuntu, it just does not work as intended (exit code is non-zero which crashes the ci.sh script by design) Fixes hyperledger-cacti#556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
Mar 26, 2021
…er-cacti#556 Added a fallback || true suffix to the lsmem, lscpu and smem commands which are used by the ci.sh script to dump diagnostic information about the hardware of the CI runner prior to actually proceeding with the execution of the full build+test suite. The existing guards examining the presence of the lsmem binary were not enough because the lsmem binary is indeed present on WSL 2 Ubuntu, it just does not work as intended (exit code is non-zero which crashes the ci.sh script by design) Fixes hyperledger-cacti#556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
Mar 30, 2021
…er-cacti#556 Added a fallback || true suffix to the lsmem, lscpu and smem commands which are used by the ci.sh script to dump diagnostic information about the hardware of the CI runner prior to actually proceeding with the execution of the full build+test suite. The existing guards examining the presence of the lsmem binary were not enough because the lsmem binary is indeed present on WSL 2 Ubuntu, it just does not work as intended (exit code is non-zero which crashes the ci.sh script by design) Fixes hyperledger-cacti#556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
petermetz
added a commit
that referenced
this issue
Mar 30, 2021
Added a fallback || true suffix to the lsmem, lscpu and smem commands which are used by the ci.sh script to dump diagnostic information about the hardware of the CI runner prior to actually proceeding with the execution of the full build+test suite. The existing guards examining the presence of the lsmem binary were not enough because the lsmem binary is indeed present on WSL 2 Ubuntu, it just does not work as intended (exit code is non-zero which crashes the ci.sh script by design) Fixes #556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
Apr 8, 2021
…er-cacti#556 Added a fallback || true suffix to the lsmem, lscpu and smem commands which are used by the ci.sh script to dump diagnostic information about the hardware of the CI runner prior to actually proceeding with the execution of the full build+test suite. The existing guards examining the presence of the lsmem binary were not enough because the lsmem binary is indeed present on WSL 2 Ubuntu, it just does not work as intended (exit code is non-zero which crashes the ci.sh script by design) Fixes hyperledger-cacti#556 Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
good-first-issue
Good for newcomers
Hacktoberfest
Hacktoberfest participants are welcome to take a stab at issues marked with this label.
Describe the bug
On WSL2 the
./tools/ci.sh
script fails.To Reproduce
./tools/ci.sh
Expected behavior
CI to not fail
Logs/Stack traces
Screenshots
N/A
Cloud provider or hardware configuration:
Development machine, WSL 2
Operating system name, version, build:
Windows 10, WSL2 -> Ubuntu 20.04 LTS
Hyperledger Cactus release version or commit (git rev-parse --short HEAD):
04ea8bb
Hyperledger Cactus Plugins/Connectors Used
N/A
Additional context
N/A
cc: @takeutak @sfuji822 @hartm @jonathan-m-hamilton @AzaharaC @jordigiam @kikoncuo
The text was updated successfully, but these errors were encountered: