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

Remove beta flag for Enterprise Search metricbeat module #33329

Closed
wants to merge 1 commit into from

Conversation

richkuz
Copy link

@richkuz richkuz commented Oct 12, 2022

We've been using this module internally for Stack monitoring since 8.2. I think we can remove the beta flag.

cc @aznick @carlosdelest @ioanatia

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.

We've been using this module internally for Stack monitoring since 8.2. I think we can remove the beta flag.

cc @aznick @carlosdelest @ioanatia
@richkuz richkuz added the docs label Oct 12, 2022
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 12, 2022
@elasticmachine
Copy link
Collaborator

💔 Build Failed

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: 2022-10-12T12:47:53.594+0000

  • Duration: 12 min 29 sec

Steps errors 2

Expand to view the steps failures

make check
  • Took 4 min 12 sec . View more details here
  • Description: make check
Error signal
  • Took 0 min 0 sec . View more details here
  • Description: Error 'hudson.AbortException: script returned exit code 2'

❕ Flaky test report

No test was executed to be analysed.

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

  • /package : Generate the packages and run the E2E tests.

  • /beats-tester : Run the installation tests with beats-tester.

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

@richkuz
Copy link
Author

richkuz commented Oct 12, 2022

I'm pausing on this PR for a moment, awaiting further discussion with @ioanatia on how we want to proceed in general with the Enterprise Search metricbeat module.

@richkuz richkuz assigned carlosdelest and unassigned richkuz Jan 30, 2023
@richkuz
Copy link
Author

richkuz commented Jan 30, 2023

Passing the baton to @carlosdelest . This PR is not urgent, but we should agree to cancel (defer) or merge it so it isn't lingering.

@carlosdelest
Copy link
Member

@ioanatia , what will be our strategy for Metricbeat? I think it's going to be replaced by Agent eventually, but I guess we can make it GA. Any reason not to?

@endorama endorama added the Team:Infra Monitoring UI - DEPRECATED Infrastructure Monitoring UI team - DEPRECATED - Use Team:Monitoring label Mar 1, 2023
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Mar 1, 2023
@carlosdelest
Copy link
Member

@ioanatia , thoughts on this one?

@carlosdelest
Copy link
Member

Closing as there is no potential benefit in getting metricbeat plugin GA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Team:Infra Monitoring UI - DEPRECATED Infrastructure Monitoring UI team - DEPRECATED - Use Team:Monitoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants