Skip to content

[7.x] Use the apm-server version everywhere* (#4725)#4821

Merged
axw merged 1 commit intoelastic:7.xfrom
axw:backport/7.x/pr-4725
Feb 19, 2021
Merged

[7.x] Use the apm-server version everywhere* (#4725)#4821
axw merged 1 commit intoelastic:7.xfrom
axw:backport/7.x/pr-4725

Conversation

@axw
Copy link
Copy Markdown
Member

@axw axw commented Feb 19, 2021

Backports the following commits to 7.x:

* publish: parse version to set version_major

Instead of hardcoding the major version, parse the full version.

* beater: pass apm-server version into root handler

Rather than returning the libbeat version from the
root/healthcheck handler, inject the apm-server version
so it can be returned.

Also, in the integration tests use a hard-coded version
"1.2.3", so we don't have to change approval files each
time we bump the version. This should enable us to
automate the version bump.
# Conflicts:
#	beater/api/mux.go
#	beater/api/root/test_approved/integration/TestRootHandler_AuthorizationMiddleware/Authorized.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationErrors.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationEvents.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationMetricsets.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationMinimalEvents.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationProfileCPUProfile.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationProfileCPUProfileMetadata.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationProfileHeapProfile.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationSpans.approved.json
#	beater/test_approved_es_documents/TestPublishIntegrationTransactions.approved.json
#	publish/pub.go
@axw axw added the backport label Feb 19, 2021
@axw axw enabled auto-merge (squash) February 19, 2021 03:23
@ghost
Copy link
Copy Markdown

ghost 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 #4821 opened

  • Start Time: 2021-02-19T03:24:09.992+0000

  • Duration: 31 min 15 sec

  • Commit: 70f4110

Test stats 🧪

Test Results
Failed 0
Passed 4373
Skipped 126
Total 4499

Trends 🧪

Image of Build Times

Image of Tests

Steps errors 4

Expand to view the steps failures

Windows build
  • Took 3 min 39 sec . View more details on here
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 4 min 30 sec . View more details on here
  • Description: ./.ci/scripts/sync.sh

@axw axw merged commit 72e2bc1 into elastic:7.x Feb 19, 2021
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.

1 participant