Skip to content

magefile: use beats/go.mod for TestPackages#5471

Merged
axw merged 1 commit intoelastic:masterfrom
axw:fix-testpackages
Jun 17, 2021
Merged

magefile: use beats/go.mod for TestPackages#5471
axw merged 1 commit intoelastic:masterfrom
axw:fix-testpackages

Conversation

@axw
Copy link
Copy Markdown
Member

@axw axw commented Jun 17, 2021

Motivation/summary

TestPackages uses dependencies that are only defined in beats go.mod/go.sum, but as we're running it from apm-server it tries
to use apm-server's go.mod/go.sum. Make sure we use the one in beats.

TestPackages uses dependencies that are only defined in beats
go.mod/go.sum, but as we're running it from apm-server it tries
to use apm-server's go.mod/go.sum. Make sure we use the one in
beats.
@axw axw mentioned this pull request Jun 17, 2021
2 tasks
@ghost
Copy link
Copy Markdown

ghost commented Jun 17, 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 #5471 opened

  • Start Time: 2021-06-17T03:10:54.774+0000

  • Duration: 47 min 40 sec

  • Commit: 2dd3cd4

Test stats 🧪

Test Results
Failed 0
Passed 6094
Skipped 120
Total 6214

Trends 🧪

Image of Build Times

Image of Tests

@axw axw merged commit 74e8422 into elastic:master Jun 17, 2021
@axw axw deleted the fix-testpackages branch June 17, 2021 04:05
@axw axw added the v7.14.0 label Jun 17, 2021
mergify bot pushed a commit that referenced this pull request Jun 17, 2021
TestPackages uses dependencies that are only defined in beats
go.mod/go.sum, but as we're running it from apm-server it tries
to use apm-server's go.mod/go.sum. Make sure we use the one in
beats.

(cherry picked from commit 74e8422)
axw added a commit that referenced this pull request Jun 17, 2021
TestPackages uses dependencies that are only defined in beats
go.mod/go.sum, but as we're running it from apm-server it tries
to use apm-server's go.mod/go.sum. Make sure we use the one in
beats.

(cherry picked from commit 74e8422)

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