Skip to content

FIX Preserve silent when loading config overrides#1500

Merged
romanlutz merged 1 commit intomicrosoft:mainfrom
biefan:preserve-silent-config-overrides
Mar 19, 2026
Merged

FIX Preserve silent when loading config overrides#1500
romanlutz merged 1 commit intomicrosoft:mainfrom
biefan:preserve-silent-config-overrides

Conversation

@biefan
Copy link
Copy Markdown
Contributor

@biefan biefan commented Mar 17, 2026

Summary

  • preserve the silent flag when ConfigurationLoader.load_with_overrides() reads default or explicit config files
  • add regression coverage for config-file-backed silent: true values

Problem

ConfigurationLoader.load_with_overrides() merges memory_db_type, initializers, initialization_scripts, and env_files from configuration files, but it drops the silent field entirely.

As a result, config files that set silent: true are loaded with silent=False, so initialization output is not suppressed even though the configuration explicitly requests it.

Testing

  • .venv/bin/pytest tests/unit/setup/test_configuration_loader.py -q

@romanlutz romanlutz changed the title Preserve silent when loading config overrides FIX Preserve silent when loading config overrides Mar 19, 2026
@romanlutz romanlutz merged commit 1264508 into microsoft:main Mar 19, 2026
37 checks passed
riyosha pushed a commit to riyosha/PyRIT that referenced this pull request Mar 24, 2026
jbolor21 pushed a commit to jbolor21/jbolor-PyRIT that referenced this pull request Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants