Skip to content

[metricbeat] [mongodb] increase default polling interval period 10s -> 60s #44781

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.

Sign up for GitHub

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

Jump to bottom
Merged
mykola-elastic merged 8 commits into from
Jun 17, 2025
Merged

[metricbeat] [mongodb] increase default polling interval period 10s -> 60s #44781

mykola-elastic merged 8 commits into from
Jun 17, 2025

Conversation

@mykola-elastic
Copy link
Contributor

@mykola-elastic mykola-elastic commented Jun 12, 2025 •
edited
Loading

Proposed commit message

See title

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

@mykola-elastic mykola-elastic self-assigned this Jun 12, 2025
@mykola-elastic mykola-elastic added enhancement Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team backport-active-all Automated backport with mergify to all the active branches labels Jun 12, 2025
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jun 12, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jun 12, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mykola-elastic mykola-elastic requested a review from shmsr June 12, 2025 12:35
@mykola-elastic mykola-elastic marked this pull request as ready for review June 12, 2025 13:00
@mykola-elastic mykola-elastic requested review from a team as code owners June 12, 2025 13:00
@mykola-elastic mykola-elastic requested review from faec and rdner June 12, 2025 13:00
shmsr
shmsr reviewed Jun 13, 2025
View reviewed changes
shmsr
shmsr reviewed Jun 13, 2025
View reviewed changes
Copy link
Member

@shmsr shmsr left a comment

Choose a reason for hiding this comment

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

Just one comment; rest looks good!

@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team label Jun 15, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jun 15, 2025

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@ishleenk17
Copy link
Member

ishleenk17 commented Jun 16, 2025

@mykola-elastic : Backport of this PR would be needed as well.

@mykola-elastic mykola-elastic changed the title [metricbeat] [mongodb] increase period 10s -> 60s [metricbeat] [mongodb] increase default polling interval period 10s -> 60s Jun 17, 2025
@mykola-elastic
Copy link
Contributor Author

mykola-elastic commented Jun 17, 2025

Hello @pierrehilbert @faec @rdner , can please any of you review this PR today? The feature freeze for 9.0.3 is today, if you review this, we can add this change to 9.0.3. The PR requires someone from @elastic/elastic-agent-data-plane to review it.

@pierrehilbert
Copy link
Contributor

pierrehilbert commented Jun 17, 2025

@mykola-elastic just approved it, thanks for updating the changelog.

@pierrehilbert
Copy link
Contributor

pierrehilbert commented Jun 17, 2025

@colleenmcginnis could you please take a look here for the changelog part?

@colleenmcginnis
Copy link
Contributor

colleenmcginnis commented Jun 17, 2025

@colleenmcginnis could you please take a look here for the changelog part?

If this is going into the 9.0.3 release, can you please:

  1. Create a new section for 9.0.3 in docs/release-notes/index.md.
  2. Add this changelog entry to that new section.

@pierrehilbert is there any reason to keep CHANGELOG.next.asciidoc on the main branch? We're no longer publishing any AsciiDoc files from the main branch, but I want to make sure there isn't another reason to keep it.

@mykola-elastic
Copy link
Contributor Author

mykola-elastic commented Jun 17, 2025 •
edited
Loading

@colleenmcginnis so, main goes to 9.1.0 and the backport-9.0 branch will go to 9.0.3? Do I need to add this in this only in the back port PR?

@mykola-elastic mykola-elastic requested a review from a team as a code owner June 17, 2025 13:44
@colleenmcginnis
Copy link
Contributor

colleenmcginnis commented Jun 17, 2025

Oh. Wait. Maybe you can keep it in the CHANGELOG.next.asciidoc and we'll pull it into the updated docs via automation when @pierrehilbert creates the release notes?

@mykola-elastic mykola-elastic force-pushed the increase-default-interval branch from 413af0d to 7bac6f4 Compare June 17, 2025 13:47
@pierrehilbert
Copy link
Contributor

pierrehilbert commented Jun 17, 2025

@pierrehilbert is there any reason to keep CHANGELOG.next.asciidoc on the main branch? We're no longer publishing any AsciiDoc files from the main branch, but I want to make sure there isn't another reason to keep it.

We will need to onboard all teams to a new system before getting ride of that one but as usual I will generate a changelog you will be able to use to generate the real one.

@mykola-elastic mykola-elastic merged commit f889b64 into elastic:main Jun 17, 2025
60 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Jun 17, 2025

@Mergifyio backport 8.17 8.18 8.19 9.0

@mergify
Copy link
Contributor

mergify bot commented Jun 17, 2025 •
edited
Loading

backport 8.17 8.18 8.19 9.0

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Jun 17, 2025
@mykola-elastic @mergify
[metricbeat] [mongodb] increase default polling interval period 10s…
17a3d71 
… -> 60s (#44781)

(cherry picked from commit f889b64)
mergify bot pushed a commit that referenced this pull request Jun 17, 2025
@mykola-elastic @mergify
[metricbeat] [mongodb] increase default polling interval period 10s…
ad9c2dd 
… -> 60s (#44781)

(cherry picked from commit f889b64)
mergify bot pushed a commit that referenced this pull request Jun 17, 2025
@mykola-elastic @mergify
[metricbeat] [mongodb] increase default polling interval period 10s…
6d7abd8 
… -> 60s (#44781)

(cherry picked from commit f889b64)
This was referenced Jun 17, 2025
Merged
Merged
Merged
mergify bot pushed a commit that referenced this pull request Jun 17, 2025
@mykola-elastic @mergify
[metricbeat] [mongodb] increase default polling interval period 10s…
ac4afc4 
… -> 60s (#44781)

(cherry picked from commit f889b64)
@mergify mergify bot mentioned this pull request Jun 17, 2025
Merged
6 tasks
mykola-elastic added a commit that referenced this pull request Jun 17, 2025
@mergify @mykola-elastic
[metricbeat] [mongodb] increase default polling interval period 10s…
1ffe850 
… -> 60s (#44781) (#44878)

(cherry picked from commit f889b64)

Co-authored-by: mykola-elastic <[email protected]>
mykola-elastic added a commit that referenced this pull request Jun 18, 2025
@mergify @mykola-elastic
[8.18](backport #44781) [metricbeat] [mongodb] increase default polli…
04a802b 
…ng interval `period` 10s -> 60s (#44875)

* [metricbeat] [mongodb] increase default polling interval `period` 10s -> 60s (#44781)

(cherry picked from commit f889b64)

* Update CHANGELOG.next.asciidoc

* make update

---------

Co-authored-by: mykola-elastic <[email protected]>
mykola-elastic added a commit that referenced this pull request Jun 19, 2025
@mergify @mykola-elastic
[8.19](backport #44781) [metricbeat] [mongodb] increase default polli…
b09e0ec 
…ng interval `period` 10s -> 60s (#44876)

* [metricbeat] [mongodb] increase default polling interval `period` 10s -> 60s (#44781)

(cherry picked from commit f889b64)

* make update

---------

Co-authored-by: mykola-elastic <[email protected]>
mykola-elastic added a commit that referenced this pull request Jun 19, 2025
@mergify @mykola-elastic
[8.17](backport #44781) [metricbeat] [mongodb] increase default polli…
ff7bc48 
…ng interval `period` 10s -> 60s (#44877)

* [metricbeat] [mongodb] increase default polling interval `period` 10s -> 60s (#44781)

(cherry picked from commit f889b64)

* Update CHANGELOG.next.asciidoc

* Update CHANGELOG.next.asciidoc

* make update

---------

Co-authored-by: mykola-elastic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pierrehilbert pierrehilbert pierrehilbert approved these changes

@shmsr shmsr shmsr approved these changes

@rdner rdner Awaiting requested review from rdner rdner is a code owner automatically assigned from elastic/elastic-agent-data-plane

@faec faec Awaiting requested review from faec faec is a code owner automatically assigned from elastic/elastic-agent-data-plane

@ishleenk17 ishleenk17 Awaiting requested review from ishleenk17

Assignees

@mykola-elastic mykola-elastic

Labels

backport-active-all Automated backport with mergify to all the active branches enhancement Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

metricbeat/module/mongodb: Increase the default interval in configuration and documentation

6 participants

@mykola-elastic @elasticmachine @ishleenk17 @pierrehilbert @colleenmcginnis @shmsr