Skip to content

Conversation

@nachoalonsoportillo
Copy link
Member

No description provided.

@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Mar 22, 2022
@ghost
Copy link

ghost commented Mar 22, 2022

Thank you for your contribution nachoalonsoportillo! We will review the pull request and get back to you soon.

@ghost ghost requested a review from yonzhan March 22, 2022 09:51
@ghost ghost added the Auto-Assign Auto assign by bot label Mar 22, 2022
@ghost ghost assigned zhoxing-ms Mar 22, 2022
@ghost ghost added this to the Mar 2022 (2022-04-06) milestone Mar 22, 2022
@ghost ghost added the Compute az vm/vmss/image/disk/snapshot label Mar 22, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 22, 2022

VM


with self.argument_context('vm encryption enable') as c:
c.argument('encrypt_format_all', action='store_true', help='Encrypts-formats data disks instead of encrypting them. Encrypt-formatting is a lot faster than in-place encryption but wipes out the partition getting encrypt-formatted.')
c.argument('encrypt_format_all', action='store_true', help='Encrypts-formats data disks instead of encrypting them. Encrypt-formatting is a lot faster than in-place encryption but wipes out the partition getting encrypt-formatted. (Only supported for Linux virtual machines.)')
Copy link
Contributor

@zhoxing-ms zhoxing-ms Mar 22, 2022

Choose a reason for hiding this comment

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

(Only supported for Linux virtual machines.)

@nachoalonsoportillo Could you please explain the background of this modification?

Copy link
Member Author

Choose a reason for hiding this comment

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

Only Linux extension supports that encryption operation. Windows extension doesn't, and if you specify that parameter for a Windows VM it reports the following error MicrosoftDocs/azure-docs-cli#549.

Copy link
Contributor

Choose a reason for hiding this comment

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

Got it, thanks~

@zhoxing-ms zhoxing-ms changed the title [VM] Clarify that encrypt-format-all is only applicable to Linux VMs {Compute} Clarify that encrypt-format-all is only applicable to Linux VMs Mar 22, 2022
@zhoxing-ms zhoxing-ms merged commit b999f13 into Azure:dev Mar 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Compute az vm/vmss/image/disk/snapshot customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants