Skip to content

Conversation

@kush-elastic
Copy link
Contributor

@kush-elastic kush-elastic commented Sep 19, 2024

Re-Raised from: #40744

  • Enhancement

The vSphere Performance API offers support for Intervals and Levels, allowing users to collect various types of counters and metrics from vSphere environments. Depending on the configured Interval, the API provides detailed or aggregated performance data from vSphere.

For more information, refer to the following resources:

We can add support for configuration that help users collect the specific performance data they need.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Related issues


This is manually backport of pull request #40678.

…ance API (#40678)

* initial commit for intervalId supports for performance metrics

* update docs and fix CI

* Add changelog entry

* fix CI

* resolve review comments

* fix loggers

* resolved review comments

* update versions

* update UTs

* update integration tests

* 10s -> 20s

* Update CHANGELOG.next.asciidoc

Co-authored-by: Aman <[email protected]>

* Update metricbeat/docs/modules/vsphere.asciidoc

Co-authored-by: Aman <[email protected]>

* make update

* add recover for ToMetricSeries panic

* return error instead just logging it.

* remove restriction of interval IDs

* remove unnecessary validations

* remove recover and add empty condition

* update changelog entry

* Fix wrapping of errors in loggers

* update data.json

* update data.json

* fix CI and loggers

* update changelog entries

* make update

* fix changelog entries

* update changelog entry

---------

Co-authored-by: Aman <[email protected]>
(cherry picked from commit c75a7a4)
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 19, 2024
@kush-elastic kush-elastic self-assigned this Sep 19, 2024
@kush-elastic kush-elastic added enhancement Metricbeat Metricbeat Module:beat The beat Metricbeat module Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team labels Sep 19, 2024
@mergify
Copy link
Contributor

mergify bot commented Sep 19, 2024

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b mergify/bp/8.x/pr-40678 upstream/mergify/bp/8.x/pr-40678
git merge upstream/8.x
git push upstream mergify/bp/8.x/pr-40678

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Sep 19, 2024
@kush-elastic kush-elastic marked this pull request as ready for review September 30, 2024 06:16
@kush-elastic kush-elastic requested review from a team as code owners September 30, 2024 06:16
@kush-elastic kush-elastic requested review from VihasMakwana and faec and removed request for a team September 30, 2024 06:16
- Log the total time taken for GCP `ListTimeSeries` and `AggregatedList` requests {pull}40661[40661]
- Add `use_kubeadm` config option in kubernetes module in order to toggle kubeadm-config api requests {pull}40086[40086]
- Log the total time taken for GCP `ListTimeSeries` and `AggregatedList` requests {pull}40661[40661]
- Add new metrics for the vSphere Host metricset. {pull}40429[40429]
Copy link
Member

Choose a reason for hiding this comment

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

Is this used from the last commit in main/8.15 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is after something merged to 8.x.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think i should remove them

Copy link
Member

Choose a reason for hiding this comment

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

Yes, lets go as and when whatever was merged.

@ishleenk17
Copy link
Member

Please use the format/description for backport as done here
https://github.com/elastic/beats/pull/40828/files

@kush-elastic kush-elastic changed the title [Metricbeat][vSphere] Support for configurable IntervalId for perform… [8.x](backport #40678) [Metricbeat][vSphere] Support for configurable IntervalId for performance API Sep 30, 2024
@ishleenk17
Copy link
Member

This is an automatic backport of pull request #40678 done by Mergify.

Its manually create, correct ? Pls update

- Log the total time taken for GCP `ListTimeSeries` and `AggregatedList` requests {pull}40661[40661]
- Add `use_kubeadm` config option in kubernetes module in order to toggle kubeadm-config api requests {pull}40086[40086]
- Log the total time taken for GCP `ListTimeSeries` and `AggregatedList` requests {pull}40661[40661]
- Add new metrics for the vSphere Host metricset. {pull}40429[40429]
Copy link
Member

Choose a reason for hiding this comment

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

Yes, lets go as and when whatever was merged.

@kush-elastic kush-elastic merged commit 35f7b8e into 8.x Oct 1, 2024
@kush-elastic kush-elastic deleted the mergify/bp/8.x/pr-40678 branch October 1, 2024 06:15
@khushijain21 khushijain21 mentioned this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Metricbeat Metricbeat Module:beat The beat Metricbeat module Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants