Re: [Starlingx-discuss] [multi-os] replace "/etc" in config files by a variable "sysconfdir"
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.
participants (1)
-
Khalil, Ghada