Hi Imesh,

The current setup is as following:

Configurator check for module.ini and identify all the variables.
(Un-commented variables)
If read from environment is enabled, the values for these variables will be
read from environment variables. If these values are not set as environment
variables then the default values from the module.ini will be read.

What we can do is read environment variables with prefix CONFIG_PARAMS_*
and read them if read_from_env is true. But it will reduce the control over
variables.

WDYT ?


On Thu, Jun 25, 2015 at 10:09 AM, Imesh Gunaratne <im...@wso2.com> wrote:

> I see a problem with the existing logic.
>
> IMO configurator should give priority to the environment variables if that
> option is enabled. Requiring to comment variables in module.ini is an
> overhead.
>
> Thanks
>
>
> On Thursday, June 25, 2015, Anuruddha Liyanarachchi <anurudd...@wso2.com>
> wrote:
>
>> Hi Lahiru,
>>
>> If we do that we will always have to set all the parameters.
>>
>> On Thu, Jun 25, 2015 at 8:23 AM, Lahiru Sandaruwan <lahi...@wso2.com>
>> wrote:
>>
>>> Ah.. I see, Thanks.
>>>
>>> Shall we make it read those values even if those values are commented in
>>> module.ini?
>>>
>>> Thanks.
>>>
>>> On Thu, Jun 25, 2015 at 8:14 AM, Anuruddha Liyanarachchi <
>>> anurudd...@wso2.com> wrote:
>>>
>>>> Hi Lahiru,
>>>>
>>>> It looks like you haven't un-comment the line in module.ini. You need
>>>> to uncomment the lines relevant to the parameters you are sending as
>>>> payload, then only configurator will read those from environment and
>>>> override the value with environment value.
>>>>
>>>> Thanks,
>>>>
>>>> On Thu, Jun 25, 2015 at 12:25 AM, Lahiru Sandaruwan <lahi...@wso2.com>
>>>> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> I have sent parameters that are required to configure ESB cluster in
>>>>> payload as at [1]. Payload contains all the values i sent.
>>>>>
>>>>> But the configurator only picks few, as per the log [2].
>>>>>
>>>>> Any idea, on what i might be doing wrong?
>>>>>
>>>>> Thanks.
>>>>>
>>>>> [1]
>>>>>
>>>>> CONFIG_PARAM_CLUSTERING=true,
>>>>>
>>>>> CONFIG_PARAM_REGISTRY_DB_PASSWORD=asdfs,
>>>>>
>>>>> CONFIG_PARAM_WKA_MEMBERS="127.0.0.1:4000",
>>>>>
>>>>> CONFIG_PARAM_REGISTRY_DB_USER_NAME=root,
>>>>>
>>>>> CONFIG_PARAM_REGISTRY_DB_URL=jdbc:mysql://
>>>>> 172.17.8.1:3306/reg_db?autoReconnect=true,
>>>>>
>>>>> CONFIG_PARAM_USER_MGT_DB_USER_NAME=root,
>>>>>
>>>>> CONFIG_PARAM_DRIVER_CLASS_NAME=com.mysql.jdbc.Driver,
>>>>>
>>>>> CONFIG_PARAM_PT_HTTP_PROXY_PORT=8280,
>>>>>
>>>>> CONFIG_PARAM_LOCAL_MEMBER_HOST=127.0.0.1,
>>>>>
>>>>> CONFIG_PARAM_PT_HTTPS_PROXY_PORT=8243,
>>>>>
>>>>> CONFIG_PARAM_PORT_OFFSET=0,
>>>>>
>>>>> START_CMD=PCA,
>>>>>
>>>>> CONFIG_PARAM_DOMAIN=wso2.esb.com,
>>>>>
>>>>> CONFIG_PARAM_USER_MGT_DB_PASSWORD=asdf,
>>>>>
>>>>> CONFIG_PARAM_MEMBERSHIP_SCHEME=wka,
>>>>>
>>>>> CONFIG_PARAM_USER_MGT_DB_URL=jdbc:mysql://172.17.8.1:3306/user_db,
>>>>>
>>>>> CONFIG_PARAM_LOCAL_MEMBER_PORT=4100
>>>>>
>>>>> [2]
>>>>> 2015-06-24 18:15:43,408 __main__     INFO     Context generated:
>>>>> {'CONFIG_PARAM_DOMAIN': 'wso2.esb.com',
>>>>> 'CONFIG_PARAM_LOCAL_MEMBER_PORT': '4100', 
>>>>> 'CONFIG_PARAM_LOCAL_MEMBER_HOST':
>>>>> '127.0.0.1', 'CONFIG_PARAM_WKA_MEMBERS': {'127.0.0.1': '4000'},
>>>>> 'CONFIG_PARAM_PORT_OFFSET': '0', 'CONFIG_PARAM_SUB_DOMAIN': 'worker',
>>>>> 'CONFIG_PARAM_MEMBERSHIP_SCHEME': 'wka', 'CONFIG_PARAM_CLUSTERING': 
>>>>> 'true',
>>>>> 'CONFIG_PARAM_PT_HTTP_PROXY_PORT': '8280',
>>>>> 'CONFIG_PARAM_PT_HTTPS_PROXY_PORT': '8243'}
>>>>>
>>>>> --
>>>>> --
>>>>> Lahiru Sandaruwan
>>>>> Committer and PMC member, Apache Stratos,
>>>>> Senior Software Engineer,
>>>>> WSO2 Inc., http://wso2.com
>>>>> lean.enterprise.middleware
>>>>>
>>>>> phone: +94773325954
>>>>> email: lahi...@wso2.com blog: http://lahiruwrites.blogspot.com/
>>>>> linked-in: http://lk.linkedin.com/pub/lahiru-sandaruwan/16/153/146
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> *Thanks and Regards,*
>>>> Anuruddha Lanka Liyanarachchi
>>>> Software Engineer - WSO2
>>>> Mobile : +94 (0) 712762611
>>>> Tel      : +94 112 145 345
>>>> anurudd...@wso2.com
>>>>
>>>
>>>
>>>
>>> --
>>> --
>>> Lahiru Sandaruwan
>>> Committer and PMC member, Apache Stratos,
>>> Senior Software Engineer,
>>> WSO2 Inc., http://wso2.com
>>> lean.enterprise.middleware
>>>
>>> phone: +94773325954
>>> email: lahi...@wso2.com blog: http://lahiruwrites.blogspot.com/
>>> linked-in: http://lk.linkedin.com/pub/lahiru-sandaruwan/16/153/146
>>>
>>>
>>
>>
>> --
>> *Thanks and Regards,*
>> Anuruddha Lanka Liyanarachchi
>> Software Engineer - WSO2
>> Mobile : +94 (0) 712762611
>> Tel      : +94 112 145 345
>> anurudd...@wso2.com
>>
>
>
> --
> *Imesh Gunaratne*
> Senior Technical Lead
> WSO2 Inc: http://wso2.com
> T: +94 11 214 5345 M: +94 77 374 2057
> W: http://imesh.gunaratne.org
> Lean . Enterprise . Middleware
>
>
>


-- 
*Thanks and Regards,*
Anuruddha Lanka Liyanarachchi
Software Engineer - WSO2
Mobile : +94 (0) 712762611
Tel      : +94 112 145 345
a <thili...@wso2.com>nurudd...@wso2.com
_______________________________________________
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to