[BUG] Name mismatch for config section (workspace_default_config) about mage-ai

[csharplus]

@wangxiaoyou1993 workspace_default_config does not show up in the code at all. Based on the current implementation of repo_manager.py, should this variable be renamed to workspace_shared_config instead?

...
        self.variables_dir = None
        self.variables_retention_period = None
        self.workspace_initial_metadata = None
        self.workspace_shared_config = None
...

[mangobillbay]

Um, I don't really get it... I might have misunderstand the code, but I'll try to clarify my point:

In the repo_manager.py (at line 147) I can see it reads config from workspace_config_defaults.

However in the provided template for metadata.yaml (at line 10) there is an object/section called workspace_default_config.

I believe the idea is that these names should match, but they don't. So when I use the provided template and make changes to the default workspace configuration it nothing happens. If I change (correct?) the name in the template file it works as expected.

So my suggestion is to change the name in the metadata.yaml from workspace_default_config to workspace_config_defaults.

[wangxiaoyou1993]

It might be a name mismatch. We'll fix it.