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

[receiver/receivercreator] Add support for k8s service discovery #29022

Merged
merged 13 commits into from
Nov 16, 2023

Conversation

a-thaler
Copy link
Contributor

@a-thaler a-thaler commented Nov 8, 2023

Description:

  • Added a new watch to the k8s_observer extension for k8s services, which can be enabled using a new flag "observe_services".
  • Discovered entities are transformed into a new endpoint type k8s.service.
  • Adjusted the receivercreator to support the new type k8s.service

Link to tracking Issue: #29021

Testing: Added unit tests analogue to the available tests

Documentation: Adjusted readme's of k8s_observer and receivercreator. Added description of new flags and typers.

Note:
Current implementation is working as described in the linked ticket. Please check the potential discussion points mentioned in the ticket: #29021 (comment)

* main:
  [receiver/dockerstats] rename struct and function to keep expected receiver.Factory and pass checkapi (open-telemetry#27086)
  [all][chore] Moved from interface{} to any for all go code (open-telemetry#29072)
  [chore] bump go versions in workflows to 1.20.11 and 1.21.4 (open-telemetry#29080)
  hostmetricsreceiver: remove unused function (open-telemetry#29075)
  [exporter/influxdb] Remove //nolint indent-error-flow (open-telemetry#29073)
  [chore] dependabot updates Wed Nov  8 21:01:03 UTC 2023 (open-telemetry#29071)
  [chore] dependabot updates Wed Nov  8 18:29:02 UTC 2023 (open-telemetry#29052)
  [chore] dependabot updates Wed Nov  8 16:58:54 UTC 2023 (open-telemetry#29028)
  [receiver/collectd] move collectdreceiver to beta (open-telemetry#28997)
  [chore] update affiliation (open-telemetry#29019)
  Codesmon/exporter/azuremonitor/persistent queue (open-telemetry#26258)

# Conflicts:
#	extension/observer/k8sobserver/handler.go
@a-thaler a-thaler changed the title [receiver/receivercreator] Add support k8s service discovery [receiver/receivercreator] Add support for k8s service discovery Nov 9, 2023
@a-thaler
Copy link
Contributor Author

a-thaler commented Nov 9, 2023

Fixed all problems reported by the github actions, requires another run now.

Copy link
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

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

just one doc nit

@atoulme atoulme added the ready to merge Code review completed; ready to merge by maintainers label Nov 16, 2023
@djaglowski djaglowski merged commit 4f8b075 into open-telemetry:main Nov 16, 2023
87 of 88 checks passed
@github-actions github-actions bot added this to the next release milestone Nov 16, 2023
@a-thaler a-thaler deleted the httpcheck branch November 17, 2023 07:45
graphaelli pushed a commit to graphaelli/opentelemetry-collector-contrib that referenced this pull request Nov 17, 2023
…n-telemetry#29022)

**Description:**
- Added a new watch to the k8s_observer extension for k8s services,
which can be enabled using a new flag "observe_services".
- Discovered entities are transformed into a new endpoint type
`k8s.service`.
- Adjusted the receivercreator to support the new type `k8s.service`


**Link to tracking Issue:**
[open-telemetry#29021](open-telemetry#29021)

**Testing:** Added unit tests analogue to the available tests

**Documentation:** Adjusted readme's of k8s_observer and
receivercreator. Added description of new flags and typers.

**Note:**
Current implementation is working as described in the linked ticket.
Please check the potential discussion points mentioned in the ticket:
open-telemetry#29021 (comment)

---------

Co-authored-by: Antoine Toulme <antoine@toulme.name>
RoryCrispin pushed a commit to ClickHouse/opentelemetry-collector-contrib that referenced this pull request Nov 24, 2023
…n-telemetry#29022)

**Description:**
- Added a new watch to the k8s_observer extension for k8s services,
which can be enabled using a new flag "observe_services".
- Discovered entities are transformed into a new endpoint type
`k8s.service`.
- Adjusted the receivercreator to support the new type `k8s.service`


**Link to tracking Issue:**
[open-telemetry#29021](open-telemetry#29021)

**Testing:** Added unit tests analogue to the available tests

**Documentation:** Adjusted readme's of k8s_observer and
receivercreator. Added description of new flags and typers.

**Note:**
Current implementation is working as described in the linked ticket.
Please check the potential discussion points mentioned in the ticket:
open-telemetry#29021 (comment)

---------

Co-authored-by: Antoine Toulme <antoine@toulme.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension/observer ready to merge Code review completed; ready to merge by maintainers receiver/receivercreator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants