-
Notifications
You must be signed in to change notification settings - Fork 4k
Add support for querying encryption status from the AzureDiskEncryptionForLinux extension #2849
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
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
09bf816
Remove hardcoded default extension names
f489d96
Set ExtensionName property when getting extension status
3666987
Fix extension name assignment in Disable cmdlet
ec6424f
Use the Name property for extension type
ae53598
Merge branch 'release-1.5.0' into dev
b2cf886
Remove duplicate assignment
dfc53c8
Added ProgressMessage field for disk encryption status
2cc44ac
Display EncryptionProgress in cmdlet output
d7f17e4
Fix label name
6626475
Merge branch 'dev' of github.com:Azure/azure-powershell into dev
c079b3c
Merge branch 'dev' of github.com:Azure/azure-powershell into dev
de50276
Allow OS disk encryption on Linux
f91016f
If extension name is provided, use it when getting the status
f5451e9
Merge branch 'dev' of github.com:Azure/azure-powershell into dev
3be1bc1
Merge branch 'dev' of github.com:Azure/azure-powershell into dev
930cadb
Use QueryEncryptionStatus operation for Linux VMs
e825b29
Parse the encryption status from JSON in substatus field for Linux VMs
79f789a
Merge remote-tracking branch 'upstream/dev' into dev
6b0bf42
Use Unknown for encryption status for issue Azure/WALinuxAgent#388
d4d5774
Capitalize "Windows"
aaa84f3
Always use the default name for extension type parameter
d1b4c7b
Use default extension name for type in Disable cmdlet
7209864
Use default extension name when disabling on Linux VM
e7231d8
Merge remote-tracking branch 'upstream/dev' into dev
8c284b4
Do null-checks on context and context.SubsStatuses
727639c
Show message to user if no extension status is available
def485a
Add disable encryption volume check for Linux VMs
a9b53c4
Merge remote-tracking branch 'upstream/dev' into dev
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this catch too narrow? In the case of invalid input, it seems there might be other exceptions that could be thrown by this block that we might want to handle in the same way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the exception that we throw ourselves in GetExtensionStatusMessage:
For other exceptions, we probably want to fail like we would've before this substatus change.