Skip to content

[O11y][Airflow] Add system tests for the integration package - #8564

Merged
ali786XI merged 6 commits into
elastic:mainfrom
ali786XI:airflow_add_system_tests
Nov 28, 2023
Merged

ali786XI merged 6 commits into
elastic:mainfrom
ali786XI:airflow_add_system_tests

Conversation

@ali786XI

@ali786XI ali786XI commented Nov 23, 2023 •

Copy link
Copy Markdown
Contributor

Proposed commit message

  • Added system tests for Airflow integration using the observability/stream image
  • Updated the Kibana version to 8.11.0 as per comment

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.
  • I have verified that Kibana version constraints are current according to guidelines.

How to test this PR locally

  1. elastic-package check
  2. elastic-package stack up -d -v
  3. elastic-package test -v

Related issues

Screenshots

image

@ali786XI ali786XI added enhancement New feature or request Integration:airflow Airflow labels Nov 23, 2023
@ali786XI ali786XI self-assigned this Nov 23, 2023
@ali786XI ali786XI changed the title add system tests [O11y][Airflow] Add system tests for the integration Nov 23, 2023
@elasticmachine

elasticmachine commented Nov 23, 2023 •

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-11-28T13:07:21.743+0000

  • Duration: 16 min 27 sec

Test stats 🧪

Test Results
Failed 0
Passed 4
Skipped 0
Total 4

🤖 GitHub comments

Expand to view the GitHub comments

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

  • /test : Re-trigger the build.

@elasticmachine

elasticmachine commented Nov 23, 2023 •

Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (0/0) 💚
Files 100.0% (0/0) 💚
Classes 100.0% (0/0) 💚
Methods 75.0% (3/4) 👎 -18.75
Lines 100.0% (0/0) 💚
Conditionals 100.0% (0/0) 💚

@ali786XI
ali786XI marked this pull request as ready for review November 23, 2023 07:20
@ali786XI
ali786XI requested a review from a team as a code owner November 23, 2023 07:20
@ali786XI
ali786XI force-pushed the airflow_add_system_tests branch from 870ccf7 to d9c899f Compare November 24, 2023 06:13
@ritalwar
ritalwar self-requested a review November 24, 2023 06:35
@harnish-crest-data harnish-crest-data changed the title [O11y][Airflow] Add system tests for the integration [O11y][Airflow] Add system tests for the integration package Nov 24, 2023
@ritalwar

ritalwar commented Nov 28, 2023 •

Copy link
Copy Markdown
Contributor

I suggest upgrading the Kibana version to 8.11 to align with the modifications in the statsd module implementation in Beats, which are available in version 8.11. Subsequently, we should execute the tests based on the correct implementation.
@lalit-satapathy @SubhrataK

@ali786XI

Copy link
Copy Markdown
Contributor Author

I suggest upgrading the Kibana version to 8.11 to align with the modifications in the statsd module implementation in elastic/beats#36925, which are available in version 8.11. Subsequently, we should execute the tests based on the correct implementation.

Sure Let me update it accordingly

Comment thread packages/airflow/changelog.yml Outdated