Skip to content

[O11y][StatsD Input] Add system test cases - #6594

Merged
ali786XI merged 7 commits into
elastic:mainfrom
ali786XI:statsd_input_system_test
Jun 30, 2023
Merged

[O11y][StatsD Input] Add system test cases#6594
ali786XI merged 7 commits into
elastic:mainfrom
ali786XI:statsd_input_system_test

Conversation

@ali786XI

@ali786XI ali786XI commented Jun 16, 2023

Copy link
Copy Markdown
Contributor
  • Enhancement

What does this PR do?

  • This PR adds system test cases for the StatsD Input package.

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.

How to test this PR locally

  • Clone integrations repo.
  • Install elastic-package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/statsd_input directory.
  • Run the following command to run the system tests. elastic-package test system

Related issues
@ali786XI ali786XI added the Integration:statsd_input StatsD Input label Jun 16, 2023
@ali786XI ali786XI self-assigned this Jun 16, 2023
@ali786XI
ali786XI requested a review from milan-elastic June 16, 2023 09:53
@elasticmachine

elasticmachine commented Jun 16, 2023 •
edited
Loading

Copy link
Copy Markdown

💚 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-06-26T16:25:03.762+0000

  • Duration: 15 min 58 sec

Test stats 🧪

Test Results
Failed 0
Passed 2
Skipped 0
Total 2

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.
@shmsr shmsr mentioned this pull request Jun 16, 2023
Merged
4 tasks
@shmsr

shmsr commented Jun 16, 2023

Copy link
Copy Markdown
Member

@aliabbas-elastic Please read this comment: #6592 (comment). @andrewkroh suggested a very nice tool.
@elasticmachine

elasticmachine commented Jun 19, 2023

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @aliabbas-elastic
andrewkroh
andrewkroh reviewed Jun 22, 2023
View reviewed changes
Comment thread packages/statsd_input/_dev/test/system/test-default-config.yml
@cla-checker-service

cla-checker-service Bot commented Jun 22, 2023 •
edited
Loading

Copy link
Copy Markdown

💚 CLA has been signed
@ali786XI
ali786XI force-pushed the statsd_input_system_test branch from 2b3c300 to 100af5a Compare June 22, 2023 10:53
@ali786XI
ali786XI marked this pull request as ready for review June 26, 2023 16:25
@ali786XI
ali786XI requested a review from a team as a code owner June 26, 2023 16:25
@ali786XI
ali786XI requested a review from ishleenk17 June 26, 2023 16:26
@ali786XI
ali786XI requested a review from andrewkroh June 27, 2023 05:53
@shmsr

shmsr commented Jun 27, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

Just adding the link here so that anyone who looks at the PR in future can understand why stream was added.

Here: #6592 (comment)
ishleenk17
ishleenk17 approved these changes Jun 29, 2023
View reviewed changes

@ishleenk17 ishleenk17 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good!
@ali786XI
ali786XI merged commit 7a5eab3 into elastic:main Jun 30, 2023
@elasticmachine

elasticmachine commented Jun 30, 2023

Copy link
Copy Markdown

Package statsd_input - 0.2.2 containing this change is available at https://epr.elastic.co/search?package=statsd_input
orestisfl pushed a commit to orestisfl/integrations that referenced this pull request May 15, 2026
@ali786XI
[O11y][StatsD Input] Add system test cases (elastic#6594)
9890e37 
* added system tests for the package

* update changelog entry

* add network in docker compose

* update hostname

* update system tests

* update system test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:statsd_input StatsD Input

7 participants

@ali786XI @elasticmachine @shmsr @andrewkroh @ishleenk17 @harnish-crest-data @milan-elastic