suse-edge/Factory
SHA256
14
0
Fork 15
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove unused prometheus exporter #204

Merged
mchiappero merged 1 commits from mchiappero/Factory:prometheus into main 2025-07-07 23:31:30 +02:00
Conversation 2 Commits 1 Files Changed 3 -18
mchiappero commented 2025-07-04 17:31:49 +02:00
Owner
Copy Link

The Prometheus exporter is effectively, not only unused, but unusable, due to missing dependencies.
Since currently we don't have use case for it, opt for dropping the exporter entirely from the image.

Signed-off-by: Marco Chiappero marco.chiappero@suse.com

The Prometheus exporter is effectively, not only unused, but unusable, due to missing dependencies. Since currently we don't have use case for it, opt for dropping the exporter entirely from the image. Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
mchiappero added 1 commit 2025-07-04 17:31:49 +02:00
Remove unused prometheus exporter
All checks were successful
Check Release Manifest Local Charts Versions / Check Release Manifest Local Charts Versions (pull_request) Successful in 10s
Details
Build PR in OBS / Build PR in OBS (pull_request_target) Successful in -1m29s
Details
24801e072d 
Signed-off-by: Marco Chiappero <marco.chiappero@suse.com>
mchiappero changed title from Remove unused prometheus exporter to WIP: Remove unused prometheus exporter 2025-07-04 17:31:54 +02:00
mchiappero force-pushed prometheus from 24801e072d to b2ca623d14 2025-07-07 15:13:53 +02:00 Compare
mchiappero commented 2025-07-07 15:14:41 +02:00
Author
Owner
Copy Link

Commit message reworked and change tested, should be okay to merge.

Commit message reworked and change tested, should be okay to merge.
mchiappero requested review from amorgante 2025-07-07 15:16:13 +02:00
mchiappero requested review from nbelouin 2025-07-07 15:16:14 +02:00
mchiappero requested review from steven.hardy 2025-07-07 15:16:15 +02:00
mchiappero changed title from WIP: Remove unused prometheus exporter to Remove unused prometheus exporter 2025-07-07 15:27:54 +02:00
nbelouin approved these changes 2025-07-07 15:30:28 +02:00
nbelouin left a comment
Owner
Copy Link

LGTM, we should check we don't have anything in the chart (even unused/disabled) that may try to use this endpoint

LGTM, we should check we don't have anything in the chart (even unused/disabled) that may try to use this endpoint
mchiappero merged commit b2ca623d14 into main 2025-07-07 23:31:30 +02:00
mchiappero deleted branch prometheus 2025-07-07 23:31:31 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
amorgante
steven.hardy
nbelouin
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
amorgante atanasdinov atrendafilov dbekhit dprodanov eminguez fdegirmenci geoagriogiannis ipetrov117 jtomasek kzhelyazkov mchiappero nbelouin roxenham steven.hardy
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: suse-edge/Factory#204
No description provided.
Delete Branch "mchiappero/Factory:prometheus"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?