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
Update a host's display name #4931
Comments
@chiiph @zwinnerman-fleetdm Are there any performance concerns to adding the <computer_name> field to the searchable criteria on the hosts, query target, etc pages? Is there a way we could test this out in the load test environment? |
We would need to add it to this index:
But in general this is not a source of load at this time. This can be tested in loadtesting if a branch with the change is pushed to github by following the readme in the loadtesting folder. |
Thank you @zwinnerman-fleetdm. Note to Reed and @noahtalerman : We need to revisit where all changing hostname to computer_name could cause a breaking change in the api. As examples, web hooks, logging, etc. A good place to start looking for these issues would be searching https://fleetdm.com/docs/using-fleet/rest-api#hosts for hostname. If we are doing a breaking change, we should consider moving this to a 5.x.y release. We should also consider doing a larger change from using device (or some other descriptor) instead of host in the UI. |
@tgauda this issue was discussed during product office hours on 06-16-2022. Customer codename "customer-schur" requested this improvement. For now, this issue will not be prioritized (released in the next 6 weeks). This is because the interface team has been asked to prioritize the "Expansion of host vitals" #397 Please feel free to bring this issue back to any future product office hours. This way, the issue can be reconsidered. |
@lukeheath I assigned you this issue for UI review and specification. Here's a Loom video in which I demo the feature: https://www.loom.com/share/4ad17ce84e6a43d5845b14937280606f |
@noahtalerman @xpkoala Currently the Regarding this item:
I'm not seeing how adding a Is it necessary for us to have both |
@noahtalerman, can you give any updates about that topic? We would like to start the roll-out with around 2000 clients, but having weird display hostnames like in your examples is a problem. |
@davidschoefberger sorry you're running into this issue. This feature is in development and will be included in Fleet version 4.22.0 targeted for release on 10/20/22. |
@noahtalerman thanks for that quick reply! Really looking forward to the release. |
Goal
As a Fleet UI user, I want to see a default display name for hosts that makes the Fleet UI and Fleet product easy to understand and use.
Figma
https://www.figma.com/file/hdALBDsrti77QuDNSzLdkx/?node-id=5930%3A255765
Notes
aianas-mbp.cable.rcn.com
when what I really want iscomputer_name
from thesystem_info
table.Child Issues
display_name
to theGET /hosts
andGET /hosts/{id}
API endpoint responses #7135display_name
property provided byGET /hosts
andGET /hosts/{id}
endpoints #7136The text was updated successfully, but these errors were encountered: