Bug 967842

Summary: RolloutPlanTestCase#testMaxFailServersPercentageRolloutPlan incorrectly refers to a rollout plan from different test method
Product: [JBoss] JBoss Enterprise Application Platform 6 Reporter: Ivo Studensky <istudens>
Component: TestsuiteAssignee: Ivo Studensky <istudens>
Status: CLOSED DUPLICATE QA Contact:
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.1.0CC: cdewolf
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 969381 (view as bug list) Environment:
Last Closed: 2013-07-02 06:40:34 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 969381    

Description Ivo Studensky 2013-05-28 12:15:58 UTC
Description of problem:
The RolloutPlanTestCase#testMaxFailServersPercentageRolloutPlan in EAP testsuite contains a bug as it refers to 'maxFailOnePlan' plan instead of 'maxFailPercPlan' on line 298.

Version-Release number of selected component (if applicable):
EAP 6.1.0, upstream

How reproducible:
If the test methods are not run in alphabetical order they will fail due to this bug.

Comment 1 JBoss JIRA Server 2013-05-28 14:17:11 UTC
Ivo Studensky <istudens> made a comment on jira WFLY-1403

Created a PR with the fix.

Comment 2 JBoss JIRA Server 2013-05-29 12:15:18 UTC
Ivo Studensky <istudens> made a comment on jira WFLY-1403

Merged.

Comment 4 Ivo Studensky 2013-05-31 10:19:55 UTC
Changing flags from jboss‑eap‑6.1.1 to jboss‑eap‑6.2.0 as this bugzilla is an EAP 6.2 one. I will clone this BZ for 6.1.1 and also create a new PR for EAP 6.1.1.

Comment 5 Ivo Studensky 2013-05-31 11:40:46 UTC
Github PR:

https://github.com/jbossas/jboss-eap/pull/155

Comment 6 Ivo Studensky 2013-07-02 06:40:34 UTC

*** This bug has been marked as a duplicate of bug 969381 ***

Comment 7 Rostislav Svoboda 2013-08-28 08:50:23 UTC
Removing flags