Bug: --config option not properly using defined database #172

New Issue

Dniel97 · 2024-09-06T05:23:41Z

Dniel97 commented

2024-09-06 05:23:41 +00:00

Expected:

When using the cli argument --config <cfg_dir> I expect that ARTEMiS would properly use the defined database connection inside <cfg_dir>/core.yaml.

Actual:

While using the cmd option --config (f.e. python index.py --config config_test it will actually use the databse connection inside config/core.yaml. The interesting part is that alembic can succesfully create the database table alembic_version from the cli cdefined config folder but ARTEMiS will always use the databse connection from config/core.yaml.

Reproduction steps:

Rename the example_config to something else than config, f.e. config_test (make sure to not have a folder called config!)
Correctly adjust the config_test/core.yaml with your database credentials
Execute python dbutils.py --config config_test create (this will fail)
python index.py --config config_test (this will also fail)

### Expected: When using the cli argument `--config <cfg_dir>` I expect that ARTEMiS would properly use the defined database connection inside `<cfg_dir>/core.yaml`. ### Actual: While using the cmd option `--config` (f.e. `python index.py --config config_test` it will actually use the databse connection inside `config/core.yaml`. The interesting part is that alembic can succesfully create the database table `alembic_version` from the cli cdefined config folder but ARTEMiS will always use the databse connection from `config/core.yaml`. ### Reproduction steps: 1. Rename the `example_config` to something else than `config`, f.e. `config_test` (make sure to not have a folder called `config`!) 2. Correctly adjust the `config_test/core.yaml` with your database credentials 3. Execute `python dbutils.py --config config_test create` (this will fail) 4. `python index.py --config config_test` (this will also fail)

Dniel97 added the

bug

label 2024-09-06 05:23:41 +00:00

Hay1tsme referenced this issue from a commit

2024-09-06 14:37:28 +00:00

Fix --config option not being respected, fixes #172

Dniel97 commented

2024-09-14 09:36:08 +00:00

Can confirm d1048694d4 fixed the bug, thanks!

Can confirm d1048694d4 fixed the bug, thanks!

Dniel97 closed this issue

2024-09-14 09:36:08 +00:00