diff --git a/config/main.py b/config/main.py index 6641328c36..8b16a42b6c 100644 --- a/config/main.py +++ b/config/main.py @@ -1907,8 +1907,8 @@ def override_config_table(db, input_config_db, dry_run): if ns in config_input.keys(): ns_config_input = config_input[ns] else: - click.secho("Wrong config format! {} not found in asic config! cannot override.. abort".format(ns)) - sys.exit(1) + click.echo("Override config not present for {}".format(ns)) + continue if not ns_config_input: # if ns_config_input is not defined, define it # it could be single-asic dut, or config_input is empty diff --git a/tests/config_override_input/multi_asic_missing_asic.json b/tests/config_override_input/multi_asic_missing_asic.json index 4399bfb32e..db8ba8ec80 100644 --- a/tests/config_override_input/multi_asic_missing_asic.json +++ b/tests/config_override_input/multi_asic_missing_asic.json @@ -1,5 +1,8 @@ { "localhost": { "DEVICE_METADATA": {} + }, + "asic0": { + "DEVICE_METADATA": {} } } diff --git a/tests/config_override_test.py b/tests/config_override_test.py index beeafaa82b..19d2ddc197 100644 --- a/tests/config_override_test.py +++ b/tests/config_override_test.py @@ -413,9 +413,9 @@ def read_json_file_side_effect(filename): runner = CliRunner() result = runner.invoke(config.config.commands["override-config-table"], ['golden_config_db.json'], obj=db) - assert "not found in asic config" in result.output - # make sure program aborted with return code 1 - assert result.exit_code == 1 + assert "Override config not present for asic1" in result.output + # make sure program did not abort + assert result.exit_code == 0 @classmethod def teardown_class(cls):