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

[v2.7] Add docker install support for RKE1 custom cluster #168

Merged

Conversation

rishabhmsra
Copy link

With this PR we can pass docker version in cattle-config which can be used for installing docker on RKE1 custom cluster nodes.

In order to pass docker version, provide below field in the config:

rke1CustomClusterDockerInstall:
  installDockerURL: "https://releases.rancher.com/install-docker/<version-number>.sh"

This change is for the AMIs that doesn't have docker pre-installed and are used in docker and OS checks.

igomez06
igomez06 previously approved these changes May 23, 2024
Copy link

@igomez06 igomez06 left a comment

Choose a reason for hiding this comment

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

LGTM

slickwarren
slickwarren previously approved these changes May 23, 2024
Comment on lines 229 to 230
type RKE1CustomClusterDockerInstall struct {
InstallDockerURL string `json:"installDockerURL" yaml:"installDockerURL"`
Copy link
Contributor

Choose a reason for hiding this comment

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

same nit here

Copy link
Author

Choose a reason for hiding this comment

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

@slickwarren, I've moved it above config, can i get a quick review on this

Copy link

@igomez06 igomez06 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@caliskanugur caliskanugur left a comment

Choose a reason for hiding this comment

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

LGTM

@igomez06 igomez06 merged commit 8ceddcb into rancher:release/v2.7 Aug 7, 2024
1 check passed
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