Skip to content

Conversation

@postnati
Copy link

This PR modernizes the wildfly-mixin to use grafonnet v11 and the signals architecture pattern.

@CLAassistant
Copy link

CLAassistant commented Oct 30, 2025

CLA assistant check
All committers have signed the CLA.

@postnati postnati changed the title Modernize wildfly-mixin to grafonnet v11 and signals architecture chore: Modernize wildfly-mixin Oct 30, 2025
@postnati postnati requested a review from schmikei October 30, 2025 17:57
@postnati postnati self-assigned this Oct 30, 2025
@postnati postnati force-pushed the chore/wildfly-mixin-modernization branch 2 times, most recently from 6cd1b59 to 216eb64 Compare November 11, 2025 15:56
@postnati postnati force-pushed the chore/wildfly-mixin-modernization branch from d357bd6 to 1c13b3b Compare November 21, 2025 12:51
@postnati
Copy link
Author

postnati commented Nov 21, 2025

Screenshots

Overview Dashboard

wildfly_overview_1 wildfly_overview_2

Datasource Dashboard

wildfly_datasource

Logs Dashboard

wildfly_logs

Copy link
Contributor

@schmikei schmikei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking really good to me aside from one thing with the config.libsonnet

alertsErrorRequestErrorRate: '30',
alertsErrorRejectedSessions: '20',
uid: 'wildfly',
dashboardTags: [self.uid],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
dashboardTags: [self.uid],
dashboardTags: [self.uid+-'mixin'],

This'd be a breaking default otherwise

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants