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

✨ Expose Hub metrics #240

Merged
merged 1 commit into from
Jun 1, 2023
Merged

✨ Expose Hub metrics #240

merged 1 commit into from
Jun 1, 2023

Conversation

mansam
Copy link
Contributor

@mansam mansam commented May 31, 2023

This exposes the Hub's prometheus metrics via an additional port on the existing Hub service. This is necessary to properly implement https://issues.redhat.com/projects/MTA/issues/MTA-402.

Related to konveyor/tackle2-hub#335

@mansam mansam requested a review from fbladilo May 31, 2023 22:37
Related to konveyor/tackle2-hub#335

Signed-off-by: Sam Lucidi <slucidi@redhat.com>
Copy link
Contributor

@fbladilo fbladilo left a comment

Choose a reason for hiding this comment

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

LGTM

@mansam mansam merged commit 0820397 into konveyor:main Jun 1, 2023
mansam added a commit to mansam/tackle2-operator that referenced this pull request Jun 1, 2023
This exposes the Hub's prometheus metrics via an additional port on the
existing Hub service. This is necessary to properly implement
https://issues.redhat.com/projects/MTA/issues/MTA-402.

Related to konveyor/tackle2-hub#335

Signed-off-by: Sam Lucidi <slucidi@redhat.com>
mansam added a commit that referenced this pull request Jun 5, 2023
This exposes the Hub's prometheus metrics via an additional port on the
existing Hub service. This is necessary to properly implement
https://issues.redhat.com/projects/MTA/issues/MTA-402.

Related to konveyor/tackle2-hub#335

Signed-off-by: Sam Lucidi <slucidi@redhat.com>
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.

2 participants