-
Notifications
You must be signed in to change notification settings - Fork 120
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
Update to skyhook 0.2.0 and fix failing workflow #532
Conversation
998094a
to
4c4afaa
Compare
@JayjeetAtGithub something is wrong in your test container, it is trying to build llvmlite from scratch. Please check this out. |
Hi @lgray , I am trying to reproduce and fix this issue locally already. It's taking longer than expected. |
No worries - let me know if there's anything I can do. |
Is this PR to be discarded for #545 ? |
executor_args = {"client": client, "ceph_config_path": "/etc/ceph/ceph.conf"} | ||
executor_args = { | ||
"client": client, | ||
"ceph_config_path": "/tmp/testradosparquetjob/ceph.conf", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JayjeetAtGithub with Skyhook 0.2.0 do we still need to have explicitly passing ceph.conf
as executor args?
At coffea-casa, we couldn't put ceph.conf
in Docker container (security) and we simply mounted Skyhook available through CephFS using kubernetes. Using ceph-fuse
is also not possible as well mount because we don't have root access in containers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closed 545 and merged the changes in this PR. It is required for the fixing of this PR. |
f15a61f
to
f085578
Compare
6145c9c
to
9aa09c9
Compare
No description provided.