-
Notifications
You must be signed in to change notification settings - Fork 418
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
[RFC] Host and Hostname fields - Stage 0 #1512
base: main
Are you sure you want to change the base?
Conversation
💚 CLA has been signed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for submitting this stage 0 RFC, @hadadata59!
I left some initial feedback.
@@ -0,0 +1,7 @@ | |||
- name: agent | |||
fields: | |||
- name: hostname |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The agent.*
fields are meant to describe the software entity collecting events on a host or observer. As a software entity, the agent.hostname
field has been left out intentionally since the hostname is instead an attribute of a host.*
or an observer.*
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have seen evidence of records (observer.) which report on a host (host.) and regarding the agent (agent.*) where the hostnames of each (observer, host, and agent) are unique.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you have an example you'd be willing to share for this discussion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ebeahan
Here is an anonymized representation of the event output from the agent (represented with 123) reporting on the HOST and the other agent deployed on the host (321).
application.reporting_agent.assetid | GUID123_HOSTGUID |
---|---|
application.reporting_agent.endpoint.log.level | INFO |
application.reporting_agent.endpoint.product.version | 2.7 |
host.broker_guid | BROKERGUID |
host.domain | HOSTNAME.DOMAIN.COM |
host.resident_agent.name | HOSTNAME123 |
host.hostname | HOSTNAME |
host.id | HOSTGUID |
host.ip | HOSTIP |
reporting_agent.server | HOSTNAME.DOMAIN.COM |
reporting_agentguid | GUID123 |
event.resident_agent.version | 10.4 |
host.resident_agent_server.guid | GUID321 |
host.resident_agent_server.name | "HOSTNAMESVR321" |
reporting_agent.server.name | "HOSTNAMESVR123.DOMAIN.COM" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @hadadata59! Just catching up here and want to verify I understand correctly.
From your example, you have agent 123 and agent 321 and both run on the same host that they are monitoring?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kgeller yes. two agents, one host.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome.
So in this scenario, we would say we are receiving logs from both agent 123 and agent 321 about host 1? If so, could we not just populate the host.name field in both of those sets of logs from the agent?
I don't quite follow how, in this scenario, we'd need additional hostname fields.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are thoughts on leveraging the existing agent.name
to capture the hostname?
For example, Beats does this as the default, unless overridden in the configuration: elastic/beats#18000
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a single log record (not sets of logs) where the box (host.hostname), the non-reporting 'resident' agent (?) and the reporting agent (agent.hostname) all provide unique hostnames in the record.
@@ -0,0 +1,7 @@ | |||
- name: destination | |||
fields: | |||
- name: hostname |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the source
/destination
/client
/server
field sets, the address value should populate the .address
field and be duplicated to the appropriate field based on the value:.ip
for IP addresses, .domain
for FQDNs or hostnames.
I believe the .domain
field serves the same function you're proposing here. Or do you have different motivations for proposing this addition?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using the example of somehost.example.com as a fully qualified domain name:
where some somehost is the hostname
and example.com is the domain name
See https://docs.microsoft.com/en-us/windows/win32/dns/naming-conventions
There is general confusion when the FQDN can be both .domain as well as host.name.
The point here is to isolate the hostname (I.E. somehost), as well as the domain (.domain) for a more accurate and reliable representation of the data and for ease of user search.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this addition can make sense here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very early drafts of ECS did include source.hostname
and destination.hostname
fields, but the project later removed the fields. The discussion was that having both source.hostname
and source.domain
caused confusion, and arguably using hostname
vs. host
in a network-centric context was incorrect.
Here are some of the past conversations, if anyone's curious: #175 #84
Sometimes revisiting past decisions is valuable, though, of course! However, there would be a good bit of work to reassign the [source|destination|client|server].domain
field's intent; this would be a significant breaking change for ECS.
@@ -0,0 +1,25 @@ | |||
- name: host | |||
fields: | |||
- name: model |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Capturing these types of inventory attributes has come up in past ECS discussions. One pitfall to avoid would be limiting them to certain field sets that wouldn't allow them to describe a broader range of assets someone might have in their inventory or CMDB.
Examples could be power supplies, generators, or server racks. These items would still have model
, manufacturer
, or serial_number
attributes to capture but wouldn't necessarily still be considered hosts in the ECS sense of a host.
In past brainstorming, the idea of creating an inventory.*
or asset.*
field set has been suggested, but I think that idea would best be discussed as its own RFC.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure I understand the distinction you're trying to make here. We have other lower level objects that are disimmilar (.name). Not sure why using a .model for host would preclude using a .model with a different description for another object would be problematic? The inventory. or asset. concept is still talking about an entity (a host), so it would be nice for context but it would also make my search problematic (E.G. if I needed to see a SW inventory on a host). In that case the fact that the scan came from tenable or my EDR host module would be the indicator that it was a point in time inventory vice an event with an associated host.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think what @ebeahan was trying to say was that in ECS, we describe host broadly as a 'general computing instance' meaning it can be anything from hardware to virtual machine to docker container, etc. The intention with a inventory.*
or asset.*
would be specifically a physical item we want to keep track of.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Understood. 1.12 has container fields, so I think that we can not worry about that issue. As far as a VM, host.model would be hard to populate, as it is not a current vmware field I am aware of. But, serial_number and vendor are capturable and exist.
I think it's fine to move toward a new object level. I think knowing the host is a Dell ModelX Serial#Y in context of the OS the host is running etc. is valuable and you might lose some of that context in the record, but otherwise;
If the inventory or asset tag is the way forward, let's choose a path.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I personally prefer asset
, but I think the RFC process could certainly guide us towards a name. Is this something you are interested in leading?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I revisited and am rethinking my suggestion, I'm starting to think adding these fields under host.*
as proposed may be the better option.
Like @hadadata59 mentioned, we already store asset details about a host, like architecture, OS details, geolocation data, underneath host.*
fields already. We also explicitly list hardware
as a host type without specifying that a piece of hardware must be compute hardware.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned in #1512 (comment), I like the symmetry of using product
over model
to match the existing observer.product
field naming.
@melissaburpo this RFC may be relevant for mappings osquery host data. Is there someone on your team that could provide feedback as to whether the fields could be leveraged by osquery? |
Hi @jamiehynds - I'll take a look, but @aleksmaus or @james-elastic from our team may have some input as well. Thanks for the ping! |
I think these three proposed fields in particular map well to values that are retrievable via osquery from a query to the system_info table:
Regarding |
|
description: > | ||
The model associated with the host. | ||
|
||
- name: manufacturer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thoughts about using vendor
over manufacturer
?
vendor
provides symmetry with observer.vendor
. Also, if someone had a specific use case to capture a host's ODM along with the vendor, I could see possible confusion over which one to place in the manufacturer
field.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have been using vendor for software and manufacturer for hardware. However, I do not see a reason to object.
@melissaburpo @jamiehynds If possible, I'd propose we keep symmetry with the existing
|
I've added comments in-line to the open conversation threads, but I wanted to summarize the discussion in a single place. Additional
|
Perhaps we need to also update the docs, with an elaboration, see mention of |
Hi @hadadata59, did you see the summary of the next steps in #1512 (comment) to move this proposal forward? |
This PR is stale because it has been open for 60 days with no activity. |
make test
?make
and committed those changes?