Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Oct 11, 2022

(created using eb --new-pr)

Without this fix, only the user account used to install NextGenMap can run the ngm command...

$ ls -ld /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm*
-rwxr-xr-- 1 boegel users      483 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm
-rwxr-xr-x 1 boegel users  9053168 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm-core
-rwxr-xr-x 1 boegel users  7636096 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm-core-debug
-rwxr-xr-- 1 boegel users      483 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm-debug
-rwxr-xr-- 1 boegel users      472 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm-log
-rwxr-xr-x 1 boegel users 10278608 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm-utils
-rwxr-xr-x 1 boegel users  4211448 Oct 11 09:49 /software/NextGenMap/0.5.5-GCC-11.2.0/bin/ngm-utils-debug

@boegel boegel added the bug fix label Oct 11, 2022
@boegel boegel added this to the next release (4.6.2?) milestone Oct 11, 2022
@boegel
Copy link
Member Author

boegel commented Oct 11, 2022

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@boegel: Request for testing this PR well received on login1

PR test command 'EB_PR=16383 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_16383 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 9264

Test results coming soon (I hope)...

Details

- notification for comment with ID 1274347714 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegel
Copy link
Member Author

boegel commented Oct 11, 2022

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3102.skitty.os - Linux RHEL 8.4, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz (skylake_avx512), Python 3.6.8
See https://gist.github.com/f9e859396b5f5a7d71c8d996f010cd03 for a full test report.

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/0b9f4f9786c34b6f375dd381d2a81d6c for a full test report.

@casparvl
Copy link
Contributor

casparvl commented Oct 11, 2022

(removed test report, forgot to run with --robot)

1 similar comment
@casparvl
Copy link
Contributor

casparvl commented Oct 11, 2022

(removed test report, forgot to run with --robot)

@casparvl
Copy link
Contributor

casparvl commented Oct 11, 2022

(removed test report, due to checksum error of locally cached source)

@casparvl
Copy link
Contributor

Test report by @casparvl
SUCCESS
Build succeeded for 2 out of 2 (1 easyconfigs in total)
software1.lisa.surfsara.nl - Linux debian 10.13, x86_64, Intel(R) Xeon(R) Silver 4110 CPU @ 2.10GHz, Python 3.7.3
See https://gist.github.com/9d5e7049d53c054c82d3cd5966b58990 for a full test report.

@casparvl
Copy link
Contributor

Test report by @casparvl
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
tcn1.local.snellius.surf.nl - Linux RHEL 8.4, x86_64, AMD EPYC 7H12 64-Core Processor, Python 3.6.8
See https://gist.github.com/cd02eb837e7e8d266bafa22096e6995e for a full test report.

@casparvl
Copy link
Contributor

ls -al /home/casparl/.local/easybuild/Debian10/2021/software/NextGenMap/0.5.5-GCC-11.2.0/bin/
total 30424
drwxr-sr-x 3 casparl casparl      196 Oct 11 15:31 .
drwxr-sr-x 4 casparl casparl       46 Oct 11 15:31 ..
-rwxr-xr-x 1 casparl casparl      483 Oct 11 15:31 ngm
-rwxr-xr-x 1 casparl casparl  9030536 Oct 11 15:31 ngm-core
-rwxr-xr-x 1 casparl casparl  7620616 Oct 11 15:31 ngm-core-debug
-rwxr-xr-x 1 casparl casparl      483 Oct 11 15:31 ngm-debug
-rwxr-xr-x 1 casparl casparl      472 Oct 11 15:31 ngm-log
-rwxr-xr-x 1 casparl casparl 10261920 Oct 11 15:31 ngm-utils
-rwxr-xr-x 1 casparl casparl  4199552 Oct 11 15:31 ngm-utils-debug
-rwxr-xr-x 1 casparl casparl    18176 Oct 11 15:31 oclTool
drwxr-sr-x 4 casparl casparl       43 Oct 11 15:31 opencl

Copy link
Contributor

@casparvl casparvl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, the permission on the ngm binaries are now ok

@casparvl casparvl merged commit c37b792 into easybuilders:develop Oct 11, 2022
@boegel boegel deleted the 20221011095452_new_pr_NextGenMap055 branch October 11, 2022 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants