ceph_objectstore_tool: minor fixes

Remove development comment
nullfd isn't used anymore
s/log.errors/log.error/

Signed-off-by: David Zafman <dzafman@redhat.com>

diff --git a/tasks/ceph_objectstore_tool.py b/tasks/ceph_objectstore_tool.py
index 8e0dab0aeada31908dcd151aa0d5f6f5c4ffc735..5d3e533817ae8c7f5464bd303ebae440cd5f8c2f 100644 (file)
--- a/tasks/ceph_objectstore_tool.py
+++ b/tasks/ceph_objectstore_tool.py
@@ -57,11 +57,9 @@ def cod_setup_remote_data(log, ctx, remote, NUM_OBJECTS, DATADIR, BASE_NAME, DAT
         teuthology.write_file(remote, DDNAME, DATA)
 
 
-# def rados(ctx, remote, cmd, wait=True, check_status=False):
 def cod_setup(log, ctx, remote, NUM_OBJECTS, DATADIR, BASE_NAME, DATALINECOUNT, POOL, db, ec):
     ERRORS = 0
     log.info("Creating {objs} objects in pool".format(objs=NUM_OBJECTS))
-    nullfd = open(os.devnull, "w")
 
     objects = range(1, NUM_OBJECTS + 1)
     for i in objects:
@@ -117,7 +115,6 @@ def cod_setup(log, ctx, remote, NUM_OBJECTS, DATADIR, BASE_NAME, DATALINECOUNT,
                 log.critical("setomapval failed with {ret}".format(ret=ret))
             db[NAME]["omap"][mykey] = myval
 
-    nullfd.close()
     return ERRORS
 
 
@@ -588,7 +585,7 @@ def test_objectstore(ctx, config, cli_remote, REP_POOL, REP_NAME, ec=False):
 
             ret = proc.wait()
             if ret != 0:
-                log.errors("After import, rados get failed with {ret}".format(ret=proc.exitstatus))
+                log.error("After import, rados get failed with {ret}".format(ret=proc.exitstatus))
                 ERRORS += 1
                 continue