Install script doesn't seem to install the dotrun snap about dotrun HOT 9 CLOSED

@jkfran any thoughts on this one?

from dotrun.

@sowasred2012 to pair with @bartaz to check the issue

from dotrun.

Hey, sorry I couldn't check on this. I believe the issue might be on this line:

https://github.com/canonical-web-and-design/dotrun/blob/main/scripts/cloud-init-dotrun.yaml#L7

It's probably because of the pipe | bash. @Saviq do you know if using curl ... | bash on cloud-init scripts is not possible?

from dotrun.

Hey all, as I let @jkfran know, the problem is the | bash construct, which assumes a shell interpreting the command - which it doesn't, by default.

There's two solutions I can think of:

1. be explicit about the shell (sh -c "wget … | bash")
2. do it in two steps: wget …; bash install-dotrun-docker.sh

Also, /var/run/cloud-init*log should have helpful errors. We want to surface them on launch (canonical/multipass#1039), but didn't get to it yet.

from dotrun.

Ah, yes, I'm on zsh, so this gonna be it.

from dotrun.

Ah, yes, I'm on zsh, so this gonna be it.

No, it's not that. It's the fact that cloud-init executes the commands directly, there just isn't a shell anywhere in there:

https://cloudinit.readthedocs.io/en/latest/topics/modules.html#runcmd

Another (worse, IMO) option is to have it all as a string, so that cloud-init does actually use sh to interpret them:

runcmd: |
  wget … | bash
  mkdir …

from dotrun.

I'm reopening this, I just used script to install dotrun on new Mac and dotrun snap was not installed in multipass.

from dotrun.

@jkfran any thoughts?

from dotrun.

Hey @bartaz could you QA this one #83

from dotrun.