conjure-up / spells

conjure-up spells registry
79 stars 37 forks source link

Add cdk-nvidia spell #159

Closed kwmonroe closed 6 years ago

kwmonroe commented 6 years ago

This is a monster, but it's mostly duplicating the cdk spell as cdk-nvidia. In fact, the only real meat is:

--- a/canonical-kubernetes-nvidia/metadata.yaml
+++ b/canonical-kubernetes-nvidia/metadata.yaml
@@ -1,6 +1,6 @@
 friendly-name: Canonical Distribution of Kubernetes with nVidia GPU workers
 version: 1
-bundle-name: canonical-kubernetes-nvidia
+bundle-name: ~containers/canonical-kubernetes-nvidia

To test this in conjure-up, do:

$ git clone -b feature/cdk-nvidia-separate-bundle https://github.com/juju-solutions/spells.git /tmp/myspells
$ conjure-up --nosync --spells-dir=/tmp/myspells canonical-kubernetes-nvidia
ktsakalozos commented 6 years ago

LGTM

kwmonroe commented 6 years ago

screen shot 2018-03-16 at 2 31 27 pm

I think we're ready.. conjure-up 2.5.5 on Darwin made it all the way to completion with this spell.

kwmonroe commented 6 years ago

wait! i thought wrong.. need to set an extra arg on the worker. don't you dare merge this yet; new test running now.

kwmonroe commented 6 years ago

Good to go for real this time.. Tested this spell with 2.5.5 on AWS. p2.xlarge worker came up fine and the nvidia daemonset was deployed successfully.