Skip to content

Commit 33a59f1

Browse files
committed
Fix semaphore
1 parent 3972fd1 commit 33a59f1

File tree

3 files changed

+5
-6
lines changed

3 files changed

+5
-6
lines changed

tests/ducktape/benchmark_metrics.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -266,16 +266,14 @@ def _load_from_config_file(self, config_path: str):
266266
config = json.load(f)
267267

268268
# Always use environment-based format
269-
environment = os.getenv('BENCHMARK_ENVIRONMENT',
270-
config.get('_default_environment', 'local'))
271-
269+
environment = os.getenv('BENCHMARK_ENVIRONMENT',
270+
config.get('_default_environment', 'local'))
272271
if environment not in config:
273272
available_envs = [k for k in config.keys() if not k.startswith('_')]
274273
raise ValueError(
275274
f"Environment '{environment}' not found in config. "
276275
f"Available environments: {available_envs}"
277276
)
278-
279277
bounds_config = config[environment]
280278
print(f"Loading benchmark bounds for environment: {environment}")
281279

tests/ducktape/run_ducktape_test.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def main():
5858
import pkg_resources
5959
version = pkg_resources.get_distribution('ducktape').version
6060
print(f"Using ducktape version: {version}")
61-
except:
61+
except Exception:
6262
print("Using ducktape version: unknown")
6363

6464
# Check if confluent_kafka is available

tests/ducktape/test_producer.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,8 @@ def delivery_callback(err, msg):
373373
if not is_valid:
374374
self.logger.error("Performance bounds validation failed for %s compression: %s",
375375
compression_type, "; ".join(violations))
376-
assert False, f"Performance bounds validation failed for {compression_type} compression: {'; '.join(violations)}"
376+
assert False, (f"Performance bounds validation failed for {compression_type} "
377+
f"compression: {'; '.join(violations)}")
377378

378379
self.logger.info("Successfully completed %s compression test with comprehensive metrics", compression_type)
379380

0 commit comments

Comments
 (0)