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

doc: Add maintenance mode improvements #523

Merged
merged 1 commit into from
Sep 2, 2024

Conversation

votdev
Copy link
Member

@votdev votdev commented Mar 11, 2024

@harvesterhci-io-github-bot
Copy link
Collaborator

harvesterhci-io-github-bot commented Mar 11, 2024

Deploy Preview for harvester-preview ready!

Name Link
🔨 Latest commit bcb9f77
🔍 Latest deploy log https://app.netlify.com/sites/harvester-preview/deploys/6662ddc0079decfaa93e4e20
😎 Deploy Preview https://6662ddc0079decfaa93e4e20--harvester-preview.netlify.app

@votdev votdev force-pushed the issue_5069_improve_maintenance_mode branch from e543adf to 07162d1 Compare March 11, 2024 15:00
@votdev votdev force-pushed the issue_5069_improve_maintenance_mode branch from 07162d1 to 747db8d Compare March 19, 2024 15:14
@votdev votdev marked this pull request as ready for review March 27, 2024 11:11
@votdev votdev force-pushed the issue_5069_improve_maintenance_mode branch 2 times, most recently from 01f088b to 2e63c7d Compare June 4, 2024 07:30
Copy link
Member

@w13915984028 w13915984028 left a comment

Choose a reason for hiding this comment

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

Few questions and suggestions. thanks.

docs/host/host.md Outdated Show resolved Hide resolved
docs/host/host.md Outdated Show resolved Hide resolved
@votdev
Copy link
Member Author

votdev commented Jun 6, 2024

I am currently writing a HEP for that issue to clarify how the label and everything is named.

@votdev votdev force-pushed the issue_5069_improve_maintenance_mode branch from 2e63c7d to bcb9f77 Compare June 7, 2024 09:36
votdev added a commit to votdev/harvester that referenced this pull request Jul 3, 2024
Introduce the label `harvesterhci.io/maintain-mode-strategy` which can be added to a `VirtualMachine` resource to specify the behaviour of the VM in case of a node maintenance event. The following modes are supported:

- `Migrate` (default): The VM will be live-migrated to another node in the cluster. This is the behavior of the maintenance feature as it is at the moment and will be default with this enhancement.
- `ShutdownAndRestartAfterEnable`: Shut down and restart the VM after maintenance mode is enabled. The VM will be scheduled on a different node.
- `ShutdownAndRestartAfterDisable`: Shut down when maintenance mode is enabled and restart the VM after maintenance mode is disabled. The VM will stay on the same node.
- `Shutdown`: Shut down when maintenance mode is enabled. Do NOT restart the VM, it remains switched off.

If the `Force` checkbox is selected in the maintenance dialog, a configured maintenance mode strategy of a VM is overridden and it is shut down and remains off regardless of the configured strategy.
For all other VMs that do not have the label `harvesterhci.io/maintain-mode-strategy`, the current behavior of the forced maintenance mode is applied.

Related to: harvester#5069
HEP: harvester#5984
Docs: harvester/docs#523

Signed-off-by: Volker Theile <vtheile@suse.com>
bk201 pushed a commit to harvester/harvester that referenced this pull request Jul 5, 2024
Introduce the label `harvesterhci.io/maintain-mode-strategy` which can be added to a `VirtualMachine` resource to specify the behaviour of the VM in case of a node maintenance event. The following modes are supported:

- `Migrate` (default): The VM will be live-migrated to another node in the cluster. This is the behavior of the maintenance feature as it is at the moment and will be default with this enhancement.
- `ShutdownAndRestartAfterEnable`: Shut down and restart the VM after maintenance mode is enabled. The VM will be scheduled on a different node.
- `ShutdownAndRestartAfterDisable`: Shut down when maintenance mode is enabled and restart the VM after maintenance mode is disabled. The VM will stay on the same node.
- `Shutdown`: Shut down when maintenance mode is enabled. Do NOT restart the VM, it remains switched off.

If the `Force` checkbox is selected in the maintenance dialog, a configured maintenance mode strategy of a VM is overridden and it is shut down and remains off regardless of the configured strategy.
For all other VMs that do not have the label `harvesterhci.io/maintain-mode-strategy`, the current behavior of the forced maintenance mode is applied.

Related to: #5069
HEP: #5984
Docs: harvester/docs#523

Signed-off-by: Volker Theile <vtheile@suse.com>
Copy link
Contributor

@jillian-maroket jillian-maroket left a comment

Choose a reason for hiding this comment

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

Let me know if you have concerns about the changes.

docs/host/host.md Outdated Show resolved Hide resolved
docs/host/host.md Outdated Show resolved Hide resolved
bk201 pushed a commit to bk201/harvester that referenced this pull request Jul 5, 2024
Introduce the label `harvesterhci.io/maintain-mode-strategy` which can be added to a `VirtualMachine` resource to specify the behaviour of the VM in case of a node maintenance event. The following modes are supported:

- `Migrate` (default): The VM will be live-migrated to another node in the cluster. This is the behavior of the maintenance feature as it is at the moment and will be default with this enhancement.
- `ShutdownAndRestartAfterEnable`: Shut down and restart the VM after maintenance mode is enabled. The VM will be scheduled on a different node.
- `ShutdownAndRestartAfterDisable`: Shut down when maintenance mode is enabled and restart the VM after maintenance mode is disabled. The VM will stay on the same node.
- `Shutdown`: Shut down when maintenance mode is enabled. Do NOT restart the VM, it remains switched off.

If the `Force` checkbox is selected in the maintenance dialog, a configured maintenance mode strategy of a VM is overridden and it is shut down and remains off regardless of the configured strategy.
For all other VMs that do not have the label `harvesterhci.io/maintain-mode-strategy`, the current behavior of the forced maintenance mode is applied.

Related to: harvester#5069
HEP: harvester#5984
Docs: harvester/docs#523

Signed-off-by: Volker Theile <vtheile@suse.com>
(cherry picked from commit d4166eb)
Copy link

github-actions bot commented Jul 5, 2024

Name Link
🔨 Latest commit e60b32b
😎 Deploy Preview https://66bdc110ff38d893ba423eda--harvester-preview.netlify.app

bk201 pushed a commit to harvester/harvester that referenced this pull request Jul 5, 2024
Introduce the label `harvesterhci.io/maintain-mode-strategy` which can be added to a `VirtualMachine` resource to specify the behaviour of the VM in case of a node maintenance event. The following modes are supported:

- `Migrate` (default): The VM will be live-migrated to another node in the cluster. This is the behavior of the maintenance feature as it is at the moment and will be default with this enhancement.
- `ShutdownAndRestartAfterEnable`: Shut down and restart the VM after maintenance mode is enabled. The VM will be scheduled on a different node.
- `ShutdownAndRestartAfterDisable`: Shut down when maintenance mode is enabled and restart the VM after maintenance mode is disabled. The VM will stay on the same node.
- `Shutdown`: Shut down when maintenance mode is enabled. Do NOT restart the VM, it remains switched off.

If the `Force` checkbox is selected in the maintenance dialog, a configured maintenance mode strategy of a VM is overridden and it is shut down and remains off regardless of the configured strategy.
For all other VMs that do not have the label `harvesterhci.io/maintain-mode-strategy`, the current behavior of the forced maintenance mode is applied.

Related to: #5069
HEP: #5984
Docs: harvester/docs#523

Signed-off-by: Volker Theile <vtheile@suse.com>
(cherry picked from commit d4166eb)
Related to: harvester/harvester#5069

Co-authored-by: Jillian <67180770+jillian-maroket@users.noreply.github.com>
Signed-off-by: Volker Theile <vtheile@suse.com>
@votdev votdev force-pushed the issue_5069_improve_maintenance_mode branch from 9dbb8a2 to e60b32b Compare August 15, 2024 08:33
Copy link
Member

@w13915984028 w13915984028 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@votdev
Copy link
Member Author

votdev commented Sep 2, 2024

@Mergifyio backport v1.4

Copy link

mergify bot commented Sep 2, 2024

backport v1.4

❌ No backport have been created

  • Backport to branch v1.4 failed

GitHub error: Branch not found

@bk201 bk201 merged commit c65a770 into harvester:main Sep 2, 2024
3 checks passed
@bk201
Copy link
Member

bk201 commented Sep 2, 2024

@votdev there is no need to backport a doc PR to other branches.

@votdev votdev deleted the issue_5069_improve_maintenance_mode branch September 25, 2024 12:03
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