Skip to content

Commit

Permalink
Use os.type instead of os.family (#161)
Browse files Browse the repository at this point in the history
The semantic conventions define os.type, but the examples use os.family. Let's follow the convention!

See also OpAMP spec: https://github.com/open-telemetry/opamp-spec/blob/main/specification.md#agentdescriptionnon_identifying_attributes
  • Loading branch information
jlegoff authored Mar 20, 2023
1 parent 34ba098 commit 06011e4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion internal/examples/agent/agent/agent.go
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ func (agent *Agent) createAgentIdentity() {
},
NonIdentifyingAttributes: []*protobufs.KeyValue{
{
Key: "os.family",
Key: "os.type",
Value: &protobufs.AnyValue{
Value: &protobufs.AnyValue_StringValue{
StringValue: runtime.GOOS,
Expand Down
2 changes: 1 addition & 1 deletion internal/examples/supervisor/supervisor/supervisor.go
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ func (s *Supervisor) createAgentDescription() *protobufs.AgentDescription {
keyVal("service.version", s.agentVersion),
},
NonIdentifyingAttributes: []*protobufs.KeyValue{
keyVal("os.family", runtime.GOOS),
keyVal("os.type", runtime.GOOS),
keyVal("host.name", hostname),
},
}
Expand Down

0 comments on commit 06011e4

Please sign in to comment.