[GitHub] kafka pull request: MINOR: Bump system test ducktape dependency to...

2016-05-17 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1397 MINOR: Bump system test ducktape dependency to 0.5.1 You can merge this pull request into a Git repository by running: $ git pull https://github.com/granders/kafka minor-increment-ducktape

[GitHub] kafka pull request #1659: MINOR: cleanup apache license in python files

2016-07-24 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1659 MINOR: cleanup apache license in python files @ijuma As discussed in https://github.com/apache/kafka/pull/1645, this patch removes an extraneous line from several __init__.py files, and a few

[GitHub] kafka pull request: MINOR: Advance system test ducktape dependency...

2016-03-29 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1165 MINOR: Advance system test ducktape dependency from 0.3.10 to 0.4.0 Previous version of ducktape was found to have a memory leak which caused occasional failures in nightly runs. You can merge

[GitHub] kafka pull request: KAFKA-3592: System test - configurable paths [...

2016-04-20 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1245 KAFKA-3592: System test - configurable paths [WIP] This patch adds logic for the following: - remove hard-coded paths to various scripts and jars in kafkatest service classes - provide a

[GitHub] kafka pull request: KAFKA-3581: add timeouts to joins in backgroun...

2016-04-29 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1297 KAFKA-3581: add timeouts to joins in background thread services This actually removes joins altogether, as well as references to self.worker_threads, which is best left as an implementation detail

[GitHub] kafka pull request: MINOR: Add version check on enable-systest-eve...

2016-04-29 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1298 MINOR: Add version check on enable-systest-events flag Recent patch adding enable-systest-events flag without any version check breaks all uses of versioned console consumer. E.g. upgrade tests

[GitHub] kafka pull request: KAFKA-2772: Stabilize failures on replication ...

2015-12-09 Thread granders
Github user granders closed the pull request at: https://github.com/apache/kafka/pull/481 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] kafka pull request: KAFKA-2928: system test: fix version sanity ch...

2015-12-09 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/656 KAFKA-2928: system test: fix version sanity checks Fixed version sanity checks by updated kafkatest version to match kafka version You can merge this pull request into a Git repository by running

[GitHub] kafka pull request: KAFKA-2927: reduce system test storage footpri...

2015-12-09 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/657 KAFKA-2927: reduce system test storage footprint Split kafka logging into two levels - DEBUG and INFO, and do not collect DEBUG by default. You can merge this pull request into a Git repository by

[GitHub] kafka pull request: MINOR: vagrant aws overrideable ec2 instance n...

2016-01-21 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/801 MINOR: vagrant aws overrideable ec2 instance name prefix @ewencp This small change allows users to use Vagrantfile.local to specify a custom prefix for names of ec2 instances brought up with

[GitHub] kafka pull request: MINOR: Increment ducktape dependency

2016-02-02 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/852 MINOR: Increment ducktape dependency Pin kafka system tests to a newer version of ducktape. Ran in branch builder; only one preexisting (transient) failure: http://confluent-kafka

[GitHub] kafka pull request: MINOR: Pin to system tests to ducktape 0.3.10

2016-02-03 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/863 MINOR: Pin to system tests to ducktape 0.3.10 You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc/kafka increment-ducktape-0.9.0

[GitHub] kafka pull request: MINOR: Pin to system tests to ducktape 0.3.10

2016-02-04 Thread granders
Github user granders closed the pull request at: https://github.com/apache/kafka/pull/863 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] kafka pull request: MINOR: Removed unnecessary Vagrantfile hack

2016-02-04 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/867 MINOR: Removed unnecessary Vagrantfile hack The hack here is no longer necessary with up-to-date versions of Vagrant, vagrant-hostmanager, and vagrant-aws. What's more, the change in c8b

[GitHub] kafka pull request: MINOR: Updated kafkatest version to match vers...

2016-02-05 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/878 MINOR: Updated kafkatest version to match version specified in gradle.properties This discrepancy was causing a few system tests which to some version validation to fail consistently on 0.9.0

[GitHub] kafka pull request: MINOR: Updated kafkatest version to match vers...

2016-02-05 Thread granders
Github user granders closed the pull request at: https://github.com/apache/kafka/pull/878 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] kafka pull request: MINOR: Stabilize transient replication test fa...

2016-02-09 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/890 MINOR: Stabilize transient replication test failures in 0.9.0 - ported timeout values in `produce_consume_validate.py` from trunk to 0.9.0 - ported `producer_throughput_value` in

[GitHub] kafka pull request: MINOR: Stabilize transient replication test fa...

2016-02-09 Thread granders
Github user granders closed the pull request at: https://github.com/apache/kafka/pull/890 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] kafka pull request: MINOR: Add vagrant up wrapper for simple paral...

2016-02-26 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/982 MINOR: Add vagrant up wrapper for simple parallel bringup on aws The main impediment to bringing up aws machines in parallel using vagrant was the interaction between `vagrant-hostmanager` and

[GitHub] kafka pull request: MINOR: Ignoring streams tests until there is f...

2016-03-08 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1031 MINOR: Ignoring streams tests until there is fix for KAFKA-3354 Per discussion with @guozhangwang, `@ignore` failing streams system tests until fix for KAFKA-3354 is checked in. You can merge

[GitHub] kafka pull request: Kafka 2276

2015-06-16 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/70 Kafka 2276 Initial patch for KIP-25 Note that to install ducktape, do *not* use pip to install ducktape. Instead: ``` $ git clone g...@github.com:confluentinc/ducktape.git

[GitHub] kafka pull request: KAFKA-2327

2015-07-09 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/73 KAFKA-2327 Added unit tests as well. These fail without the fix, but pass with the fix. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request: KAFKA-2408 ConsoleConsumerService direct log o...

2015-08-06 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/123 KAFKA-2408 ConsoleConsumerService direct log output to file You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc/kafka KAFKA-2408

[GitHub] kafka pull request: MINOR: expose vagrant base box as variable

2015-08-13 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/137 MINOR: expose vagrant base box as variable Added base_box variable to Vagrantfile. This makes it possible to override the base box in Vagrantfile.local. You can merge this pull request into a Git

[GitHub] kafka pull request: KAFKA-2439: Add MirrorMaker service class for ...

2015-08-18 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/148 KAFKA-2439: Add MirrorMaker service class for system tests Added MirrorMaker service and a few corresponding sanity checks, as well as necessary config template files. A few additional updates to

[GitHub] kafka pull request: MINOR: kafkatest add manifest

2015-08-27 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/173 MINOR: kafkatest add manifest This patch makes it possible to publish kafkatest (system test package) to pypi and use it as a library in other projects by: - including necessary static

[GitHub] kafka pull request: KAFKA-2489: add benchmark for new consumer

2015-08-29 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/179 KAFKA-2489: add benchmark for new consumer @ewencp The changes here are smaller than they look - mostly refactoring/cleanup. - ConsumerPerformance.scala - corrected timeout inequality

[GitHub] kafka pull request #1834: [WIP] KAFKA-4140: make system tests parallel frien...

2016-09-07 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/1834 [WIP] KAFKA-4140: make system tests parallel friendly Updates to take advantage of soon-to-be-released ducktape features. You can merge this pull request into a Git repository by running

[GitHub] kafka pull request: Updated testing readme

2015-09-02 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/187 Updated testing readme Minor update to point to testing tutorial, and install the correct version of vagrant-hostmanager You can merge this pull request into a Git repository by running

[GitHub] kafka pull request: MINOR: Added scripts to automate Vagrant setup...

2015-09-09 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/201 MINOR: Added scripts to automate Vagrant setup for system tests Updated testing README accordingly. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request: KAFKA-1888 [WIP]: rolling upgrade test

2015-09-21 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/229 KAFKA-1888 [WIP]: rolling upgrade test @ewencp @gwenshap This needs some refactoring to avoid the duplicated code between replication test and upgrade test, but in shape for initial feedback

[GitHub] kafka pull request: MINOR: update to correct clock skew

2015-10-29 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/383 MINOR: update to correct clock skew @ewencp Updated the provisioning script to install ntp daemon. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request: Update which makes kafkatest services useable ...

2015-10-29 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/385 Update which makes kafkatest services useable by external dependencies @ewencp Some *.properties files were missing from `kafkatest` package I've tested this change on aws

[GitHub] kafka pull request: MINOR: Removed previous system_test folder

2015-10-30 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/392 MINOR: Removed previous system_test folder @ewencp Nothing too complicated here You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc

[GitHub] kafka pull request: KAFKA-2726: Fix port collision between ntpdate...

2015-11-02 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/407 KAFKA-2726: Fix port collision between ntpdate and ntp daemon @gwenshap Can you take a quick look? I have verified the change allows successful `vagrant provision` even with ntp daemon already

[GitHub] kafka pull request: KAFKA-2258[WIP]: add failover to mirrormaker t...

2015-11-04 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/427 KAFKA-2258[WIP]: add failover to mirrormaker test This PR adds failover to simple end to end mirror maker test Marked as WIP for 2 reasons: - We may want to add a couple more test cases

[GitHub] kafka pull request: KAFKA-2773: Fixed broken vagrant provision scr...

2015-11-08 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/454 KAFKA-2773: Fixed broken vagrant provision scripts for static zk/broker cluster You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc

[GitHub] kafka pull request: KAFKA-2773: Fixed broken vagrant provision scr...

2015-11-08 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/455 KAFKA-2773: Fixed broken vagrant provision scripts for static zk/broker cluster You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc

[GitHub] kafka pull request: KAFKA-2773: (0.9.0 branch)Fixed broken vagrant...

2015-11-09 Thread granders
Github user granders closed the pull request at: https://github.com/apache/kafka/pull/455 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] kafka pull request: KAFKA-2772: Stabilize failures on replication ...

2015-11-09 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/481 KAFKA-2772: Stabilize failures on replication with hard bounce You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc/kafka KAFKA-2772

[GitHub] kafka pull request: MINOR: update system test readme

2015-11-10 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/487 MINOR: update system test readme You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc/kafka minor-update-test-readme Alternatively you

[GitHub] kafka pull request: MINOR: Do not collect zk persistent data by de...

2015-11-11 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/504 MINOR: Do not collect zk persistent data by default In system tests zookeeper service, it is overkill and space-intensive to collect zookeeper data logs by default. This minor patch turns off

[GitHub] kafka pull request: MINOR: Tuned timeout parameter to reduce chanc...

2015-11-12 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/521 MINOR: Tuned timeout parameter to reduce chance of transient failure Increased timeout in downstream consumer doing validation step. This addresses a transient failure case in mirror maker tests

[GitHub] kafka pull request: KAFKA-2845: new client old broker compatibilit...

2015-11-16 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/537 KAFKA-2845: new client old broker compatibility You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc/kafka KAFKA-2845-new-client-old

[GitHub] kafka pull request: KAFKA-2820: systest log level

2015-11-16 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/538 KAFKA-2820: systest log level This restores control over log level in KafkaService, and adds SASL debug logging when SASL is enabled You can merge this pull request into a Git repository by

[GitHub] kafka pull request: KAFKA-2820: Remove log threshold on appender i...

2015-11-18 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/556 KAFKA-2820: Remove log threshold on appender in tools-log4j.properties Removed a config in tools-log4j.properties which prevented certain service classes from running at TRACE level. You can merge

[GitHub] kafka pull request: MINOR: Log at INFO level in Benchmark tests

2015-11-20 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/569 MINOR: Log at INFO level in Benchmark tests You can merge this pull request into a Git repository by running: $ git pull https://github.com/confluentinc/kafka minor-reduce-benchmark-logs

[GitHub] kafka pull request: MINOR: Manually ported changes in 8c3c9548b636...

2015-12-02 Thread granders
GitHub user granders opened a pull request: https://github.com/apache/kafka/pull/620 MINOR: Manually ported changes in 8c3c9548b636cdf760d2537afe115942d13bc003 Porting manually to 0.9.0 (no cherry-pick due to conflicts) This will allow concurrent system test runs on the