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

Update install docs to reflect http scheme requirement when adding new nodes to pre-existing cluster #659

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

drewmullen
Copy link

@drewmullen drewmullen commented Oct 28, 2024

This small change would have saved me quite a bit of heart ache...

If anyone can post a good pic of this screen I can include in the PR

@drewmullen drewmullen changed the title Update iso-install.md Update install docs to reflect http scheme requirement when adding new nodes to pre-existing cluster Oct 28, 2024
Copy link

Name Link
🔨 Latest commit 4ce0a23
😎 Deploy Preview https://671fb9856826bf5b0d22b397--harvester-preview.netlify.app

@bk201 bk201 requested a review from starbops October 29, 2024 16:03
Copy link
Member

@starbops starbops left a comment

Choose a reason for hiding this comment

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

Thank you for enhancing the documentation! I cropped a screenshot, feel free to update the PR with it.

image


:::note
If using DHCP to configure the IP address, you need to configure a static MAC-to-IP address mapping on your DHCP server to have a persistent virtual IP (VIP), and the VIP must be unique.
:::

![config-virtual-ip.png](/img/v1.2/install/config-virtual-ip.png)

1. (Joining pre-existing cluster only) Configure with URL from pre-existing Harvester Cluster Management URL. Must include http scheme (ex: `https://x.x.x.x` if using IPs). Management URLs can be found on the management node after bootup.
Copy link
Member

Choose a reason for hiding this comment

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

If I recall correctly, it does not need to have the HTTP scheme included.

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.

2 participants