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

[7.x] systemtest: test apm-server in Fleet mode (#4650) | systemtest: fix missing error check (#4683) | systemtest: compensate for Fleet API bug (#4693) #4831

Merged
merged 4 commits into from
Feb 20, 2021

Conversation

axw
Copy link
Member

@axw axw commented Feb 19, 2021

Backports the following commits to 7.x:

axw added 3 commits February 19, 2021 20:42
* systemtest: test apm-server in Fleet mode

We run a custom package registry into which
the "apm" integration package is bind-mounted
from the tree, enabling us to test with
unreleased package changes.

Add a system test that runs Elastic Agent in
Fleet mode, with a locally built APM Server
binary injected into the container. The test
adds the "apm" integration package to a new
agent policy, waits for elastic-agent to start
apm-server, and verifies it is functional.
# Conflicts:
#	systemtest/elasticsearch.go
* systemtest: compensate for Fleet API bug

elastic/kibana#90544 is
causing the system test to respond with 404 Not Found,
despite the policy existing and being successfully
deleted.

Update the test to temporarily expect a 404 response to
avoid failing all our builds. Once the Kibana bug is
resolved this will start failing, and we can again check
that no error is returned.
@axw axw added the backport label Feb 19, 2021
@axw axw enabled auto-merge (squash) February 19, 2021 12:42
@codecov-io
Copy link

codecov-io commented Feb 19, 2021

Codecov Report

Merging #4831 (dfda58e) into 7.x (49449e6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              7.x    #4831   +/-   ##
=======================================
  Coverage   76.77%   76.77%           
=======================================
  Files         166      166           
  Lines       10232    10232           
=======================================
  Hits         7856     7856           
  Misses       2376     2376           

@apmmachine
Copy link
Contributor

apmmachine commented Feb 19, 2021

💚 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

Expand to view the summary

Build stats

  • Build Cause: Pull request #4831 updated

  • Start Time: 2021-02-20T00:39:20.882+0000

  • Duration: 40 min 31 sec

  • Commit: dfda58e

Test stats 🧪

Test Results
Failed 0
Passed 4493
Skipped 126
Total 4619

Trends 🧪

Image of Build Times

Image of Tests

Steps errors 3

Expand to view the steps failures

Compress
  • Took 0 min 0 sec . View more details on here
  • Description: tar --exclude=coverage-files.tgz -czf coverage-files.tgz coverage
Compress
  • Took 0 min 0 sec . View more details on here
  • Description: tar --exclude=system-tests-linux-files.tgz -czf system-tests-linux-files.tgz system-tests
Test Sync
  • Took 3 min 28 sec . View more details on here
  • Description: ./.ci/scripts/sync.sh

@axw axw merged commit 4175425 into elastic:7.x Feb 20, 2021
@axw axw deleted the backport/7.x/pr-4650_pr-4683_pr-4693 branch February 20, 2021 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants