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

Use rancher/wharfie code shared with k3s for for bootstrap #971

Merged
merged 1 commit into from
May 13, 2021

Conversation

brandond
Copy link
Member

Proposed Changes

  • Remove bootstrap code that has been moved to rancher/wharfie
  • Support kubelet image credential plugins (both for bootstrap pull, and for kubelet)
  • Use --system-default-registry flag from k3s

Removal of --system-default-registry from agent flags should be called out in release notes, and removed from docs.

Types of Changes

  • Bootstrap
  • Private registry

Verification

  • Set --system-default-registry on server, note that agents use same setting
  • Configure kubelet image credential plugins, note that bootstrap image and core images can be pulled from ECR if ECR is specified as --system-default-registry

Linked Issues

For #958 and #959

Further Comments

@brandond brandond changed the title Feat 958 Use rancher/wharfie code shared with k3s for for bootstrap May 11, 2021
@brandond brandond marked this pull request as ready for review May 12, 2021 01:31
@brandond brandond requested a review from a team as a code owner May 12, 2021 01:31
Moves the runtime image bootstrap to after config has been pulled from
the server, so that we can promote --system-default-registry to a
server var, like --cluster-dns and other things that should be set the
same on all nodes.

With this change we no longer drop manifests for packaged components on
agents, which never made use of them anyway.
@brandond brandond merged commit e6ae167 into rancher:master May 13, 2021
@MonzElmasry
Copy link
Contributor

Includes a fix for #1019

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.

4 participants