Hi Yong,

This proposal will need to be reviewed with the multi-OS technical leads (Brent, Saul). Once reviewed and accepted, it will need to tracked via the appropriate story in storyboard.

 

Regards,

Ghada

 

From: Hu, Yong [mailto:yong.hu@intel.com]
Sent: Friday, September 28, 2018 8:24 PM
To: starlingx-discuss@lists.starlingx.io
Subject: [Starlingx-discuss] [multi-os] replace "/etc" in config files by a variable "sysconfdir"

 

As a part of initiatives, I would like to suggest to replace “/etc”, which is hardcoded in some StarlingX config files,

by a variable something like “sysconfdir”, which would be “instantiated” a bit later upon the OS type (CentOS, ClearLinux or Ubuntu).

 

Taking “stx-nfv/nfv/nfv-vim/nfv_vim/config.ini” as an example,

“/etc” is coded in this “config.ini” directly to refer other config files:

config_file=/etc/nfv/nfv_plugins/alarm_handlers/config.ini

config_file=/etc/nfv/nfv_plugins/nfvi_plugins/config.ini

 

My recommendation is to use a variable “sysconfdir” as instead, and to “define” this variable on later stage,

like in “nfv.spec” for CentOS, in which “sysconfdir” is “/etc”.

 

The benefit I see here is in other OSs, such as Clear Linux in which we can set different value,

for example, “/usr/local/etc” for this variable “sysconfdir”.

 

Should you have any questions or feedback, please let me know.

If I don’t see objections with this kind of changes, I will make some patches accordingly afterward.