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

[Fleet] Agent details page UI #64983

Merged
merged 19 commits into from
May 4, 2020
Merged

Conversation

jen-huang
Copy link
Contributor

Resolves #57968

Summary

This PR updates the existing agent details page to match designs. As part of this work, I need to uniquely identify each agent event in the activity log table by their saved object IDs. A new type NewAgentEvent as added to differentiate new events coming from agent from existing events already stored as saved objects (AgentEvent).

In the designs, a Payload column was shown as part of the activity log table but designs weren't created for how to view the payload details. Instead of adding that column, I made each row expandable to view full message and payload (agent doesn't send payload for any events currently).

A few header styling/spacing inconsistencies between app sections were also cleaned up.

Screenshots

image

image

@jen-huang jen-huang added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Feature:Fleet Fleet team's agent central management project v7.8.0 Team:Fleet Team label for Observability Data Collection Fleet team labels May 1, 2020
@jen-huang jen-huang requested a review from a team May 1, 2020 00:18
@jen-huang jen-huang self-assigned this May 1, 2020
@elasticmachine
Copy link
Contributor

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

Copy link
Member

@nchaulet nchaulet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@jen-huang
Copy link
Contributor Author

@elasticmachine merge upstream

@jen-huang
Copy link
Contributor Author

@elasticmachine merge upstream

@jen-huang
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@jen-huang jen-huang merged commit f9be590 into elastic:master May 4, 2020
@jen-huang jen-huang deleted the ingest/agent-details branch May 4, 2020 23:06
jen-huang added a commit to jen-huang/kibana that referenced this pull request May 4, 2020
* Fix empty host name column in agent list

* Fix empty version column in agent list

* Consolidate page header styling inconsistencies

* Add tabs to agent details

* Add right-side header content and actions menu

* Give headers more spacing when there are tabs present

* Add details tab

* Use ECS formatted metadata

* Make activity log table pretty

* Return agent event SO id from list API

* Fix i18n

* Add types for new agent events and differentiate from stored agent events

* Adjust test

Co-authored-by: Elastic Machine <[email protected]>
jen-huang added a commit that referenced this pull request May 5, 2020
* Fix empty host name column in agent list

* Fix empty version column in agent list

* Consolidate page header styling inconsistencies

* Add tabs to agent details

* Add right-side header content and actions menu

* Give headers more spacing when there are tabs present

* Add details tab

* Use ECS formatted metadata

* Make activity log table pretty

* Return agent event SO id from list API

* Fix i18n

* Add types for new agent events and differentiate from stored agent events

* Adjust test

Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Fleet Fleet team's agent central management project release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.8.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Ingest] Fleet agent details UI
4 participants