Skip to content

Conversation

inge4pres
Copy link
Contributor

Proposed commit message

We implement a new configuration structure for exposing the configuration of application metrics.
Users will be able to customize the HTTP endpoints for both Prometheus and expvar metrics.

Since we are going to release a beta for Profiling Self-Managed (running on-prem), these new configuration options will allow users to monitor more effectively their Profiling backend installations.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

How to test this PR locally

Build the package with elastic-package, deploy to a local instance of Kibana and deploy the integrations through the details page.

Related issues

https://github.com/elastic/prodfiler/issues/4140

@inge4pres inge4pres added enhancement New feature or request breaking change labels Nov 27, 2023
@inge4pres inge4pres self-assigned this Nov 27, 2023
@inge4pres inge4pres requested a review from a team as a code owner November 27, 2023 17:30
@elasticmachine
Copy link

elasticmachine commented Nov 27, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-12-01T17:17:16.971+0000

  • Duration: 36 min 12 sec

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@inge4pres inge4pres force-pushed the profiluing/customize-metrics-config branch from b85f2ac to 2569408 Compare November 28, 2023 14:08
Copy link
Member

@christos68k christos68k left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@florianl florianl left a comment

Choose a reason for hiding this comment

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

for saftey, we could also increase kibana.version in conditions in manifest.yml to `^8.12.0.

@inge4pres inge4pres force-pushed the profiluing/customize-metrics-config branch from f71cfd2 to 14ca2c0 Compare November 30, 2023 22:22
@inge4pres inge4pres requested a review from florianl November 30, 2023 22:23
@inge4pres
Copy link
Contributor Author

we could also increase kibana.version in conditions in manifest.yml to `^8.12.0.

Thanks! great idea 👍🏼

Signed-off-by: inge4pres <francesco.gualazzi@elastic.co>
Co-authored-by: Florian Lehner <florianl@users.noreply.github.com>
Co-authored-by: Christos Kalkanis <christos.kalkanis@elastic.co>
@inge4pres inge4pres force-pushed the profiluing/customize-metrics-config branch from 14ca2c0 to 8cc8e87 Compare December 1, 2023 17:16
@inge4pres inge4pres merged commit f6a433c into main Dec 1, 2023
@elasticmachine
Copy link

Package profiler_collector - 8.12.0 containing this change is available at https://epr.elastic.co/search?package=profiler_collector

@elasticmachine
Copy link

Package profiler_symbolizer - 8.12.0 containing this change is available at https://epr.elastic.co/search?package=profiler_symbolizer

florianl added a commit to elastic/kibana that referenced this pull request Dec 4, 2023
## Summary

With elastic/integrations#8582 the integration
packages for Universal Profiling got upgraded. With Kibana bundle these
new integration packages.

Signed-off-by: Florian Lehner <florian.lehner@elastic.co>
@andrewkroh andrewkroh added Integration:universal_profiling_collector Universal Profiling Collector Integration:universal_profiling_symbolizer Universal Profiling Symbolizer labels Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Integration:universal_profiling_collector Universal Profiling Collector Integration:universal_profiling_symbolizer Universal Profiling Symbolizer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants