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

Kubernetes Backend; Go Module Support #626

Merged
merged 23 commits into from
Mar 27, 2020
Merged

Kubernetes Backend; Go Module Support #626

merged 23 commits into from
Mar 27, 2020

Conversation

adamstruck
Copy link
Contributor

@adamstruck adamstruck commented Feb 20, 2020

For setup see: https://github.com/ohsu-comp-bio/funnel/blob/kube/deployments/kubernetes/README.md

closes #414

TODO:

  • More documentation
  • Map disk request to volume in job template

@adamstruck adamstruck requested a review from kellrott February 21, 2020 21:13
@adamstruck adamstruck changed the title Kubernetes Backend Kubernetes Backend; Module Support Feb 21, 2020
@adamstruck adamstruck changed the title Kubernetes Backend; Module Support Kubernetes Backend; Go Module Support Feb 21, 2020
@kellrott
Copy link
Contributor

Tested against K8S cluster built using https://github.com/ohsu-comp-bio/openstack_k8s
When submitting a job, Funnel set it to a 'Queued' state and never updated. Looking into the logs found:

{"error":"creating job: jobs.batch is forbidden: User \"system:serviceaccount:default:default\" cannot create resource \"jobs\" in API group \"batch\" in the namespace \"default\"","level":"error","msg":"error submitting task to compute backend","ns":"server","taskID":"bpq39a19jgp5u6qfrh70","time":"2020-03-20T03:12:40Z"}

This setting probably needs to be added to the documentation.

.goreleaser.yml Outdated Show resolved Hide resolved
@adamstruck
Copy link
Contributor Author

@kellrott I think this PR is ready for merge

@kellrott
Copy link
Contributor

It's been tested on multiple K8S setups, so looks good.

@kellrott kellrott merged commit 83f3fdf into master Mar 27, 2020
@adamstruck adamstruck deleted the kube branch March 27, 2020 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

compute: kubernetes support
3 participants