You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 27, 2022. It is now read-only.
Let's define as a group issue #138 first, and then we can properly fix this. In any case, we will still have to move these hard-coded instantiations outside of the code-base.
Hi Guys,
I propose we move the following, from being hard-coded in ims/common/constants.py to the config file that usually resides in
/etc/bmi/
:BMI_ADMIN_PROJECT = "bmi_infra"
This causes issues with deployment to new environments.
To fix this, it requires changes to the following five files:
/etc/bmi/bmiconfig*.cfg
with something like this:ims/common/constants.py
with something like this:ims/common/bmi_config.py
with something like this:cfg.option(constants.BMI_SECTION, constants.BMI_ADMIN_PROJECT_OPT)
In
ims/cli/cli.py
change allconstants.BMI_ADMIN_PROJECT
to_cfg.bmi.BMI_ADMIN_PROJECT
.In
ims/rpc/server/rpc_server.py
change the following two lines:server.remake_mappings()
->server.remake_mappings( server.remake_mappings( cfg.bmi.BMI_ADMIN_PROJECT )
with BMI("", "", constants.BMI_ADMIN_PROJECT) as bmi:
->with BMI("", "", BMI_ADMIN_PROJECT) as bmi:
Let me know, as I would like to have done fairly quickly for the 0.5 release.
Thanks,
Paul
The text was updated successfully, but these errors were encountered: