Skip to content

[Feature] Export metrics in OpenMetrics text format#295

Open
kaushikmadhu wants to merge 1 commit into
apache:mainfrom
kaushikmadhu:metrics
Open

[Feature] Export metrics in OpenMetrics text format#295
kaushikmadhu wants to merge 1 commit into
apache:mainfrom
kaushikmadhu:metrics

Conversation

@kaushikmadhu
Copy link
Copy Markdown

Purpose of this pull request

Implemented observability feature by exporting seatunnel-web metrics in openMetrics text standard. This will help us to raise necessary alerts in cases of resource utilization exceeding thresholds, test connection failures, or unavailability issues.
OpenMetrics format can integrate seamlessly with monitoring tools like grafana and prometheus.

Issue: apache/seatunnel#9191

Check list

@shashwatsai
Copy link
Copy Markdown
Contributor

Can we please approve the workflow runs.
CC: @corgy-w , @Hisoka-X , @arshadmohammad

@shashwatsai
Copy link
Copy Markdown
Contributor

@kaushikmadhu, please add the missing license headers, fix the codestyles.

@kaushikmadhu
Copy link
Copy Markdown
Author

Formatting is fixed and license headers added to new files. Please approve the workflow runs.

@shashwatsai
Copy link
Copy Markdown
Contributor

@arshadmohammad can you please re-approve the workflows?

@arshadmohammad
Copy link
Copy Markdown
Collaborator

Nowadays, workflows are executed without requiring approval. It seems that the CI process has already been completed, and some failures have been detected.

@kaushikmadhu
Copy link
Copy Markdown
Author

Please review this PR.

@kaushikmadhu
Copy link
Copy Markdown
Author

Please review and merge @arshadmohammad @Hisoka-X @corgy-w

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants