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

Report the status message field to Fleet Server checkins #1151

Closed
joshdover opened this issue Sep 12, 2022 · 6 comments · Fixed by elastic/kibana#142220
Closed

Report the status message field to Fleet Server checkins #1151

joshdover opened this issue Sep 12, 2022 · 6 comments · Fixed by elastic/kibana#142220
Assignees

Comments

@joshdover
Copy link
Contributor

We should report an agent's reason for being in the status it's in to Fleet Server rather than only relying on Agent logs. This will make it easier to debug why an agent is in a particular status.

Related changes need to be made to Fleet Server to save this state in the .fleet-agents index.

@joshdover
Copy link
Contributor Author

joshdover commented Sep 12, 2022

At minimum we should get these changes into Agent and Fleet to begin persisting this data into ES.

A great addition would be to also expose this in the UI:

  • On the agent details page in the table of metadata we have today
  • As a tooltip on the agent status badge in the table view

@joshdover
Copy link
Contributor Author

@jen-huang It'd also be great to have this reported in Fleet's agent status telemetry in the UI. Maybe counts by unique message with some cap.

@narph narph removed their assignment Sep 28, 2022
@juliaElastic
Copy link
Contributor

juliaElastic commented Sep 28, 2022

@joshdover can I use the existing last_checkin_message agent field for this, that was introduced for this feature?

I found that the Agent v2 branches started using the CheckinRequest.Message field and Agent last_checkin_message field, do we expect the v2 reporting to override this logic or should we store in separate fields?

@joshdover
Copy link
Contributor Author

Makes total sense, yes let's use that since v2 should be shipping in this release as well.

@ghost
Copy link

ghost commented Nov 21, 2022

Hi @juliaElastic,

We have created below 04 test cases for this feature.

Please let us know if we are missing anything.
Thanks

@ghost
Copy link

ghost commented Dec 6, 2022

Hi Team,

We have executed 04 testcases for this feature under our Fleet Test run at Fleet Team 8.6 New features test plan and found that it's working fine.

Build Details:

Version: 8.6.0 BC5
BUILD: 58693
COMMIT: ed40c16ce9999cc47ad55c11bb097d2e443b31a6
Artifact link: https://staging.elastic.co/8.6.0-8cf9e954/summary-8.6.0.html

Below are the test cases.

Please let us know if we are missing anything.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants