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

Remove all Elastic APM code from Telescope #2498

Closed
humphd opened this issue Nov 19, 2021 · 4 comments · Fixed by #2720
Closed

Remove all Elastic APM code from Telescope #2498

humphd opened this issue Nov 19, 2021 · 4 comments · Fixed by #2720
Assignees
Labels
area: back-end area: elasticsearch Elasticsearch Related issues and pull requests area: microservices area: satellite Issues related to the Satellite microservice project dependencies Pull requests that update a dependency file type: bug Something isn't working
Milestone

Comments

@humphd
Copy link
Contributor

humphd commented Nov 19, 2021

We had an idea to use Elastic APM for monitoring our logs and services. In the end we haven't gone that route, and use a simpler Portainer setup.

Let's get rid of all the APM code from our docker/compose files, envs, logging, Satellite, docs, etc. code.

@humphd humphd added type: bug Something isn't working area: back-end area: elasticsearch Elasticsearch Related issues and pull requests dependencies Pull requests that update a dependency file area: microservices area: satellite Issues related to the Satellite microservice project labels Nov 19, 2021
@ycechungAI
Copy link

ill add this one as well

@humphd
Copy link
Contributor Author

humphd commented Nov 22, 2021

Great stuff, thanks.

@humphd
Copy link
Contributor Author

humphd commented Dec 2, 2021

npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package: 'elastic-apm-node@3.13.0',
npm WARN EBADENGINE   required: { node: '^8.6.0 || 10 || 12 || 13 || 14' },
npm WARN EBADENGINE   current: { node: 'v16.13.0', npm: '8.1.0' }
npm WARN EBADENGINE }

@humphd
Copy link
Contributor Author

humphd commented Jan 27, 2022

We need to use Fixes #.... when we do the PR description @aserputov

@humphd humphd closed this as completed Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: back-end area: elasticsearch Elasticsearch Related issues and pull requests area: microservices area: satellite Issues related to the Satellite microservice project dependencies Pull requests that update a dependency file type: bug Something isn't working
Projects
None yet
4 participants