Skip to content

CLOUDSTACK-8555:Skip testcase for HyperV as it doesn't support volume resize operation #391

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

Jump to bottom
Closed
pritisarap12 wants to merge 2 commits into from
Closed

CLOUDSTACK-8555:Skip testcase for HyperV as it doesn't support volume resize operation #391

pritisarap12 wants to merge 2 commits into from

Conversation

@pritisarap12
Copy link

@pritisarap12 pritisarap12 commented Jun 12, 2015

No description provided.

@asfbot
Copy link

asfbot commented Jun 12, 2015

cloudstack-pull-requests #366 SUCCESS
This pull request looks good

gauravaradhye
gauravaradhye reviewed Jun 15, 2015
View reviewed changes
test/integration/component/test_browse_volumes.py Outdated
Copy link

@gauravaradhye gauravaradhye Jun 15, 2015

Choose a reason for hiding this comment

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

Should be %s instead of %. Change for all applicable places.

@pritisarap12
Copy link
Author

pritisarap12 commented Jun 15, 2015

Done with the review changes.

gauravaradhye
gauravaradhye reviewed Jun 15, 2015
View reviewed changes
test/integration/component/test_browse_volumes.py Outdated
Copy link

@gauravaradhye gauravaradhye Jun 15, 2015

Choose a reason for hiding this comment

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

Can we skip only the resize volume test case with if condition and run test case for other operations?

Copy link
Author

@pritisarap12 pritisarap12 Jun 15, 2015

Choose a reason for hiding this comment

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

Yes, we can skip only resize operation and done with modification of the code doing the same.

@asfbot
Copy link

asfbot commented Jun 15, 2015

cloudstack-pull-requests #443 SUCCESS
This pull request looks good

@asfbot
Copy link

asfbot commented Jun 15, 2015

cloudstack-pull-requests #444 SUCCESS
This pull request looks good

gauravaradhye
gauravaradhye reviewed Jun 15, 2015
View reviewed changes
test/integration/component/test_browse_volumes.py Outdated
Copy link

@gauravaradhye gauravaradhye Jun 15, 2015

Choose a reason for hiding this comment

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

Remove this old code snippet which skips the whole test.

CLOUDSTACK-8555:Skip testcase for HyperV as it doesn't support volume…
655c84a 
… resize operation

    --Modified skipTest message text
@asfbot
Copy link

asfbot commented Jun 15, 2015

cloudstack-pull-requests #445 SUCCESS
This pull request looks good

@gauravaradhye
Copy link

gauravaradhye commented Jun 15, 2015

LGTM, merging.

@asfgit asfgit closed this in 019f246 Jun 15, 2015
kishankavala pushed a commit to kishankavala/cloudstack that referenced this pull request Jun 25, 2015
@kishankavala
CLOUDSTACK-8555: Skip volume resize operation for HyperV as it's not
dc66e52 
supported

Signed-off-by: Gaurav Aradhye <gaurav.aradhye@clogeny.com>
This closes apache#391
yadvr added a commit that referenced this pull request Jan 20, 2021
@yadvr
config: don't show egress, ip address tab for VPC network
39bf93c 
Fixes #391

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
shwstppr pushed a commit to shapeblue/cloudstack that referenced this pull request Apr 11, 2024
@vishesh92
Add missing indexes for vmstats (apache#391)
a1122d1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pritisarap12 @asfbot @gauravaradhye