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

packages/prometheus: Upgrade to package format_version v3 #11988

Merged
merged 6 commits into from
Dec 5, 2024

Conversation

shmsr
Copy link
Member

@shmsr shmsr commented Dec 3, 2024

Proposed commit message

Upgrade to package format_version v3. Tested using elastic-package (pipeline, static, etc.) and also linting and building. Also did basic testing by scraping data from Prometheus Server itself and populating the dashboard. Works as expected. For testing, I used 8.16.1 cluster.

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.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Related issues

#8028

Screenshots

Dashboards screenshots after using collector to scrape metrics from Prometheus Server itself.

Screenshot 2024-12-04 at 11 21 22 PM Screenshot 2024-12-04 at 11 21 51 PM

@shmsr shmsr self-assigned this Dec 3, 2024
@shmsr shmsr added the enhancement New feature or request label Dec 3, 2024
@shmsr shmsr requested review from ishleenk17 and muthu-mps December 3, 2024 17:54
@shmsr shmsr marked this pull request as ready for review December 4, 2024 05:11
@shmsr shmsr requested review from a team as code owners December 4, 2024 05:11
@ishleenk17
Copy link
Contributor

Nice to see that we did not have to exclude any checks in validaton.yml

@shmsr
Copy link
Member Author

shmsr commented Dec 4, 2024

@elastic/obs-cloudnative-monitoring Could someone PTAL? Thank you!

@shmsr
Copy link
Member Author

shmsr commented Dec 4, 2024

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @shmsr

@andrewkroh andrewkroh added Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team [elastic/obs-cloudnative-monitoring] Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Dec 4, 2024
@gizas
Copy link
Contributor

gizas commented Dec 4, 2024

LGTM!
I only did a package build! Apart from that it would be useful to such stories to add the screenshots section or some verification logs that some basic tests had run and nothing broke

@shmsr
Copy link
Member Author

shmsr commented Dec 4, 2024

@gizas Makes sense. I should've put it. So for testing I have all the elastic-package tests (e.g., pipeline, static, etc.), and also linting to check for validations (SRV...). Everything looks good. I did basic testing by scraping data from Prometheus Server itself and populated the dashboard. I'll put the pics here:

Screenshot 2024-12-04 at 11 21 22 PM Screenshot 2024-12-04 at 11 21 51 PM

I have also updated the PR description.

@shmsr shmsr merged commit e0f1121 into elastic:main Dec 5, 2024
5 checks passed
@elastic-vault-github-plugin-prod

Package prometheus - 1.21.0 containing this change is available at https://epr.elastic.co/package/prometheus/1.21.0/

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:prometheus Prometheus Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team [elastic/obs-cloudnative-monitoring] Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants