Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kubernetes: Support starting with no versions #7461

Merged
merged 3 commits into from
Sep 12, 2024

Conversation

mook-as
Copy link
Contributor

@mook-as mook-as commented Sep 11, 2024

If we had no internet access on first start and fail to download the list of k3s versions, force disable Kubernetes instead of failing.

This also checks the k3s channel response harder, so that if we got a success code but some JSON that didn't include the correct contents, we'll reject it correctly.

Fixes #7412

On macOS, this is easier to test with echo nameserver 0.0.0.0 | sudo tee /etc/resolver/update.k3s.io before factory resetting.

Check that the channels list response contains the expected `responseType`,
so that if we got back invalid data we don't try to use it and end up
clearing the cached list.

Signed-off-by: Mark Yen <[email protected]>
We were accidentally sending the GitHub token to the non-GitHub host.
This should be typically unset, but that's still kind of bad.

Signed-off-by: Mark Yen <[email protected]>
If we had no internet access on first start and fail to download the list
of k3s versions, force disable Kubernetes instead of failing.

Signed-off-by: Mark Yen <[email protected]>
@jandubois jandubois self-requested a review September 12, 2024 00:09
Copy link
Member

@jandubois jandubois left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.

I could not however reproduce the error dialog from #7412, so cannot verify that this PR actually fixes the issue or not. Given that it traps the error in a try block, it seems likely that it will though.

@jandubois jandubois merged commit dcc9841 into rancher-sandbox:main Sep 12, 2024
17 checks passed
@mook-as mook-as deleted the k3s/channel-server-dependency branch October 3, 2024 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

K8s unable to start due to issue with update server
2 participants