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

Do not log ERROR for watchers in not upgradable scenarios #28334

Closed
Tracked by #26930
michalpristas opened this issue Oct 11, 2021 · 5 comments
Closed
Tracked by #26930

Do not log ERROR for watchers in not upgradable scenarios #28334

michalpristas opened this issue Oct 11, 2021 · 5 comments
Assignees
Labels
bug debugging good first issue Indicates a good issue for first-time contributors Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team v8.1.0

Comments

@michalpristas
Copy link
Contributor

At the moment when we run agent in a container and it's not on usual path but e.g /app/..
we see error saying failed to invoke rollback watcher with a detail explaining executable could not be found.

When running in a container from different path as usual we should

  • decrease log level to WARN
  • investigate options to skip watcher execution in this scenario
@michalpristas michalpristas added bug Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team labels Oct 11, 2021
@michalpristas michalpristas self-assigned this Oct 11, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@jlind23
Copy link
Collaborator

jlind23 commented Oct 11, 2021

@michalpristas as you have self assigned this issue does it mean you are currently working on it?

@jlind23 jlind23 added the good first issue Indicates a good issue for first-time contributors label Dec 7, 2021
@jlind23
Copy link
Collaborator

jlind23 commented Dec 7, 2021

@AndersonQ this one could be a pretty good to start ramping up on the agent.

@AndersonQ
Copy link
Member

This cannot be reproduced neither on the latest official docker image, docker.elastic.co/beats/elastic-agent:7.16.2, nor on master. Thus I'm closing it

@jlind23
Copy link
Collaborator

jlind23 commented Jan 4, 2022

Thanks @AndersonQ for the work provided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug debugging good first issue Indicates a good issue for first-time contributors Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team v8.1.0
Projects
None yet
Development

No branches or pull requests

5 participants