-
Notifications
You must be signed in to change notification settings - Fork 98
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
Compute: update to package-2019-12 and support simulate eviction #1199
Conversation
.../java/com/microsoft/azure/management/compute/implementation/ProximityPlacementGroupImpl.java
Show resolved
Hide resolved
SdkContext.sleep(30 * 60 * 1000); | ||
|
||
virtualMachine = computeManager.virtualMachines().getById(virtualMachine.id()); | ||
Assert.assertNotNull(virtualMachine); |
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.
Assert if the vm state is deallocated. When it is deallocated, vm entity will still be there.
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.
Do we need to get such state via InstanceView?
SdkContext.sleep(30 * 60 * 1000); | ||
|
||
for (VirtualMachineScaleSetVM instance: vmInstances) { | ||
instance.refresh(); |
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.
Same as Single VM. Check the vm state of each VM.
* Compute: upgrade api version * Compute: generate package-2019-12 * fix compile errors * update record sessions * sync Azure/azure-libraries-for-java#1049 * sync Azure/azure-libraries-for-java#1177 * sync Azure/azure-libraries-for-java#1199 * fix spotbugs error * sync specialized image changes and rename generalized methods * update session records * fix checkstyle * update session record
No description provided.