Skip to content

[Beats] Added ssl config support to beats module#4916

Merged
mohamedhamed-ahmed merged 2 commits intoelastic:mainfrom
mohamedhamed-ahmed:4666-add-ssl-config-to-beats-module
Jan 9, 2023
Merged

[Beats] Added ssl config support to beats module#4916
mohamedhamed-ahmed merged 2 commits intoelastic:mainfrom
mohamedhamed-ahmed:4666-add-ssl-config-to-beats-module

Conversation

@mohamedhamed-ahmed
Copy link
Copy Markdown
Contributor

@mohamedhamed-ahmed mohamedhamed-ahmed commented Jan 2, 2023

What does this PR do?

Add ssl configuration options for elasticsearch integration module

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that the new ssl configuration is being passed properly to the metricbeat module

How to this PR was tested

  • Applied changes locally, and used elastic-package to spawn up a complete stack locally.
  • Made sure the new fields are visible in the UI as below:

Screenshot 2022-12-22 at 15 21 23

  • Checked that the policy has the new fields mapped properly

Screenshot 2022-12-22 at 15 46 55

  • Spawned an Elastic agent locally and checked the matribeat logs to make sure that it initially fails when connecting to ES and then succeeds to establish a connection to ES when ssl configuration are passed properly

Related issues

@mohamedhamed-ahmed mohamedhamed-ahmed added bug Something isn't working, use only for issues draft Draft v8.7.0 Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services Integration:beat Beat labels Jan 2, 2023
@mohamedhamed-ahmed mohamedhamed-ahmed requested a review from a team as a code owner January 2, 2023 12:56
@mohamedhamed-ahmed mohamedhamed-ahmed self-assigned this Jan 2, 2023
@mohamedhamed-ahmed mohamedhamed-ahmed mentioned this pull request Jan 2, 2023
Closed
4 tasks
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Jan 2, 2023 •
edited
Loading

💚 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-01-09T12:41:24.463+0000

  • Duration: 16 min 12 sec

Test stats 🧪

Test Results
Failed 0
Passed 8
Skipped 0
Total 8

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

@klacabane
Copy link
Copy Markdown
Contributor

klacabane commented Jan 9, 2023

/test

@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Jan 9, 2023

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (0/0) 💚
Files 100.0% (0/0) 💚 33.333
Classes 100.0% (0/0) 💚 33.333
Methods 100.0% (6/6) 💚 20.312
Lines 100.0% (0/0) 💚 25.735
Conditionals 100.0% (0/0) 💚

@mohamedhamed-ahmed mohamedhamed-ahmed merged commit 4944997 into elastic:main Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@klacabane klacabane klacabane approved these changes

Assignees

@mohamedhamed-ahmed mohamedhamed-ahmed

Labels

bug Something isn't working, use only for issues draft Draft Integration:beat Beat Team:Infra Monitoring UI - DEPRECATED Label for the Infrastructure Monitoring UI team. - DEPRECATED - Use Team:obs-ux-infra_services v8.7.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mohamedhamed-ahmed @elasticmachine @klacabane