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

[8.1](backport #30388) Osquerybeat: Add install verification for osquerybeat #30404

Merged
merged 4 commits into from
Feb 22, 2022

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Feb 15, 2022

This is an automatic backport of pull request #30388 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

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

botelastic bot commented Feb 15, 2022

This pull request doesn't have a Team:<team> label.

@elasticmachine
Copy link
Collaborator

elasticmachine commented Feb 15, 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-22T18:31:33.535+0000

  • Duration: 102 min 25 sec

Test stats 🧪

Test Results
Failed 0
Passed 8556
Skipped 12
Total 8568

💚 Flaky test report

Tests succeeded.

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

@aleksmaus
Copy link
Member

/test

@aleksmaus
Copy link
Member

/test

@mergify
Copy link
Contributor Author

mergify bot commented Feb 21, 2022

This pull request has not been merged yet. Could you please review and merge it @aleksmaus? 🙏

@ph
Copy link
Contributor

ph commented Feb 21, 2022

@aleksmaus AKAIK the failures doesn't seems related to a change here.

@elastic/observablt-robots

Looking at the output of the failure PR I see logs mentioning an OSS version of the Elastic Agent, I am not sure if this is making the job fail but it shouldn't be there.

12:49:25  + MSG='parameter missing.'
12:49:25  + SOURCE_IMAGE=docker.elastic.co/beats/elastic-agent-oss:8.1.0-SNAPSHOT
12:49:25  + TARGET_IMAGE=docker.elastic.co/observability-ci/elastic-agent-oss:pr-30404-amd64
12:49:25  + docker image inspect docker.elastic.co/beats/elastic-agent-oss:8.1.0-SNAPSHOT
12:49:25  + echo 'docker image docker.elastic.co/beats/elastic-agent-oss:8.1.0-SNAPSHOT does not exist'
12:49:25  docker image docker.elastic.co/beats/elastic-agent-oss:8.1.0-SNAPSHOT does not exist

Is this possible that the 8.1 branch is missing a commit for ci jobs?

@v1v
Copy link
Member

v1v commented Feb 21, 2022

@ph , it should not fail for that reason, publishing docker images from local to the internal docker-registry is harmless implementation, since it happens in the CI rather in the build system itself and it's accessing a third party system, it tries up to 3 times, but if it fails it will not fail the build at all.

As far as I see the issue was when running the e2e-testing job, somehow the 8.1 branch is gone :/ https://beats-ci.elastic.co/job/e2e-tests/job/e2e-testing-mbp

@ph
Copy link
Contributor

ph commented Feb 21, 2022

test this please

@aleksmaus
Copy link
Member

/test

@aleksmaus aleksmaus merged commit 414d18f into 8.1 Feb 22, 2022
@mergify mergify bot deleted the mergify/bp/8.1/pr-30388 branch February 22, 2022 21:05
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 needs_team Indicates that the issue/PR needs a Team:* label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants