Specify GRUB2 kernel by default when deploying via Linode

[rmcintosh]

This ties to ansible/ansible#23100 which was submitted to help ease the issues described in #500

The change makes the deployed instance use the GRUB2 Linode kernel option which now allows use of the distribution's supplied kernel, rather than the hypervisor's kernel.

I'm not intimately familiar with streisand so If I've missed the mark somewhere, please let me know.

[jlund]

This is fantastic! Thanks for making this happen.

We will need to wait until the kernel_id argument is in a released version of Ansible before we can merge this PR, but it looks like that will be happening soon. At that point we can change the required Ansible version here and

[cpu]

@zx2c4 I might be confused, but it looked like the relevant commit was in the 425 commits to devel after the tag. Did you find release notes for v.2.3.0.0-1 that mention the required commit?

[rmcintosh]

For anyone not following along on both issues: Ansible change won't see the light of day until the 2.4 release, I believe.

…

On Sat, Apr 15, 2017 at 12:35 Daniel McCarney ***@***.***> wrote: @zx2c4 <https://github.com/zx2c4> I might be confused, but it looked like the relevant commit was in the 425 commits to devel after the tag <ansible/ansible@v2.3.0.0-1...devel>. Did you find release notes for v.2.3.0.0-1 that mention the required commit? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#574 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACBrDTK-HDac9OIewE8E1wf6uDuSHJeKks5rwPHUgaJpZM4MzOC_> .

[cpu]

When this is merged we should also revert fc73677, the workaround from #609

[rmcintosh]

@jlund for this (once the ansible change goes through), it seems like we should we just hardcode the kernelid as in #613, yeah?

[zx2c4]

I think you have the right idea here. I've submitted a PR -- #623 -- for making the distro id more like how you do it in this PR.

[rmcintosh]

fair enough, thanks @zx2c4! I'll match up after it's merged

[cpu]

I think the Ansible release we were waiting for on this PR is out. Might be time to knock the cobwebs off and see if we can land this and retire a monkey patch :-)

[rmcintosh]

nice - I will do that

thanks for the heads up!

[rmcintosh]

the conflicts here are annoying to fix so I'm just going to re-PR this since it is not a large change

[cpu]

Thanks for sticking with this one @rmcintosh :-) Nice to see a ticket from April 4th marked closed! 🎉 🎉 🎉

[rmcintosh]

Happy to have helped! ❤️