You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by gw...@apache.org on 2015/08/28 20:22:44 UTC

kafka git commit: MINOR: kafkatest add manifest

Repository: kafka
Updated Branches:
  refs/heads/trunk 8956004a5 -> 9c936b186


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 resources with the package
- renaming the version to conform w/PEP 440, since python packaging tools reject the current version name

Author: Geoff Anderson <ge...@confluent.io>

Reviewers: Ewen Cheslack-Postava, Gwen Shapira

Closes #173 from granders/minor-kafkatest-add-manifest


Project: http://git-wip-us.apache.org/repos/asf/kafka/repo
Commit: http://git-wip-us.apache.org/repos/asf/kafka/commit/9c936b18
Tree: http://git-wip-us.apache.org/repos/asf/kafka/tree/9c936b18
Diff: http://git-wip-us.apache.org/repos/asf/kafka/diff/9c936b18

Branch: refs/heads/trunk
Commit: 9c936b186d390f59f1d4ad8cc2995f800036a3d6
Parents: 8956004
Author: Geoff Anderson <ge...@confluent.io>
Authored: Fri Aug 28 11:22:39 2015 -0700
Committer: Gwen Shapira <cs...@gmail.com>
Committed: Fri Aug 28 11:22:39 2015 -0700

----------------------------------------------------------------------
 tests/MANIFEST.in                            | 16 ++++++++++++++++
 tests/kafkatest/services/console_consumer.py |  1 -
 tests/setup.py                               |  5 +++--
 3 files changed, 19 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kafka/blob/9c936b18/tests/MANIFEST.in
----------------------------------------------------------------------
diff --git a/tests/MANIFEST.in b/tests/MANIFEST.in
new file mode 100644
index 0000000..cda3ed5
--- /dev/null
+++ b/tests/MANIFEST.in
@@ -0,0 +1,16 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+recursive-include kafkatest/services/templates *.properties

http://git-wip-us.apache.org/repos/asf/kafka/blob/9c936b18/tests/kafkatest/services/console_consumer.py
----------------------------------------------------------------------
diff --git a/tests/kafkatest/services/console_consumer.py b/tests/kafkatest/services/console_consumer.py
index ffde6a2..9286654 100644
--- a/tests/kafkatest/services/console_consumer.py
+++ b/tests/kafkatest/services/console_consumer.py
@@ -174,7 +174,6 @@ class ConsoleConsumer(BackgroundThreadService):
             if self.message_validator is not None:
                 msg = self.message_validator(msg)
             if msg is not None:
-                self.logger.debug("consumed a message: " + str(msg))
                 self.messages_consumed[idx].append(msg)
 
     def start_node(self, node):

http://git-wip-us.apache.org/repos/asf/kafka/blob/9c936b18/tests/setup.py
----------------------------------------------------------------------
diff --git a/tests/setup.py b/tests/setup.py
index 2a53880..00e58dd 100644
--- a/tests/setup.py
+++ b/tests/setup.py
@@ -17,11 +17,12 @@
 from setuptools import find_packages, setup
 
 setup(name="kafkatest",
-      version="0.8.3-SNAPSHOT",
+      version="0.8.3.dev0",
       description="Apache Kafka System Tests",
       author="Apache Kafka",
       platforms=["any"], 
       license="apache2.0",
       packages=find_packages(),
-      requires=["ducktape(==0.3.1)"]
+      include_package_data=True,
+      install_requires=["ducktape==0.3.2"]
       )