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

Revert workaround for missing fleet.hosts #9762

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

axw
Copy link
Member

@axw axw commented Dec 7, 2022

Motivation/summary

This is no longer needed, with elastic/elastic-agent#1802, and it triggers a panic in ESS after upgrading from 7.x deployments.

Checklist

- [ ] Update CHANGELOG.asciidoc
- [ ] Update package changelog.yml (only if changes to apmpackage have been made)
- [ ] Documentation has been updated

How to test these changes

  1. Create a 7.17.x ESS deployment
  2. Upgrade to 8.6
  3. Make sure APM Server comes up and is accessible

Related issues

None

@axw axw added the backport-8.6 Automated backport with mergify label Dec 7, 2022
@axw axw requested a review from a team December 7, 2022 13:54
@axw axw marked this pull request as ready for review December 7, 2022 13:54
@apmmachine
Copy link
Contributor

💚 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-12-07T13:45:37.821+0000

  • Duration: 28 min 1 sec

Test stats 🧪

Test Results
Failed 0
Passed 153
Skipped 0
Total 153

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /package : Generate and publish the docker images.

  • /test windows : Build & tests on Windows.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@axw axw merged commit 71e66a1 into elastic:main Dec 7, 2022
@axw axw deleted the revert-fleet-hosts-workaround branch December 7, 2022 14:22
mergify bot pushed a commit that referenced this pull request Dec 7, 2022
This is no longer needed, with
elastic/elastic-agent#1802

(cherry picked from commit 71e66a1)
axw added a commit that referenced this pull request Dec 7, 2022
This is no longer needed, with
elastic/elastic-agent#1802

(cherry picked from commit 71e66a1)

Co-authored-by: Andrew Wilkins <[email protected]>
@lahsivjar lahsivjar self-assigned this Dec 8, 2022
@lahsivjar
Copy link
Contributor

Confirmed that APM-Server is healthy on upgrade from 7.17.8 to 8.6.0-BC6 and is accepting requests.

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.

4 participants