Skip to content

Commit ad181f2

Browse files
committed
Fix semaphore
1 parent 22fd64c commit ad181f2

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
@@ -268,16 +268,14 @@ def _load_from_config_file(self, config_path: str):
268268
config = json.load(f)
269269

270270
# Always use environment-based format
271-
environment = os.getenv('BENCHMARK_ENVIRONMENT',
272-
config.get('_default_environment', 'local'))
273-
271+
environment = os.getenv('BENCHMARK_ENVIRONMENT',
272+
config.get('_default_environment', 'local'))
274273
if environment not in config:
275274
available_envs = [k for k in config.keys() if not k.startswith('_')]
276275
raise ValueError(
277276
f"Environment '{environment}' not found in config. "
278277
f"Available environments: {available_envs}"
279278
)
280-
281279
bounds_config = config[environment]
282280
print(f"Loading benchmark bounds for environment: {environment}")
283281

tests/ducktape/run_ducktape_test.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ def main():
4343
import pkg_resources
4444
version = pkg_resources.get_distribution('ducktape').version
4545
print(f"Using ducktape version: {version}")
46-
except:
46+
except Exception:
4747
print("Using ducktape version: unknown")
4848

4949
# 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)