=== tds1 is now known as tds
=== tds1 is now known as tds
[08:47] <caribou> Hello everyone, before filing a bug on this, I'd like your opinion : when setting *disable_root: false*, shouldn't *disable_root_opts
[08:47] <caribou> be disabled as well ?
[08:48] <caribou> as if one wants root login enabled, then disable_root_opts should not fill /root/.ssh/authorized_keys with blocking options
[08:49] <caribou> I'd be happy to submit a patch for this
[09:35] <meena> caribou: generally, disable_root disables password login for root, not the entire account. that means that you deffo want ssh keys
[09:43] <caribou> ah, right fine then
[09:44] <caribou> thanks for the precision
[13:35] <Odd_Bloke> caribou: There's actually a recently-filed bug for that, let me see if I can find it.
[13:36] <Odd_Bloke> caribou: Does https://bugs.launchpad.net/cloud-init/+bug/1871879 look like what you're asking for?
[13:36] <ubot5> Ubuntu bug 1871879 in cloud-init "Configuring a user should not configure root's authorized_keys" [Wishlist,Triaged]
[13:36] <Odd_Bloke> Oh, no, I think that's actually the opposite of what you're asking, my mistake.
[13:37] <Odd_Bloke> (It's too early for me to process the double negative of "disable_root: false". ;)
[13:38] <Odd_Bloke> So https://cloudinit.readthedocs.io/en/latest/topics/modules.html#authorized-keys is the documentation we have for the option, and it's not particularly clear IMO.
[13:40] <Odd_Bloke> caribou: And, actually, reading the code, I believe that `disable_root: false` should lead to `disable_root_opts` being ignored.
[13:40] <caribou> @Odd_Bloke actually, with meena's precision, I don't even need to use disable_root
[13:41] <caribou> yeah, that's what I meant, just move the opt inside the 'if'
[13:41] <caribou> but if disable_root is only for disabling pwd login, then it may still be usefull to have that entry in authorized_keys
[13:44] <Odd_Bloke> caribou: I've just set up a cloud-config with an `authorized_keys:` section and `disable_root: false` and it works as I would expect (disable_root_opts is not applied).  If that isn't what you're seeing, could you file a bug please?
[13:45] <caribou> sure, will check htat
[13:45] <caribou> thanks for looking it up
[17:29] <punkgeek> how can i import a shell script to vm by cloud init?
[17:30] <Odd_Bloke> punkgeek: Could you explain what you mean by that?  If you want cloud-init to execute a shell script, you can either specify the script as your entire user-data (with an appropriate shebang) or use the runcmd cloud-config option: https://cloudinit.readthedocs.io/en/latest/topics/modules.html#runcmd.  If you want to write a shell script, then you probably want to take a look at
[17:30] <Odd_Bloke> https://cloudinit.readthedocs.io/en/latest/topics/modules.html#write-files.
[17:30] <Odd_Bloke> (If you want something else, let us know. :)
[17:32] <punkgeek> Thank you that was what i want
[17:33] <Odd_Bloke> Great! :)
[20:33] <blackboxsw> at long last paride https://github.com/canonical/cloud-init/pull/231#pullrequestreview-397636926
[20:36] <blackboxsw> Goneri thanks for the review updates on your PRs, grabbing that today and hopefully we can close out there
[20:42] <Goneri> Yeah!