-
Notifications
You must be signed in to change notification settings - Fork 11
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
[EVS] V3 API Update #466
base: devel
Are you sure you want to change the base?
[EVS] V3 API Update #466
Conversation
Name: volumeName, | ||
} | ||
|
||
volume1, err := volumes.Create(client, createOpts) |
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.
AZ is required parameter for disk: AZ shouldn't be '' or If the AZ does not exist, the disk will fail to create.
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.
done
Detail: true, | ||
} | ||
|
||
allStats, err := schedulerstats.List(blockClient, listOpts) |
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.
The API does not exist or has not been published in the environment
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.
skip
blockClient, err := clients.NewBlockStorageV3Client() | ||
th.AssertNoErr(t, err) | ||
|
||
allServices, err := services2.List(blockClient, services2.ListOpts{}) |
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.
The API does not exist or has not been published in the environment
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.
skip
computeClient, err := clients.NewComputeV2Client() | ||
th.AssertNoErr(t, err) | ||
|
||
volume, err := blockstorageV3.CreateVolume(t, blockClient) |
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.
Invalid input received: Availability zone '' is invalid.
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.
done
t.Skip("OS_NETWORK_ID env var is missing but ECS test requires using existing network") | ||
} | ||
|
||
imageID, err := images.IDFromName(client, "Standard_Debian_10_latest") |
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 request fails by timeout
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.
cannot make request images.ListDetail: server no response.
hard to fix, still trying.
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.
cannot fix now, do it in another PR.
because API discarded and not working.
client, err := clients.NewBlockStorageV3Client() | ||
th.AssertNoErr(t, err) | ||
|
||
volume1, err := CreateVolume(t, client) |
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 above: Invalid input received: Availability zone '' is invalid.
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.
done
th.AssertNoErr(t, err) | ||
t.Cleanup(func() { DeleteVolume(t, client, volume1) }) | ||
|
||
volume2, err := CreateVolume(t, client) |
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: Invalid input received: Availability zone '' is invalid.
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.
done
fix #429