Skip to content
This repository was archived by the owner on Feb 15, 2020. It is now read-only.

Conversation

fbertsch
Copy link
Contributor

@fbertsch fbertsch commented Apr 5, 2018

Previously, executor logs would fail with:
java.io.FileNotFoundException: /mnt/var/log/spark/spark.log (Permission denied)

Which caused the executor logs to not write.

This change will make them available, and then available in s3.

Previously, executor logs would fail with:
java.io.FileNotFoundException: /mnt/var/log/spark/spark.log (Permission denied)

Which caused the executor logs to not write.

This change will make them available, and then available in s3.
@fbertsch fbertsch requested a review from mreid-moz April 5, 2018 17:32
@acmiyaguchi
Copy link
Contributor

This will close bug 1329779, which is related to bug 1335228.

@jezdez
Copy link
Contributor

jezdez commented Apr 5, 2018

@acmiyaguchi Can we close #61 then?

Copy link
Contributor

@acmiyaguchi acmiyaguchi left a comment

Choose a reason for hiding this comment

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

This looks good, I compared the test cluster logs to a random cluster in the EMR list, and it looks like logs are showing up on executors now. Looks like it's working as expected.

@jezdez It looks like I called closing those bugs too early, this solves a different problem. In the linked cluster, the spark-history-server.out.gz has the same error message:

log4j:ERROR setFile(null,true) call failed. java.io.FileNotFoundException: /mnt/var/log/spark/spark.log (Permission denied)

@fbertsch
Copy link
Contributor Author

fbertsch commented Apr 5, 2018

I'm about 99.99% sure that if we remove the "log4j.appender.FILE.File": "/mnt/var/log/spark/spark.log", line from configuration.json then the history server logs will be available. The downside to this is the spark logs won't be available in spark.log -- I'm not actually sure where they are output by default (it could just be within the spark container logs, which isn't actually an issue, since those are also saved to s3).

@fbertsch fbertsch merged commit 7e32c3b into mozilla:master Apr 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants