ibm-mas / ansible-devops

Ansible collection supporting devops for IBM Maximo Application Suite
https://ibm-mas.github.io/ansible-devops/
Eclipse Public License 2.0
49 stars 84 forks source link

is this variable MAS_APP_SETTINGS_DB2_SCHEMA only for DB2 or can be used for other supported DB servers #1343

Closed ekambaraml closed 1 month ago

ekambaraml commented 3 months ago

Manage install using Ansible playbook, The following env. variable is required.

export MAS_APP_SETTINGS_DB2_SCHEMA=maximo export MAS_APP_SETTINGS_TABLESPACE=maxdata export MAS_APP_SETTINGS_INDEXSPACE=maxindex

But the variable "MAS_APP_SETTINGS_DB2_SCHEMA" is misleading to assume, it is for DB2 only. Instead it can also be used with Oracle or MS SQL. Please confirm

ekambaraml commented 3 months ago

Example, customer Corning Inc. is using the Oracle Database on-perm.

ekambaraml commented 3 months ago

https://github.com/ibm-mas/ansible-devops/blob/master/docs/playbooks/oneclick-manage.md

JonahLuckett commented 2 months ago

Looking into this for you now - will update when I have an answer

durera commented 1 month ago

Yes, this variable should be renamed as mas_app_settings_db_schema rather than db2_schema. The 2 looks like a mistake/autopilot when naming the variable.

@JonahLuckett When fixing this please ensure the old name is still supported ... e.g. docs etc can stop talking about db2_schema and only talk about db_schema, but have the code support both.

mas_app_settings_db2_schema: "{{ lookup('env', 'MAS_APP_SETTINGS_DB2_SCHEMA') | default('maximo', true)}}"
mas_app_settings_db_schema: "{{ lookup('env', 'MAS_APP_SETTINGS_DB_SCHEMA') | default(mas_app_settings_db2_schema, true)}}"

If my thinking is correct:

So other than when we define the variables, we don't need to reference mas_app_settings_db2_schema anywhere in the docs or the code, but we can deliver this as a patch rather than a major as we are avoiding a breaking change