Skip to content

Update azure-hybrid-benefit-linux.md #424

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.

Sign up for GitHub

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

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

Update azure-hybrid-benefit-linux.md #424

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions articles/virtual-machines/linux/azure-hybrid-benefit-linux.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -487,6 +487,20 @@ Converting to a PAYG subscription model is supported for Azure Marketplace image
# In order to revert back to the original licensing model, set license-type to None.
az vm update -g myResourceGroup -n myVmName --license-type NONE
```
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

A blank line should be added before this new paragraph to maintain consistency with the document's formatting. Looking at the surrounding context, numbered list items are followed by a blank line before new paragraphs or sections begin (see lines 484-485 and 464-465).

Suggested change
```
```

Copilot uses AI. Check for mistakes.
In some cases, you must utilize the Azure Management API to patch the license type in the virtual machine's metadata:
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The terminology is inconsistent between "virtual machine's metadata" (line 490) and "VM's metadata" (line 496). For consistency with the rest of the document, use "VM" in both places since "VM" is the more commonly used abbreviation throughout this documentation.

Suggested change
In some cases, you must utilize the Azure Management API to patch the license type in the virtual machine's metadata:
In some cases, you must utilize the Azure Management API to patch the license type in the VM's metadata:

Copilot uses AI. Check for mistakes.

1. Define a variable with a full path to the target virtual machine:
```azurecli
VM_ID=$(az vm list --query "[?name=='YourVMName'].id" -o tsv)
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The placeholder "YourVMName" is inconsistent with the naming convention used throughout this document. The document consistently uses "myVmName" (with lowercase 'm' in 'Vm') as the placeholder for VM names in the -n parameter of az vm update commands. Change "YourVMName" to "myVmName" to match the established pattern.

Suggested change
VM_ID=$(az vm list --query "[?name=='YourVMName'].id" -o tsv)
VM_ID=$(az vm list --query "[?name=='myVmName'].id" -o tsv)

Copilot uses AI. Check for mistakes.
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The command should specify the resource group to ensure the correct VM is selected, especially in subscriptions with multiple resource groups. Consider adding --resource-group myResourceGroup to match the pattern used elsewhere in this document, or update the query to include a resource group filter like "[?name=='myVmName' && resourceGroup=='myResourceGroup'].id".

Suggested change
VM_ID=$(az vm list --query "[?name=='YourVMName'].id" -o tsv)
VM_ID=$(az vm list --resource-group myResourceGroup --query "[?name=='YourVMName'].id" -o tsv)

Copilot uses AI. Check for mistakes.
```
1. Update the license type by patching the VM's metadata. For example, change PAYG license to SLES_SAP:
Copy link

Copilot AI Feb 12, 2026

Choose a reason for hiding this comment

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

The phrase "change PAYG license to SLES_SAP" is unclear. Based on the context of the document, this should be "change from PAYG to SLES_SAP" or "change the license type from PAYG to SLES_SAP" to match the pattern seen in line 406 ("conversion from PAYG to BYOS") and to clarify that you're converting between subscription models, not just changing a PAYG license.

Suggested change
1. Update the license type by patching the VM's metadata. For example, change PAYG license to SLES_SAP:
1. Update the license type by patching the VM's metadata. For example, change the license type from PAYG to SLES_SAP:

Copilot uses AI. Check for mistakes.
```azurecli
az rest --method patch --url "https://management.azure.com${VM_ID}?api-version=2024-07-01" --body '{"properties":{"licenseType":"SLES_SAP"}}'
```
1. If you want to return the original subscription model, set `license-type` to `None`.
```azurecli
az rest --method patch --url "https://management.azure.com${VM_ID}?api-version=2024-07-01" --body '{"properties":{"licenseType":"NONE"}}'
```

---

Expand Down