Skip to content

Conversation

@thebsdbox
Copy link
Contributor

Description

This adds a new updates LTS Kernel

Why is this needed

The older kernel was a little long in the tooth.

Fixes: #

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

No impact.

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

@thebsdbox thebsdbox requested review from detiber and nshalman August 11, 2021 13:22
Copy link
Member

@nshalman nshalman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. DCO check is unhappy and it looks like there are merge conflicts. Happy to re-approve once that's all cleaned up. Thanks!

@detiber
Copy link
Contributor

detiber commented Aug 11, 2021

lgtm, should there also be an updated config for aarch64, though?

@damdo
Copy link

damdo commented Aug 12, 2021

Great work!
Do we also want to update hook_debug.yaml?

@thebsdbox
Copy link
Contributor Author

Yeah, I was hoping to do the rest today but it's been a crazy day. I'll try and get it sorted tomorrow.

@tstromberg
Copy link
Contributor

@thebsdbox - Do you have time to take a look at this? It's so close! :)

@thebsdbox
Copy link
Contributor Author

Fixed aarch64 and hook_debug.yaml

make dist completes successfully and images exist on quay.io !

@nshalman nshalman added the ready-to-merge Signal to Mergify to merge the PR. label Sep 3, 2021
@thebsdbox thebsdbox merged commit cc08a3f into tinkerbell:main Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Signal to Mergify to merge the PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants