accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ke...@deenlo.com
Subject Re: Review Request 18625: ACCUMULO-2145 Create upgrade test framework
Date Fri, 14 Mar 2014 18:52:43 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18625/#review37247
-----------------------------------------------------------


* should remove old upgrade_test.sh script in the patch
* maybe format README w/ vim gq
* spellcheck README, saw the following : Usfull outpout consol
* Should to start/stop user provided hadoop or zookeeper.  Too many unknowns in test env,
for example zookeeper and hadoop could be running as different users.
* maybe clear tmp dirs used by test when it starts (if I use two different 1.6.0-SNAPSHOT
tar balls, do not want interference)
* I tried running the script and ran into ACCUMULO-2140.  After the upgrade the test were
complaining that test_ingest did not exist.  I looked at the logs and saw that verify was
running before upgrade ran in the master.   I think the script should wait for hdfs://nn/accumuloTest/version
to change before running post test.

- kturner


On March 10, 2014, 5:12 p.m., John McNamee wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/18625/
> -----------------------------------------------------------
> 
> (Updated March 10, 2014, 5:12 p.m.)
> 
> 
> Review request for accumulo.
> 
> 
> Bugs: ACCUMULO-2145
>     https://issues.apache.org/jira/browse/ACCUMULO-2145
> 
> 
> Repository: accumulo
> 
> 
> Description
> -------
> 
> 
> The framework would configure the Accumulo versions, HDFS, zookeeper, and which test
to run.
> Runs a set of upgrade tests. 
> 
> 
> Diffs
> -----
> 
>   test/system/upgrade/README PRE-CREATION 
>   test/system/upgrade/conf/accumulo/accumulo-env.sh PRE-CREATION 
>   test/system/upgrade/conf/accumulo/accumulo-env.template.sh PRE-CREATION 
>   test/system/upgrade/conf/accumulo/accumulo-metrics.xml PRE-CREATION 
>   test/system/upgrade/conf/accumulo/accumulo-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/accumulo/accumulo-site.xml PRE-CREATION 
>   test/system/upgrade/conf/accumulo/accumulo.policy.example PRE-CREATION 
>   test/system/upgrade/conf/accumulo/gc PRE-CREATION 
>   test/system/upgrade/conf/accumulo/generic_logger.xml PRE-CREATION 
>   test/system/upgrade/conf/accumulo/log4j.properties PRE-CREATION 
>   test/system/upgrade/conf/accumulo/masters PRE-CREATION 
>   test/system/upgrade/conf/accumulo/monitor PRE-CREATION 
>   test/system/upgrade/conf/accumulo/monitor_logger.xml PRE-CREATION 
>   test/system/upgrade/conf/accumulo/slaves PRE-CREATION 
>   test/system/upgrade/conf/accumulo/tracers PRE-CREATION 
>   test/system/upgrade/conf/hadoop-1/core-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/hadoop-1/hdfs-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/hadoop-1/mapred-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/hadoop-2/core-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/hadoop-2/hdfs-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/hadoop-2/mapred-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/hadoop-2/yarn-site.template.xml PRE-CREATION 
>   test/system/upgrade/conf/zookeeper/zoo_sample.cfg PRE-CREATION 
>   test/system/upgrade/setup/accumulo-setup.sh PRE-CREATION 
>   test/system/upgrade/setup/hadoop-setup.sh PRE-CREATION 
>   test/system/upgrade/setup/services.sh PRE-CREATION 
>   test/system/upgrade/setup/zookeeper-setup.sh PRE-CREATION 
>   test/system/upgrade/tests/Bulk_Ingest_Test.sh PRE-CREATION 
>   test/system/upgrade/tests/Compaction_Test.sh PRE-CREATION 
>   test/system/upgrade/tests/Ingest_Test.sh PRE-CREATION 
>   test/system/upgrade/tests/Metadata_Test.sh PRE-CREATION 
>   test/system/upgrade/upgrade_test.sh PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/18625/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> John McNamee
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message