Skip to content
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

fix permission issues when copying xvfb-run script in Xvfb easyconfigs #18834

Merged
merged 1 commit into from Sep 20, 2023

Conversation

boegel
Copy link
Member

@boegel boegel commented Sep 19, 2023

(created using eb --new-pr)

This fixes problems with the cp -a command in certain setups, in particular in the EESSI build environment (see also EESSI/software-layer#299):

cp: preserving permissions for '/cvmfs/pilot.eessi-hpc.org/versions/2023.06/software/linux/aarch64/neoverse_n1/software/Xvfb/1.20.11-GCCcore-10.3.0/bin/xvfb-run': Permission denied

@boegel boegel added this to the next release (4.8.2?) milestone Sep 19, 2023
@boegel boegel added bug fix EESSI Related to EESSI project labels Sep 19, 2023
@boegel
Copy link
Member Author

boegel commented Sep 19, 2023

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

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

PR test command 'EB_PR=18834 EB_ARGS= EB_CONTAINER= EB_REPO=easybuild-easyconfigs /opt/software/slurm/bin/sbatch --job-name test_PR_18834 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 1725549256 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 9 out of 9 (9 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/boegelbot/a7d02f1ae6a5bcda96af9d1fc9c49187 for a full test report.

@ocaisa
Copy link
Member

ocaisa commented Sep 19, 2023

@boegelbot please test @ jsc-zen2

@boegelbot
Copy link
Collaborator

@ocaisa: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster

PR test command 'EB_PR=18834 EB_ARGS= EB_REPO=easybuild-easyconfigs /opt/software/slurm/bin/sbatch --mem-per-cpu=4000M --job-name test_PR_18834 --ntasks=8 ~/boegelbot/eb_from_pr_upload_jsc-zen2.sh' executed!

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

Test results coming soon (I hope)...

- notification for comment with ID 1725712073 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).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
FAILED
Build succeeded for 9 out of 11 (9 easyconfigs in total)
jsczen2g1.int.jsc-zen2.easybuild-test.cluster - Linux Rocky Linux 8.5, x86_64, AMD EPYC 7742 64-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/boegelbot/abde9fc517698175a4d3d1f566fb1310 for a full test report.

@ocaisa
Copy link
Member

ocaisa commented Sep 20, 2023

The failing build on jsczen2 was for python and not related to this PR, merging

@ocaisa ocaisa merged commit 97e0c42 into easybuilders:develop Sep 20, 2023
5 checks passed
@boegel boegel deleted the 20230919151302_new_pr_Xvfb12011 branch September 20, 2023 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix EESSI Related to EESSI project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants