Skip to content

Conversation

@yadvr
Copy link
Member

@yadvr yadvr commented Oct 26, 2016

Fixes intermittent integration smoke test failures caused in
test_oobm_zchange_password test.

The scope is limited to the integration test only, and full integration test suite is not necessary. We can only consider code reviews and merge on basis of Travis results.

Fixes intermittent integration smoke test failures caused in
test_oobm_zchange_password test.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@yadvr
Copy link
Member Author

yadvr commented Oct 26, 2016

Jenkins failure was due to git pull issue, not the PR changes. Pinging for quick review/merge -- @abhinandanprateek @murali-reddy @karuturi @koushik-das

@yadvr
Copy link
Member Author

yadvr commented Oct 26, 2016

Test seems to be passing now:

=== TestName: test_oobm_zchange_password | Status : SUCCESS ===

@abhinandanprateek
Copy link
Contributor

abhinandanprateek commented Oct 26, 2016

LGTM on code review @rhtyd

@murali-reddy
Copy link
Contributor

LGTM

@yadvr
Copy link
Member Author

yadvr commented Oct 26, 2016

On basis of Travis result, manual test run and code reviews, I'm going ahead and merging this to avoid intermittent failures in upcoming tests.

asfgit pushed a commit that referenced this pull request Oct 26, 2016
CLOUDSTACK-9565: Fix intermittent failure in test_oobm_zchange_passwordFixes intermittent integration smoke test failures caused in
test_oobm_zchange_password test.

The scope is limited to the integration test only, and full integration test suite is not necessary. We can only consider code reviews and merge on basis of Travis results.

* pr/1731:
  CLOUDSTACK-9565: Fix intermittent failure in test_oobm_zchange_password

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@asfgit asfgit merged commit 29844a3 into apache:4.9 Oct 26, 2016
lucas-a-martins pushed a commit to scclouds/cloudstack that referenced this pull request Jan 29, 2024
…ouds'

Consertar erro 404 ao tentar acessar volume de _system_ VMs

Closes apache#1731

See merge request scclouds/scclouds!694
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants