Hi Guys, Have we fixed the two issues discussed in this thread? If not, better to track it via a JIRA.
Thanks, Raj. On Thu, Sep 10, 2015 at 4:53 PM, Pubudu Gunatilaka <pubu...@wso2.com> wrote: > Hi, > > +1 for this approach > > What we initially thought was to list down all the parameters in the > module.ini file which are not default values. User can either override > those values from environment variables or use the module.ini file. > > Thank you! > > On Thu, Sep 10, 2015 at 10:16 AM, Rajkumar Rajaratnam <rajkum...@wso2.com> > wrote: > >> What I meant is, why don't we do this? >> >> if CONFIG_PARAM_LOCAL_MEMBER_HOST is set { >> <parameter name="localMemberHost">{{ CONFIG_PARAM_LOCAL_MEMBER_HOST >> }}</parameter> >> } else { >> <parameter name="localMemberHost">127.0.0.1</parameter> >> } >> >> Thanks. >> >> On Thu, Sep 10, 2015 at 3:59 PM, Rajkumar Rajaratnam <rajkum...@wso2.com> >> wrote: >> >>> Any reason to treat the following config parameters differently, than >>> other config parameters? >>> >>> CONFIG_PARAM_CLUSTERING=false >>> CONFIG_PARAM_DOMAIN=wso2.carbon.domain >>> CONFIG_PARAM_WKA_MEMBERS=[127.0.0.1:4000] >>> CONFIG_PARAM_LOCAL_MEMBER_HOST=127.0.0.1 >>> CONFIG_PARAM_LOCAL_MEMBER_PORT=4000 >>> CONFIG_PARAM_PORT_OFFSET=0 >>> CONFIG_PARAM_MB_PORT=61616 >>> CONFIG_PARAM_EMBEDDED_LDAP=true >>> >>> These are enabled in module.ini file by default, where as other >>> parameters are disabled. And the default values of these config elements >>> are removed from templates files and moved to module.ini. >>> >>> This is not consistent with other config parameters. Because of this, we >>> always have to pass these parameters to configurator, even though we don't >>> need to override the default ones. >>> >>> By accident, I commented out all the config parameters from module.ini >>> and I have passed only the needed parameters to the configurator as >>> environment variables via puppet. I ended up with some issues since I >>> haven't pass the above parameters as environment variables (even though I >>> don't want to override the default ones!). So isn't good if leave the >>> default values in the template files itself? This way, we don't have >>> explicitly pass the default values, just like other config parameters. >>> >>> wdyt? >>> >>> Thanks, >>> Raj. >>> >>> On Tue, Sep 8, 2015 at 1:19 AM, Rajkumar Rajaratnam <rajkum...@wso2.com> >>> wrote: >>> >>>> >>>> >>>> On Tue, Sep 8, 2015 at 1:16 AM, Reka Thirunavukkarasu <r...@wso2.com> >>>> wrote: >>>> >>>>> Hi Raj, >>>>> >>>>> Yah..Seems some old reference is there in the >>>>> master-datasources.xml.template which will not affect the configurations, >>>>> if the parameters are defined correctly in the cartridge definition. >>>>> Please >>>>> ignore the section with CONFIG_PARAM_CONFIG_DB_APIM_URL. >>>>> >>>>> Please refer [1] for the sample configuration. We will clean this in >>>>> the immediate next release(beta).. >>>>> >>>> >>>> +1. Not a blocker or urgent. >>>> >>>> >>>>> >>>>> [1] >>>>> https://github.com/wso2/product-private-paas/blob/master/samples/cartridges/ec2/wso2am-190/wso2am-190-gw-manager.json >>>>> >>>>> Thanks, >>>>> Reka >>>>> >>>>> On Mon, Sep 7, 2015 at 6:19 PM, Rajkumar Rajaratnam < >>>>> rajkum...@wso2.com> wrote: >>>>> >>>>>> Hi, >>>>>> >>>>>> I think API Manager's master-datasources.xml.template is messed up. >>>>>> For example, same db username/password parameter is used for all the >>>>>> databases and jdbc/WSO2REG_DB is defined twice with two different >>>>>> if-checks, why? >>>>>> >>>>>> Please check and fix [1]. Please check other modules also. >>>>>> >>>>>> [1] >>>>>> https://github.com/wso2/product-private-paas/blob/master/cartridges/templates-modules/wso2am-1.9.0/templates/repository/conf/datasources/master-datasources.xml.template >>>>>> >>>>>> Thanks, >>>>>> Raj. >>>>>> >>>>>> -- >>>>>> Rajkumar Rajaratnam >>>>>> Committer & PMC Member, Apache Stratos >>>>>> Software Engineer, WSO2 >>>>>> >>>>>> Mobile : +94777568639 >>>>>> Blog : rajkumarr.com >>>>>> >>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> Dev mailing list >>>>>> Dev@wso2.org >>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> Reka Thirunavukkarasu >>>>> Senior Software Engineer, >>>>> WSO2, Inc.:http://wso2.com, >>>>> Mobile: +94776442007 >>>>> >>>>> >>>>> >>>> >>>> >>>> -- >>>> Rajkumar Rajaratnam >>>> Committer & PMC Member, Apache Stratos >>>> Software Engineer, WSO2 >>>> >>>> Mobile : +94777568639 >>>> Blog : rajkumarr.com >>>> >>> >>> >>> >>> -- >>> Rajkumar Rajaratnam >>> Committer & PMC Member, Apache Stratos >>> Software Engineer, WSO2 >>> >>> Mobile : +94777568639 >>> Blog : rajkumarr.com >>> >> >> >> >> -- >> Rajkumar Rajaratnam >> Committer & PMC Member, Apache Stratos >> Software Engineer, WSO2 >> >> Mobile : +94777568639 >> Blog : rajkumarr.com >> >> _______________________________________________ >> Dev mailing list >> Dev@wso2.org >> http://wso2.org/cgi-bin/mailman/listinfo/dev >> >> > > > -- > > *Pubudu Gunatilaka* > Software Engineer > WSO2, Inc.: http://wso2.com > lean.enterprise.middleware > mobile: +94 77 4078049 > -- Rajkumar Rajaratnam Committer & PMC Member, Apache Stratos Software Engineer, WSO2 Mobile : +94777568639 Blog : rajkumarr.com
_______________________________________________ Dev mailing list Dev@wso2.org http://wso2.org/cgi-bin/mailman/listinfo/dev