Skip to content

systemtest: fix apm-server binary injection#5440

Merged
axw merged 1 commit intoelastic:masterfrom
axw:systemtest-elastic-agent-image
Jun 14, 2021
Merged

systemtest: fix apm-server binary injection#5440
axw merged 1 commit intoelastic:masterfrom
axw:systemtest-elastic-agent-image

Conversation

@axw
Copy link
Copy Markdown
Member

@axw axw commented Jun 13, 2021

Motivation/summary

Due to some changes in elastic-agent (elastic/beats#24817), injection of the apm-server binary became ineffective and we have been running system tests with the published artifacts.

Artifacts (such as the apm-server) are now unpacked into state/data/install/. The state/data/install directory is expected to be owned by the elastic-agent user, so we can no longer bind mount the apm-server binary. Instead, we now create a custom Docker image and copy in the apm-server and apm-server.yml files.

Due to some changes in elastic-agent
(elastic/beats#24817), injection
of the apm-server binary became ineffective and we have
been running system tests with the published artifacts.

Artifacts (such as the apm-server) are now unpacked into
state/data/install/<artifact>. The state/data/install
directory is expected to be owned by the elastic-agent
user, so we can no longer bind mount the apm-server binary.
Instead, we now create a custom Docker image and copy in
the apm-server and apm-server.yml files.
@ghost
Copy link
Copy Markdown

ghost commented Jun 13, 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 #5440 opened

  • Start Time: 2021-06-13T03:23:16.072+0000

  • Duration: 39 min 7 sec

  • Commit: 1f09cc6

Test stats 🧪

Test Results
Failed 0
Passed 6078
Skipped 120
Total 6198

Trends 🧪

Image of Build Times

Image of Tests

@axw axw marked this pull request as ready for review June 13, 2021 06:22
@axw axw requested a review from a team June 13, 2021 06:22
@axw axw added the v7.14.0 label Jun 14, 2021
@axw axw merged commit 301caed into elastic:master Jun 14, 2021
@axw axw deleted the systemtest-elastic-agent-image branch June 14, 2021 07:26
mergify bot pushed a commit that referenced this pull request Jun 14, 2021
Due to some changes in elastic-agent
(elastic/beats#24817), injection
of the apm-server binary became ineffective and we have
been running system tests with the published artifacts.

Artifacts (such as the apm-server) are now unpacked into
state/data/install/<artifact>. The state/data/install
directory is expected to be owned by the elastic-agent
user, so we can no longer bind mount the apm-server binary.
Instead, we now create a custom Docker image and copy in
the apm-server and apm-server.yml files.

(cherry picked from commit 301caed)
stuartnelson3 pushed a commit to stuartnelson3/apm-server that referenced this pull request Jun 14, 2021
Due to some changes in elastic-agent
(elastic/beats#24817), injection
of the apm-server binary became ineffective and we have
been running system tests with the published artifacts.

Artifacts (such as the apm-server) are now unpacked into
state/data/install/<artifact>. The state/data/install
directory is expected to be owned by the elastic-agent
user, so we can no longer bind mount the apm-server binary.
Instead, we now create a custom Docker image and copy in
the apm-server and apm-server.yml files.
axw added a commit that referenced this pull request Jun 15, 2021
Due to some changes in elastic-agent
(elastic/beats#24817), injection
of the apm-server binary became ineffective and we have
been running system tests with the published artifacts.

Artifacts (such as the apm-server) are now unpacked into
state/data/install/<artifact>. The state/data/install
directory is expected to be owned by the elastic-agent
user, so we can no longer bind mount the apm-server binary.
Instead, we now create a custom Docker image and copy in
the apm-server and apm-server.yml files.

(cherry picked from commit 301caed)

Co-authored-by: Andrew Wilkins <axw@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants