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

Add KB about how to shutdown a Harvester cluster #68

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

w13915984028
Copy link
Member

Copy link

netlify bot commented Jul 22, 2024

Deploy Preview for harvester-home-preview ready!

Name Link
🔨 Latest commit 5bfec6f
🔍 Latest deploy log https://app.netlify.com/sites/harvester-home-preview/deploys/66d19fe72680b80008adc490
😎 Deploy Preview https://deploy-preview-68--harvester-home-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@bk201 bk201 left a comment

Choose a reason for hiding this comment

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

Thanks for the doc, I have some questions and suggestions.

kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
@w13915984028 w13915984028 force-pushed the doc4451 branch 11 times, most recently from 23bf437 to 9c9d9c2 Compare July 24, 2024 16:26
@w13915984028 w13915984028 force-pushed the doc4451 branch 6 times, most recently from 6deea38 to 63dc905 Compare July 24, 2024 17:10
@w13915984028 w13915984028 force-pushed the doc4451 branch 8 times, most recently from 36e6663 to 41b427d Compare August 12, 2024 09:27
@w13915984028 w13915984028 force-pushed the doc4451 branch 6 times, most recently from 616dc39 to 3e6a1c9 Compare August 14, 2024 16:51
@asettle
Copy link
Contributor

asettle commented Aug 15, 2024

@w13915984028 maybe I'm misunderstanding, but why is this a knowledge base article? I appreciate that it's helpful content, but realistically, this is a standard procedure that should go into the general documentation.

@w13915984028
Copy link
Member Author

@w13915984028 maybe I'm misunderstanding, but why is this a knowledge base article? I appreciate that it's helpful content, but realistically, this is a standard procedure that should go into the general documentation.

@asettle Thanks.
At the begging, customers requires for a guide to fulfill the task. A KB was OK then.
To now, we can consider to move to document as well. cc @jillian-maroket.

kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved

:::note

Harvester has an `embedded rancher` deployment which is used to help the lifecycle management of Harvester itself, it is different from the independently deployed **Rancher manager** for multi-cluster management and more.
Copy link
Member

Choose a reason for hiding this comment

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

To keep consistency with the wording in the following text below.

Suggested change
Harvester has an `embedded rancher` deployment which is used to help the lifecycle management of Harvester itself, it is different from the independently deployed **Rancher manager** for multi-cluster management and more.
Harvester has an embedded `rancher` deployment which is used to help the lifecycle management of Harvester itself, it is different from the independently deployed **Rancher manager** for multi-cluster management and more.

Copy link
Member Author

Choose a reason for hiding this comment

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

As this operation also touches the external Rancher server, which is referred as Rancher manager; to distinguish from it as explicitily as possbile, the Harvester's internal rancher is called and highlighted as embedded rancher.


:::

The `cattle-cluster-agent-***` pod is the [direct connection between Rancher manager and Harvester cluster](https://docs.harvesterhci.io/v1.3/rancher/virtualization-management#importing-harvester-cluster), and this pod is monitored and managed by the `embedded rancher` in Harvester, scaling down this pod does not work. The `embedded rancher` will scale it up automatically.
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure how to handle the embedded rancher wording here. Is it about the deployment or the embedded Rancher product. i think the answer influences how embedded rancher is to be treated/adapted in the following course of the text.

kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved

#### Restart the Leader Control-plane Node

Power on the [last shutdown node](#43-shutdown-the-last-control-plane-node) first. After about three minutes, continue the next step.
Copy link
Member

Choose a reason for hiding this comment

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

After about three minutes, continue the next step.

Does it make sense to use a time specification here? I think that the user's hardware is decisive in determining how long it takes. Is there not a possibility to execute a CLI command to check the status?


:::note

If the `embedded rancher` was not scaled down before, this step can also be:
Copy link
Member

Choose a reason for hiding this comment

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

Based on the previous decision (see above), this should also be adjusted here.
E.g. embedded Rancher or embedded rancher, ...

kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
kb/2024-07-22/harvester_cluster_shutdown_and_restart.md Outdated Show resolved Hide resolved
@jillian-maroket
Copy link
Contributor

@w13915984028 FYI: Instead of publishing the text as a KB article, I will add the information to the doc per @asettle 's feedback. I'll tag you as reviewer when the doc files are ready for review. cc: @bk201

@w13915984028
Copy link
Member Author

To reviewer:

This KB is still actively updating. One customer will follow this KB to relocate a production cluster.

We try best to avoid any potential risks.

Besides the formattings, grammers, please also help think&check more about the real operations steps, any was forgoten/missing..., thanks.

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.

6 participants