Skip to content

change permissions of the Java build directory to avoid permission denied error#2698

Merged
boegel merged 1 commit intoeasybuilders:developfrom
branfosj:20220321125030_new_pr_java
Mar 28, 2022
Merged

change permissions of the Java build directory to avoid permission denied error#2698
boegel merged 1 commit intoeasybuilders:developfrom
branfosj:20220321125030_new_pr_java

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Mar 21, 2022 •
edited
Loading

(created using eb --new-pr)

I've hit a permission denied error when installing Java. Making this permission change fixed it for me.

@sassy-crick
Copy link
Copy Markdown
Contributor

sassy-crick commented Mar 21, 2022

To add: I hit the same problem on our GPFS file system and that change fixed it for me as well.
Happy to provide the log-file if required. Either way: thanks!

bartoldeman
bartoldeman previously requested changes Mar 23, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@bartoldeman bartoldeman left a comment

Choose a reason for hiding this comment

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

I'm just curious how to reproduce this, with which Java version, and what files are problematic.
Perhaps this is something that can be fixed more generally in the framework?

@branfosj
Copy link
Copy Markdown
Member Author

branfosj commented Mar 23, 2022

I'm just curious how to reproduce this, with which Java version, and what files are problematic. Perhaps this is something that can be fixed more generally in the framework?

I've seen the problem this fixes with Java-11.0.2.eb and that was the version that @sassy-crick was having issues with. However, I've just tested on the system where I previously saw the issue and I had no issues building any of the Java versions I tried to build (Java-13.0.2.eb Java-15.0.1.eb Java-16.0.1.eb Java-17.0.1.eb Java-17.0.2.eb Java-1.8.0_292-OpenJDK.eb Java-11.0.2.eb). So I am unsure what caused the issue or why I no longer see it.

@bartoldeman
Copy link
Copy Markdown
Contributor

bartoldeman commented Mar 25, 2022

@sassy-crick can you please provide that logfile then?

@boegel boegel added the bug fix label Mar 25, 2022
@boegel boegel added this to the next release (4.5.4?) milestone Mar 25, 2022
@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 25, 2022

Merging this as-is should be fine, I don't see how it could cause trouble, but getting more info on the problem would indeed be nice, I don't remember anyone reporting this up until now...

@sassy-crick
Copy link
Copy Markdown
Contributor

sassy-crick commented Mar 25, 2022 •
edited
Loading

@bartoldeman Here are the two log files from the Bazel and Java builds which failed. I hope that helps.
Bazel-3.7.2-20220322.log
Java-11.0.2-20220321.log

@boegel
Copy link
Copy Markdown
Member

boegel commented Mar 28, 2022

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Java-17.0.2.eb
  • SUCCESS Java-16.0.1.eb
  • SUCCESS Java-15.0.1.eb
  • SUCCESS Java-13.0.2.eb
  • SUCCESS Java-11.0.2.eb
  • SUCCESS Java-1.8.0_311.eb
  • SUCCESS Java-17.eb
  • SUCCESS Java-16.eb
  • SUCCESS Java-15.eb
  • SUCCESS Java-13.eb
  • SUCCESS Java-11.eb
  • SUCCESS Java-1.8.eb

Build succeeded for 12 out of 12 (6 easyconfigs in total)
node2606.swalot.os - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz (haswell), Python 3.6.8
See https://gist.github.com/ea4e1e2211e0cbe786058e9232659c7f for a full test report.

@boegel boegel dismissed bartoldeman’s stale review March 28, 2022 09:48

extra info provided

boegel
boegel approved these changes Mar 28, 2022
View reviewed changes
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel merged commit 4241314 into easybuilders:develop Mar 28, 2022
@branfosj branfosj deleted the 20220321125030_new_pr_java branch March 28, 2022 09:50
@boegel boegel changed the title Change permissions of the Java builddir to avoid permission denied error change permissions of the Java build directory to avoid permission denied error Mar 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

@bartoldeman bartoldeman bartoldeman left review comments

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.5.4

Development

Successfully merging this pull request may close these issues.

4 participants

@branfosj @sassy-crick @bartoldeman @boegel