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

Add APM annotations for setupIngestManager #67069

Closed
wants to merge 10 commits into from
Closed

Conversation

jfsiii
Copy link
Contributor

@jfsiii jfsiii commented May 20, 2020

This PR mixes a few I'll likely create multiple PRs eventually, but I'm not sure of the lines yet.

  • Hacking APM tracing into some Ingest Manager workflows
    • setup Ingest Manager
    • package install
    • agent enroll, ack, and checkin
  • Change some requests in the install package workflow to be concurrent where possible
  • TS type correction for Registry Search results

You can submit the traces to an Elastic dev cluster (as described in #43548) with

ELASTIC_APM_ACTIVE=true yarn start --no-base-path --xpack.ingestManager.enabled=true

If you're running APM server locally, you can edit config/apm.dev.js to something like

module.exports = {
  active: true,
  serverUrl: 'http://localhost:8200',
};

#66507
refs

@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label May 20, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/ingest-management (Team:Ingest Management)

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jfsiii jfsiii removed the Team:Fleet Team label for Observability Data Collection Fleet team label Sep 18, 2020
@jfsiii jfsiii closed this Sep 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants