Skip to content

metricbeat - fix elasticsearch and kibana integration tests failures in 8.0#30566

Merged
kvch merged 5 commits intoelastic:mainfrom
klacabane:28659-fixes-800-testenv
Feb 28, 2022
Merged

metricbeat - fix elasticsearch and kibana integration tests failures in 8.0#30566
kvch merged 5 commits intoelastic:mainfrom
klacabane:28659-fixes-800-testenv

Conversation

@klacabane
Copy link
Copy Markdown
Contributor

@klacabane klacabane commented Feb 23, 2022

Summary

Rel #28659

The environments used by metricbeat integration tests are bumped to 8.0 and both elasticsearch and kibana packages are currently failing:

  • elasticsearch: new fields were added and needs to be documented
  • kibana: the integration tests for the kibana.settings metricset are failing since the API it hits does not exist as of 8.0. I removed the failing tests to unblock us for now and will follow up with a clean up to remove the settings metricset completely from the codebase in [Metricbeat] remove kibana.settings metricset #30592

Testing

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Feb 23, 2022
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Feb 23, 2022

This pull request does not have a backport label. Could you fix it @klacabane? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit

NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Feb 23, 2022
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 23, 2022

💚 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: 2022-02-24T11:26:17.442+0000

  • Duration: 15 min 5 sec

❕ Flaky test report

No test was executed to be analysed.

🤖 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!)

@klacabane klacabane added backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify Feature:Stack Monitoring Team:Integrations Label for the Integrations team labels Feb 24, 2022
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Feb 24, 2022
@mergify mergify bot removed the backport-skip Skip notification from the automated backport with mergify label Feb 24, 2022
@klacabane klacabane marked this pull request as ready for review February 24, 2022 11:25
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/integrations (Team:Integrations)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/stack-monitoring (Stack monitoring)

@klacabane
Copy link
Copy Markdown
Contributor Author

/test

@klacabane klacabane changed the title 28659 fixes 800 testenv metricbeat - fix elasticsearch and kibana integration tests failures in 8.0 Feb 24, 2022
@ruflin ruflin requested a review from sayden February 28, 2022 08:01
Copy link
Copy Markdown
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Change LGTM. Lets make sure we follow up.

One thing I'm curious is why we didn't see this failing earlier on main and now only saw it with the 8.0 bump? What version are we testing against in main.

Lets get this in to unblock 8.0 and then take it from there.

@kvch kvch merged commit a84f4d5 into elastic:main Feb 28, 2022
mergify bot pushed a commit that referenced this pull request Feb 28, 2022
…in 8.0 (#30566)

* add missing elasticsearch.index fields

* add missing elasticsearch.node.stats fields

* remove deprecated elasticsearch.settings integration tests

* mage update

* remove useless field

(cherry picked from commit a84f4d5)
mergify bot pushed a commit that referenced this pull request Feb 28, 2022
…in 8.0 (#30566)

* add missing elasticsearch.index fields

* add missing elasticsearch.node.stats fields

* remove deprecated elasticsearch.settings integration tests

* mage update

* remove useless field

(cherry picked from commit a84f4d5)
kvch pushed a commit that referenced this pull request Feb 28, 2022
…in 8.0 (#30566) (#30593)

* add missing elasticsearch.index fields

* add missing elasticsearch.node.stats fields

* remove deprecated elasticsearch.settings integration tests

* mage update

* remove useless field

(cherry picked from commit a84f4d5)

Co-authored-by: Kevin Lacabane <kevin.lacabane@elastic.co>
kvch pushed a commit that referenced this pull request Feb 28, 2022
…in 8.0 (#30566) (#30594)

* add missing elasticsearch.index fields

* add missing elasticsearch.node.stats fields

* remove deprecated elasticsearch.settings integration tests

* mage update

* remove useless field

(cherry picked from commit a84f4d5)

Co-authored-by: Kevin Lacabane <kevin.lacabane@elastic.co>
v1v added a commit that referenced this pull request Mar 2, 2022
…-29710

* '8.1' of github.com:elastic/beats: (51 commits)
  refactor pushDockerImages (#30414) (#30624)
  ci: add windows-2022 in the extended meta-stage (#30528) (#30630)
  Curate k8s testing versions to only keep the actively maintained (#30619) (#30625)
  [8.1](backport #30355) Add Beats upgrade docs for 8.0 (#30612)
  Remove references to gcp from the Functionbeat docs (#30579) (#30609)
  x-pack/auditbeat/module/system/socket: defend against exec with zero arguments (#30586) (#30597)
  [MySQL Enterprise] Adding default paths values to manifest.yml (#30598) (#30604)
  metricbeat - fix elasticsearch and kibana integration tests failures in 8.0 (#30566) (#30594)
  Install gawk as a replacement for mawk in Docker containers. (#30452) (#30465)
  [Filebeat] Remove RecordedFuture dataset from Threat Intel module (#30564) (#30568)
  Adjust the documentation of `backoff` options in filestream input (#30552) (#30557)
  packetbeat/beater: help the GC clean up the Npcap installer if it's not used (#30513) (#30546)
  Osquerybeat: Add install verification for osquerybeat (#30388) (#30404)
  Update docker/distribution to 2.8.0 (#30462) (#30540)
  Add `parsers` examples to `filestream` reference configuration (#30529) (#30537)
  [8.1](backport #30068) ZooKeeper module: Adapt to ZooKeeper 3.6+ `mntr` response fields' changes. (#30360)
  [8.1](backport #30512) Switch skip to use `CI` (#30525)
  Forward-port 8.0.1 changelog to 8.1 (#30517)
  packetbeat/beater: don't attempt to install npcap when already installed (#30509) (#30511)
  Add drop and explicit tests to avoid duplicate ingest of elasticsearch logs (#30440) (#30488)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v8.0.0 Automated backport with mergify backport-v8.1.0 Automated backport with mergify Feature:Stack Monitoring Team:Integrations Label for the Integrations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants