-
Notifications
You must be signed in to change notification settings - Fork 35
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
Integrate collector with new metadata api #858
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
604c35e
begin integrating collector with new metadata api
portertech 6d41b38
metadata api, use collector fields for tag details
portertech 48dd1a4
metadata api, real host details
portertech a2e614a
metadata api, get host ip
portertech 9721c1f
metadata api, proxy info
portertech 50b7943
metadata api, send metadata on start, updated tests
portertech 23cff5a
Merge branch 'main' into feature/metadata-api
portertech 36bbd10
metadata api, changelog entry
portertech 3e0ba59
metadata api, attempt to fix lint errors
portertech c0cdb3c
metadata api, update gopsutil
portertech 579472b
metadata api, go mod tidy
portertech ed15a8a
metadata api, updated go mod for exporter
portertech 0178f36
Merge branch 'main' into feature/metadata-api
portertech 88597f3
metadata api, go mod tidy and lint fixes
portertech b8313c6
metadata api, exporter go mod tidy
portertech 5111f9c
Merge branch 'main' into feature/metadata-api
portertech 3c5b28d
Merge branch 'main' into feature/metadata-api
portertech 32c9738
metadata api, use retry with backoff
portertech 3e2c33d
metadata api, added new host details environment
portertech e4f907f
metadata api, cleaned up changelog
portertech 070b226
metadata api, fixed auth
portertech d8c09f0
metadata api, fixed redirect tests
portertech a2fe469
metadata api, added environment configuration field and removed netwo…
portertech f969d6c
metadata api, corrected metadata api URL
portertech 45869b6
metadata api, always send tag details, fixed host address
portertech aebdaac
Merge branch 'main' into feature/metadata-api
portertech 53767ea
Merge branch 'main' into feature/metadata-api
portertech e4cef4a
metadata api, payload casing and agent -> collector rename
portertech b299ded
metadata api, removed omitempty all fields are required
portertech c19315a
metadata api, info -> debug request logging
portertech 49d47d6
metadata api, feature gate update metadata
portertech ec4c28a
Merge branch 'main' into feature/metadata-api
portertech d8f213e
metadata api, err check feature gate apply
portertech 366bace
Update pkg/extension/sumologicextension/extension.go
portertech 01f589a
metadata api, use real build version for running version
portertech f2d6458
metadata api, make it clear we don't connect to determine address
portertech 354f86f
metadata api, doesn't respond with a 204
portertech File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
// Copyright The OpenTelemetry Authors | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
|
||
package api | ||
|
||
type OpenMetadataHostDetails struct { | ||
Name string `json:"name"` | ||
OsName string `json:"osName"` | ||
OsVersion string `json:"osVersion"` | ||
Environment string `json:"environment"` | ||
} | ||
|
||
type OpenMetadataCollectorDetails struct { | ||
RunningVersion string `json:"runningVersion"` | ||
} | ||
|
||
type OpenMetadataNetworkDetails struct { | ||
HostIpAddress string `json:"hostIpAddress"` | ||
} | ||
|
||
type OpenMetadataRequestPayload struct { | ||
HostDetails OpenMetadataHostDetails `json:"hostDetails"` | ||
CollectorDetails OpenMetadataCollectorDetails `json:"collectorDetails"` | ||
NetworkDetails OpenMetadataNetworkDetails `json:"networkDetails"` | ||
TagDetails map[string]interface{} `json:"tagDetails"` | ||
} | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Are we going to change api in the futher? Do we need any kind of versioning?
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 don't believe so, there may be additional fields. I used registration for reference.