Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jmxEnabled property is not templated for the identity db #22220

Open
pabasara-mahindapala opened this issue Jan 9, 2025 · 0 comments
Open

jmxEnabled property is not templated for the identity db #22220

pabasara-mahindapala opened this issue Jan 9, 2025 · 0 comments

Comments

@pabasara-mahindapala
Copy link
Contributor

Description

The jmxEnabled property is not templated for the identity db in the master-datasources.xml.j2 file. Due to this jmxEanbled property has to be added under pool_options only for the identity db.

For consistency, this property should be templated for WSO2_IDENTITY_DB similar to WSO2_SHARED_DB, WSO2USER_DB and WSO2CONFIG_DB configurations. We have mentioned inconsistent configurations in the docs as well [1].

[1] https://is.docs.wso2.com/en/6.0.0/deploy/monitor/jmx-based-monitoring/#configuring-jmx-in-wso2-identity-server:~:text=Enabling%20JMX%20for%20a%20datasource

<datasource>
    <name>WSO2_SHARED_DB</name>
    <description>Shared Database for user and registry data</description>
    <jndiConfig>
        <name>jdbc/SHARED_DB</name>
    </jndiConfig>
    <definition type="RDBMS">
        <configuration>
            <url>{{database.shared_db.url}}</url>
            <username>{{database.shared_db.username}}</username>
            <password>{{database.shared_db.password}}</password>
            <driverClassName>{{database.shared_db.driver}}</driverClassName>
            {% for property_name,property_value in database.shared_db.pool_options.items() %}
            <{{property_name}}>{{property_value}}</{{property_name}}>
            {% endfor %}
            <jmxEnabled>{{database.shared_db.jmx_enable}}</jmxEnabled>
        </configuration>
    </definition>
</datasource>
<datasource>
    <name>WSO2_IDENTITY_DB</name>
    <description>Shared database for identity data</description>
    <jndiConfig>
        <name>jdbc/WSO2IdentityDB</name>
    </jndiConfig>
    <definition type="RDBMS">
        <configuration>
            <url>{{database.identity_db.url}}</url>
            <username>{{database.identity_db.username}}</username>
            <password>{{database.identity_db.password}}</password>
            <driverClassName>{{database.identity_db.driver}}</driverClassName>
            {% for property_name,property_value in database.identity_db.pool_options.items() %}
            <{{property_name}}>{{property_value}}</{{property_name}}>
            {% endfor %}
        </configuration>
    </definition>
</datasource>
</datasources>

Steps to Reproduce

N/A

Version

IS 7.0.0

Environment Details (with versions)

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants