[ 
https://issues.apache.org/jira/browse/BIGTOP-78?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13141604#comment-13141604
 ] 

Peter Linnell commented on BIGTOP-78:
-------------------------------------

> 3/ Could we use constants for run levels?   Referring to the init scripts or 
> package testing ?
                
> Misc improvments 
> -----------------
>
>                 Key: BIGTOP-78
>                 URL: https://issues.apache.org/jira/browse/BIGTOP-78
>             Project: Bigtop
>          Issue Type: Improvement
>          Components: Tests
>    Affects Versions: 0.1.0, 0.2.0
>            Reporter: Bruno Mahé
>
> From BIGTOP-76 in 
> bigtop-tests/test-artifacts/package/src/main/groovy/org/apache/bigtop/itest/packagesmoke/TestPackagesBasics.groovy:
> 0/ Could you kill:
> +    // repo = new PackageTestRepoMgr("3", "", 
> "http://nightly.cloudera.com/debian/";,
> +    //                                       
> "http://nightly.cloudera.com/debian/archive.key";);
> 1/ This one should be in the utils (not necessary for this task if another 
> ticket is opened):
> +    String arch = (new Shell()).exec("uname 
> -m").getOut().get(0).replaceAll(/i.86/,"i386").replaceAll(/x86_64/,"amd64");
> +    String archTranslated = (type == "apt") ? "" : ((arch == "amd64") ? 
> ".x86_64" : ".${arch}");
> 2/ I don't see golden (or pkgGolden) being used in TestPackagesBasics.
> 3/ Could we use constants for run levels?
> 4/ Could 3 be a constant? Maybe even use something like your  tryOrFail 
> static method
> for (i=3; pkg.install() && i>0; i--) {};
> 5/ This  list should definitely be a constant somewhere:
> +      } else if (["apt", "yum", "zypper"].contains(key)) { // poor man's XML 
> filtering

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to