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

Mackjmr/merge main to prod staging #4951

Merged
merged 68 commits into from
Nov 13, 2023
Merged

Commits on Oct 31, 2023

  1. [receiver/collectd] Move to use HTTPServerSettings with collectdrecei…

    …ver (open-telemetry#28812)
    
    **Description:**
    Overhauls collectdreceiver to use the latest config helper features
    
    **Link to tracking Issue:**
    Fixes open-telemetry#28811
    
    **Documentation:**
    No impact to docs. User interface remains the same.
    Separate changelog to notice API breaking changes, as the Config struct
    is changing.
    
    ---------
    
    Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
    atoulme and dmitryax authored Oct 31, 2023
    Configuration menu
    Copy the full SHA
    812d799 View commit details
    Browse the repository at this point in the history
  2. [chore][exporter/datadog] Re-enable TestTraceExporter (open-telemetry…

    …#28827)
    
    Re-enable TestTraceExporter.
    
    Fixes
    open-telemetry#27630
    
    Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
    songy23 and mx-psi authored Oct 31, 2023
    Configuration menu
    Copy the full SHA
    39ace36 View commit details
    Browse the repository at this point in the history
  3. [chore][receiver/hostmetrics] Skip process user error (un)muted test …

    …on non-Linux (open-telemetry#28829)
    
    **Description:**
    Fix open-telemetry#28828 - this is just disabling the test on non-Linux. The broken
    test was introduced via open-telemetry#28661.
    pjanotti authored Oct 31, 2023
    Configuration menu
    Copy the full SHA
    faada96 View commit details
    Browse the repository at this point in the history
  4. [receiver/hostmetricsreceiver] Add support for cpu frequency metric (o…

    …pen-telemetry#27445)
    
    **Description:** : Added support for host's cpu frequency as part
                       of the hostmetricsreceiver.
    
    **Link to tracking Issue:** open-telemetry#26532
    
    **Testing:**
    
    1. Using the following configuration:
    ```yml
    receivers:
      hostmetrics:
        collection_interval: 5s
        scrapers:
          cpu:
            metrics:
              system.cpu.frequency:
                enabled: true
    
    processors:
      resourcedetection/system:
        detectors: ["system"]
        system:
          hostname_sources: ["lookup", "cname", "dns", "os"]
          resource_attributes:
            host.name:
              enabled: true
            host.id:
              enabled: true
            host.cpu.cache.l2.size:
              enabled: true
            host.cpu.family:
              enabled: true
            host.cpu.model.id:
              enabled: true
            host.cpu.model.name:
              enabled: true
            host.cpu.stepping:
              enabled: true
            host.cpu.vendor.id:
              enabled: true
    
    service:
      pipelines:
        metrics:
          receivers: [hostmetrics]
          exporters: [file]
          processors: [resourcedetection/system]
    
    exporters:
      file:
        path: ./output.json
    ```
    
    2. Start the collector with ./bin/otelcontribcol_linux_amd64 --config
    examples/host_config.yaml
    3. The output reports the added metric successfully:
    
    ```json
    {
       "resourceMetrics":[
          {
             "scopeMetrics":[
                {
                   "scope":{
                      "name":"otelcol/hostmetricsreceiver/cpu",
                      "version":"0.85.0-dev"
                   },
                   "metrics":[
                      {
                         "name":"system.cpu.frequency",
                         "description":"Current frequency of the CPU core in MHz.",
                         "unit":"MHz",
                         "gauge":{
                            "dataPoints":[
                               {
                                  "attributes":[
                                     {
                                        "key":"cpu",
                                        "value":{
                                           "stringValue":"cpu0"
                                        }
                                     }
                                  ],
                                  "startTimeUnixNano":"1696487580000000000",
                                  "timeUnixNano":"1696512423758783158",
                                  "asDouble":3000
                               },
                               {
                                  "attributes":[
                                     {
                                        "key":"cpu",
                                        "value":{
                                           "stringValue":"cpu1"
                                        }
                                     }
                                  ],
                                  "startTimeUnixNano":"1696487580000000000",
                                  "timeUnixNano":"1696512423758783158",
                                  "asDouble":3000
                               },
    ...
    ```
    
    Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
    ChrsMark authored Oct 31, 2023
    Configuration menu
    Copy the full SHA
    1c63ac9 View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2023

  1. [encoding/zipkinencodingextension] add default case (open-telemetry#2…

    …8689)
    
    **Description:** Fix bug when err is nil if an invalid version value is
    supplied.
    
    **Link to tracking Issue:** open-telemetry#28686
    
    ---------
    
    Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
    VihasMakwana and dmitryax authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    7efdf16 View commit details
    Browse the repository at this point in the history
  2. [chore] Upgrade cloud.google.com/go (open-telemetry#28840)

    To resolve failing build-and-test/checks CI job
    
    **Link to tracking Issue:**
    open-telemetry#28839
    dmitryax authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    7e06d30 View commit details
    Browse the repository at this point in the history
  3. [connector/exceptions] Add trace id and span id to generated logs (op…

    …en-telemetry#28670)
    
    **Description:** <Describe what has changed.>
    The current implementation generates logs from recorded exceptions in
    spans, but is not possible to see which traces and spans generated those
    logs. This PR adds that information to the logs
    
    **Link to tracking Issue:** Fixes open-telemetry#24407
    marctc authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    fe06dee View commit details
    Browse the repository at this point in the history
  4. [chore][exporter/loadbalancing] use headless service with DNS mode in…

    … K8S(open-telemetry#27014) (open-telemetry#28687)
    
    **Description:** <Describe what has changed.>
    fix
    open-telemetry#27014
    notice when in K8S, the DNS mode should config a headless service
    
    **Link to tracking Issue:** <Issue number if applicable>
    open-telemetry#27014
    Liubey authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    7647d4c View commit details
    Browse the repository at this point in the history
  5. Update README.md (open-telemetry#28844)

    The Prometheus Remote write exporter is missing the details of default
    values for the remote write queue config. Added the values after looking
    into the code for the same.
    datsabk authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    64a787d View commit details
    Browse the repository at this point in the history
  6. exporter/datadog: disable APM stats via feature flag (open-telemetry#…

    …28616)
    
    This change adds the "exporter.datadogexporter.disable_apm_stats"
    feature flag, which can be enabled to disable APM stats computation.
    
    Updates open-telemetry#28615
    gbbr authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    5c69f33 View commit details
    Browse the repository at this point in the history
  7. [receiver/zipkin] follow receiver contract (open-telemetry#28627)

    I came across `zipkinreceiver` and observed we don't
    follow the receiver
    [contract](https://github.com/open-telemetry/opentelemetry-collector/blob/b2961b799e2c1ec128f0539764af1fa10c839e04/receiver/doc.go#L21).
    We return `InternalServerError` straight away without checking
    permanent/non-permanent errors.
    
    We should probably return BadRequest in case of permanent errors
    
    open-telemetry/opentelemetry-collector#4335
    
    **Testing:** Added test cases
    
    Co-authored-by: Andrzej Stencel <astencel@sumologic.com>
    VihasMakwana and andrzej-stencel authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    9edd6a9 View commit details
    Browse the repository at this point in the history
  8. [chore][exporter/sumologicexporter] use errors.Join instead of go.ube…

    …r.org/multierr (open-telemetry#28614)
    
    **Description:** use errors.Join instead of go.uber.org/multierr
    
    **Link to tracking Issue:** open-telemetry#25121 
    
    ---------
    
    Co-authored-by: Andrzej Stencel <astencel@sumologic.com>
    bagmeg and andrzej-stencel authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    f530bc4 View commit details
    Browse the repository at this point in the history
  9. [receiver/wavefront] wrap metrics receiver under carbon receiver inst…

    …ead of using export function (open-telemetry#27259)
    
    **Description:** 
    Wavefrontreceiver is very similar to carbonreceiver: it is TCP based in
    which each received text line represents a single metric data point. In
    order to avoid using exported function `carbonreceiver.New(...)`, we can
    wrap metrics receiver under carbon receiver.
    
    **Link to tracking Issue:** 
    
    open-telemetry#27248
    
    **Testing:** 
    make chlog-validate
    go test for wavefrontreceiver
    
    **Documentation:**
    
    ---------
    
    Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
    sakulali and mx-psi authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    1b8ccde View commit details
    Browse the repository at this point in the history
  10. [processor/k8sattributes] Fix node/ns labels/annotations extraction (o…

    …pen-telemetry#28838)
    
    Set attributes from namespace/node labels or annotations even if
    `k8s.namespace.name` and `k8s.node.name` are not extracted.
    
    Fixes
    open-telemetry#28837
    dmitryax authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    fddc10d View commit details
    Browse the repository at this point in the history
  11. [processor/remoteobserver] rename to remotetapprocessor (open-telemet…

    …ry#27874)
    
    **Description:**
    Rename remoteobserverprocessor to remotetapprocessor
    
    **Link to tracking Issue:**
    Fixes open-telemetry#27873
    atoulme authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    b88690a View commit details
    Browse the repository at this point in the history
  12. [Spanmetrics] - Add exemplars to Sum metrics (open-telemetry#28671)

    **Description:** 
    We don't have exemplars added to Sum metrics right now. This PR provides
    an enhancement to add exemplars to Sum metrics in Spanmetrics connector
    
    
    **Testing:** 
    Added unit tests and also tested it in our local environment.
    aishyandapalli authored Nov 1, 2023
    Configuration menu
    Copy the full SHA
    ad35b39 View commit details
    Browse the repository at this point in the history

Commits on Nov 2, 2023

  1. [chore] fix codeowners (open-telemetry#28855)

    Regenerate codeowners with `make gengithub`
    atoulme authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    59cfbe7 View commit details
    Browse the repository at this point in the history
  2. feat(alertmanager): Add exporter factory and config (open-telemetry#2…

    …7836)
    
    **Description:** Factory implementation of Alertmanager Exporter
    Initial PR - base configs and factory implementation
    
    **Link to tracking Issue:**
    [open-telemetry#23659](open-telemetry#23569)
    
    **Testing:** Unit tests for config and factory implementation
    
    **Documentation:** Readme and Sample Configs to use Alertmanager
    exporter
    
    ---------
    
    Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
    Co-authored-by: Juraci Paixão Kröhling <juraci@kroehling.de>
    mcube8 and jpkrohling authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    f509060 View commit details
    Browse the repository at this point in the history
  3. Adds duration sampler distinct from latency in supplying two bounds (o…

    …pen-telemetry#26115)
    
    **Description:** Adds a bounded duration sampling processor, distinct
    from the existing latency one in that it has both lower and upper bounds
    
    Apologies for this appearing as a pull request out of nothing, my intent
    had actually been to create a review area against my own fork and raise
    an issue asking if you'd accept the PR. I think the need here is pretty
    obvious from the context, though I think it's easy to imagine preferring
    this to be a change to the existing processor. I raised as a new one as
    I thought it might make existing behavior cleaner to retain.
    
    **Link to tracking Issue:** As above this is a bit of a premature PR
    since I intended to raise as an issue, and thus there isn't one, but I
    think it's easy enough to deal with here so leaving open for now and
    have learned GitHub's ways for the future (I rarely use github).
    
    **Testing:** New module so associated tests are added showing all
    relevant behavior, and passing.
    
    **Documentation:** Updated README and example config
    
    ---------
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    garry-cairns and dependabot[bot] authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    ae6d36b View commit details
    Browse the repository at this point in the history
  4. [googlemanagedprometheusexporter] Clarify support status of this expo…

    …rter (open-telemetry#28863)
    
    *   Link to related GCP docs
    *   Clarify mention of "traces"
    * Drop mention of PromQL support as a difference from `googlecloud`
    exporter
    punya authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    8913b95 View commit details
    Browse the repository at this point in the history
  5. [filelogreceiver]: Add ability to sort by mtime (open-telemetry#28850)

    **Description:** <Describe what has changed.>
    * Adds a new `mtime` sort type, which will sort files by their modified
    time
    * Add a feature gate for `mtime` sort type
    
    An optional follow-up performance improvement may be made here, to have
    the finder return fs.DirEntry directly to query the mtime without making
    an extra call to os.Stat for each file.
    
    **Link to tracking Issue:** open-telemetry#27812
    
    **Testing:**
    * Added unit tests for new functionality
    
    **Documentation:** 
    * Added new `mode` parameter to filelogreceiver docs
    BinaryFissionGames authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    59506f6 View commit details
    Browse the repository at this point in the history
  6. [pkg/traslator] move skywalking_to_traces into pkg/translator (open-t…

    …elemetry#28814)
    
    **Description:** A part of
    open-telemetry#28693
    <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
    Ex. Adding a feature - Explain what this achieves.-->
    
    move`skywalking_to_traces` in `skywalkingreceiver` into
    `pkg/translator/skywalking`
    
    **Link to tracking Issue:** <Issue number if applicable>
    
    **Testing:** <Describe what testing was performed and which tests were
    added.>
    
    **Documentation:** <Describe the documentation added.>
    
    ---------
    
    Signed-off-by: Jared Tan <jian.tan@daocloud.io>
    JaredTan95 authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    327de9f View commit details
    Browse the repository at this point in the history
  7. [chore][pkg/stanza] Adjust length of knownFiles based on number of ma…

    …tches (open-telemetry#28646)
    
    Follows
    open-telemetry#28493
    
    This adjusts the length of `knownFiles` to be roughly 4x the number of
    matches per poll cycle. In other words, we will remember files for up to
    4 poll cycles.
    
    Resolves
    open-telemetry#28567
    djaglowski authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    4f5952a View commit details
    Browse the repository at this point in the history
  8. [chore][exporter/datadog] Add a section about how to switch to Zorkian (

    open-telemetry#28836)
    
    **Description:** 
    Update README about disabling the feature gate of native metric client
    and falling back to Zorkian client.
    
    ---------
    
    Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
    songy23 and mx-psi authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    3e0a21c View commit details
    Browse the repository at this point in the history
  9. [cmd/telemetrygen] Use exporter per worker for better metrics through…

    …put (open-telemetry#27201)
    
    Adding a feature - Use exporter per worker for better metrics throughput
    
    Initially when adding more workers in the telemetrygen config when
    running "metrics" it did not increase the metrics throughput since all
    workers used the same exporter.
    
    By creating one exporter per worker we can now increase the number of
    metrics being send to the backend.
    
    Fixes open-telemetry#26709
    
    - Units tests pass
    - Ran local load tests with different configurations
    
    ## Before code change
    
    Generate metrics:
    
    ```
    telemetrygen metrics \
        --metric-type Sum \
        --duration "60s" \
        --rate "0" \
        --workers "10" \
        --otlp-http=false \
        --otlp-endpoint <HOSTNAME> \
        --otlp-attributes "service.name"=\"telemetrygen\"
    ```
    
    Output:
    ```
    metrics generated	{"worker": 8, "metrics": 139}
    metrics generated	{"worker": 0, "metrics": 139}
    metrics generated	{"worker": 9, "metrics": 141}
    metrics generated	{"worker": 4, "metrics": 140}
    metrics generated	{"worker": 2, "metrics": 140}
    metrics generated	{"worker": 3, "metrics": 140}
    metrics generated	{"worker": 7, "metrics": 140}
    metrics generated	{"worker": 5, "metrics": 140}
    metrics generated	{"worker": 1, "metrics": 140}
    metrics generated	{"worker": 6, "metrics": 140}
    ```
    
    ## After code change
    
    ```
    telemetrygen metrics \
        --metric-type Sum \
        --duration "60s" \
        --rate "0" \
        --workers "10" \
        --otlp-http=false \
        --otlp-endpoint <HOSTNAME> \
        --otlp-attributes "service.name"=\"telemetrygen\"
    ```
    
    Output:
    
    ```
    metrics generated	{"worker": 6, "metrics": 1292}
    metrics generated	{"worker": 3, "metrics": 1277}
    metrics generated	{"worker": 5, "metrics": 1272}
    metrics generated	{"worker": 8, "metrics": 1251}
    metrics generated	{"worker": 9, "metrics": 1241}
    metrics generated	{"worker": 4, "metrics": 1227}
    metrics generated	{"worker": 0, "metrics": 1212}
    metrics generated	{"worker": 2, "metrics": 1201}
    metrics generated	{"worker": 1, "metrics": 1333}
    metrics generated	{"worker": 7, "metrics": 1363}
    ```
    
    By adding more workers you can now export more metrics and use
    `telemetrygen` better for load testing use cases.
    
    With the code change I can now utilize my CPU better for load tests.
    When adding 200 workers to the above config the CPU usage can go above
    80%. Before that CPU usage would be around 1% with 200 workers.
    
    
    ![image](https://github.com/open-telemetry/opentelemetry-collector-contrib/assets/558256/66727e5f-6b0a-44a3-8436-7e6985d6a01c)
    
    ---------
    
    Co-authored-by: Alex Boten <aboten@lightstep.com>
    marcelbirkner and Alex Boten authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    9edf138 View commit details
    Browse the repository at this point in the history
  10. [scraper/processscraper] Fix TestScrapeMetrics_MuteErrorFlags failure…

    …s on windows and darwin (open-telemetry#28864)
    
    **Description:** 
    
    There were some issues related to how `mock.On` works. With default mock
    and addition `On` which is already present it appends to a list and
    won't be called as one instance of a method is already there. So some
    expectations regarding return values were not met
    
    Metrics count for darwin is 3 because disk io is disabled
    [here](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/f509060a8d1ab5ca4b5827e0c60d1149e3059908/receiver/hostmetricsreceiver/internal/scraper/processscraper/process_scraper.go#L315)
    
    Tested locally on mac, windows 11 and ubuntu 22
    
    **Link to tracking Issue:** open-telemetry#28828
    michalpristas authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    0e13821 View commit details
    Browse the repository at this point in the history
  11. [chore][testbed] Do not use export function carbonreceiver.New (ope…

    …n-telemetry#28858)
    
    **Description:**
    Do not use export function `carbonreceiver.New` and replace with
    `factory.CreateMetricsReceiver`, then we can chore carbonreceiver to
    make it pass checkapi tool.
    
    **Link to tracking Issue:**
    
    open-telemetry#28857
    sakulali authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    61c12ca View commit details
    Browse the repository at this point in the history
  12. [chore] Run make gendependabot (open-telemetry#28868)

    To fix failing `build-and-test / checks` CI job
    dmitryax authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    197a47c View commit details
    Browse the repository at this point in the history
  13. [chore] update codeowners (open-telemetry#28869)

    Run `make gengithub` locally.
    atoulme authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    bfb780f View commit details
    Browse the repository at this point in the history
  14. [receiver/sshcheck] Change keyfile -> key_file in e.g. config and docs (

    open-telemetry#28834)
    
    `keyfile` was the key used in config and documented in sshcheck, but
    `key_file` is the preferred key for these purposes.
    
    **Link to tracking Issue:** open-telemetry#27035 
    
    **Testing:** Update tests to ensure this key is used in default.
    
    **Documentation:** Updated documentation to reflect the change in key.
    nslaughter authored Nov 2, 2023
    Configuration menu
    Copy the full SHA
    b39f83d View commit details
    Browse the repository at this point in the history

Commits on Nov 3, 2023

  1. Configuration menu
    Copy the full SHA
    040426f View commit details
    Browse the repository at this point in the history
  2. [exporter/azuremonitor] Add Connection String Support to Azure Monito…

    …r Exporter (open-telemetry#28854)
    
    **Description:** <Describe what has changed.>
    <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
    Ex. Adding a feature - Explain what this achieves.-->
    This pull request introduces the ability to configure the Azure Monitor
    Exporter using a connection string, aligning the exporter configuration
    with Azure Monitor's recommended practices. The current implementation
    requires users to set the instrumentation key directly, which will soon
    be deprecated in favor of using the connection string.
    
    **Changes Made:**
    
    1. Configuration Update: Modified the `Config` struct and related
    configuration parsing logic to support a `ConnectionString` field.
    2. Parsing Logic: Implemented functionality to parse the connection
    string and extract necessary details, such as `InstrumentationKey` and
    `IngestionEndpoint`.
    3. Updated Tests: Revised existing tests and added new ones to ensure
    coverage of the new configuration option.
    
    **Benefits:**
    
    * Streamlines the configuration process for end-users.
    * Aligns with Azure Monitor's best practices and recommended
    configuration approach.
    * Paves the way for the upcoming deprecation of direct instrumentation
    key configuration.
    
    **Backwards Compatibility:**
    This update maintains full backwards compatibility. Users currently
    utilizing the instrumentation key for configuration can continue to do
    so but are advised to transition to using the connection string.
    
    **To-Do** 
    
    * Documentation Update in a follow up PR
    * Deprecation Notice: A future update will introduce a deprecation
    warning for users still configuring the exporter with the
    instrumentation key, encouraging them to switch to using a connection
    string.
    * Add support for `EndpointSuffix` in connection string -
    https://learn.microsoft.com/en-us/azure/azure-monitor/app/sdk-connection-string?tabs=dotnet5#connection-string-with-an-endpoint-suffix
    
    **Link to tracking Issue:** <Issue number if applicable> 
    open-telemetry#28853
    
    **Testing:** <Describe what testing was performed and which tests were
    added.>
    
    Conducted comprehensive testing, including unit tests, to validate that
    the new configuration option works as expected and does not introduce
    regressions. All tests are currently passing.
    
    ```
    [Wed Nov  1 12:53:42 PDT 2023] --------- Transmitting 27 items ---------
    [Wed Nov  1 12:53:43 PDT 2023] Telemetry transmitted in 331.926261ms
    [Wed Nov  1 12:53:43 PDT 2023] Response: 200
    [Wed Nov  1 12:53:43 PDT 2023] Items accepted/received: 27/27
    [Wed Nov  1 12:53:53 PDT 2023] --------- Transmitting 30 items ---------
    [Wed Nov  1 12:53:53 PDT 2023] Telemetry transmitted in 73.171392ms
    [Wed Nov  1 12:53:53 PDT 2023] Response: 200
    [Wed Nov  1 12:53:53 PDT 2023] Items accepted/received: 30/30
    [Wed Nov  1 12:54:04 PDT 2023] --------- Transmitting 27 items ---------
    [Wed Nov  1 12:54:04 PDT 2023] Telemetry transmitted in 68.037724ms
    [Wed Nov  1 12:54:04 PDT 2023] Response: 200
    [Wed Nov  1 12:54:04 PDT 2023] Items accepted/received: 27/27
    ```
    
    **Documentation:** <Describe the documentation added.>
    
    TODO, in a follow up PR.
    rajkumar-rangaraj authored Nov 3, 2023
    Configuration menu
    Copy the full SHA
    4abb732 View commit details
    Browse the repository at this point in the history
  3. [exporter/awsxray] Add aws sdk http error events to x-ray subsegment …

    …and strip prefix `AWS.SDK.` from aws remote service name (open-telemetry#27232)
    
    **Description:** <Describe what has changed.>
    <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
    Ex. Adding a feature - Explain what this achieves.-->
    
    - Convert individual HTTP error events into exceptions within
    subsegments for AWS SDK spans.
    - Normalize the service name from `awsxray.AWSServiceAttribute`
    attribute by removing the `AWS.SDK.` prefix (in some aws sdk
    instrumentation, we have added the prefix to produce metrics with the
    prefix to clearly indicate the resource). This change ensures that X-Ray
    backend recognizes standard service names like "DynamoDb", "S3", etc.,
    enabling correct identification of AWS service types.
    
    
    **Link to tracking Issue:** 
    NA
    
    **Testing:** 
    Unit tests are added.
    
    **Documentation:**
    NA
    
    ---------
    
    Co-authored-by: John Knollmeyer <jknollm@amazon.com>
    Co-authored-by: John Knollmeyer <jaknollmeyer@gmail.com>
    3 people authored Nov 3, 2023
    Configuration menu
    Copy the full SHA
    9dd2258 View commit details
    Browse the repository at this point in the history
  4. [receiver/carbon] do not expose method (open-telemetry#28872)

    Do not export function `New` and pass checkapi.
    
    open-telemetry#26304
    
    Signed-off-by: sakulali <sakulali@126.com>
    sakulali authored Nov 3, 2023
    Configuration menu
    Copy the full SHA
    f7f856e View commit details
    Browse the repository at this point in the history
  5. [chore] update testbed to embed jaeger exporter (open-telemetry#28880)

    Rather than importing a deprecated module, this embeds the contents of
    that module in the testbed. Part of open-telemetry#28647
    
    Signed-off-by: Alex Boten <aboten@lightstep.com>
    Alex Boten authored Nov 3, 2023
    Configuration menu
    Copy the full SHA
    af09177 View commit details
    Browse the repository at this point in the history
  6. Make replication stats return whole number (open-telemetry#28824)

    **Description:** 
    
    I failed to reproduce []uint8 to int64 conversion but I was able to
    repro float64 to int64 conversion error.
    Different types may be due to different versions or values reported. 
    
    The fix is forcing query to retrieve integer values. While this may seem
    like most obvious fix I'm not really aligned with it.
    
    What query is returning for is a lag as a decimal number (whole part is
    seconds) by forcing this to return just an int we kind of losing
    precision. `0.4s` are reported as `0` while it is `400ms`.
    
    My proposal here would consists of 2 options.
    First one is change reporting in a way that what we report is in fact
    time-span in `ms`. This could most likely be considered breaking.
    
    Second option (I'm more in favor of) is to change the type of what is
    reported (from int to float). This way unit is intact and does not break
    possible visualizations, but we gain precision and won't lose data.
    
    My first issue here so I wanted to get some feedback first before
    publishing something unreasonable.
    
    _EDIT_
    
    Went with the option of deprecating metrics with second precision (still
    fixing conversion failures) and introducing alternative to these metrics
    with `_ms` suffix in name and millisecond precision.
    
    Old metrics are now behind a featuregate which is enabled by default for
    now.
    
    **Link to tracking Issue:** open-telemetry#26714 
    
    **Testing:** 
    Setting up replicated postgres instances and testing method against this
    deployment.
    
    **Documentation:** -
    
    ---------
    
    Co-authored-by: Daniel Jaglowski <jaglows3@gmail.com>
    michalpristas and djaglowski authored Nov 3, 2023
    Configuration menu
    Copy the full SHA
    3b84eae View commit details
    Browse the repository at this point in the history
  7. Retract googlecloud exporter releases that don't have logging (open-t…

    …elemetry#28884)
    
    **Description:**
    
    Logging was broken after
    open-telemetry#25900
    (released in v0.84.0). It is fixed by
    open-telemetry/opentelemetry-collector#8792,
    which will be released in v0.89.0. This will help with any distributions
    that include the googlecloud exporter components.
    dashpole authored Nov 3, 2023
    Configuration menu
    Copy the full SHA
    1bfbb18 View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2023

  1. [chore] move collectdreceiver shared code to an internal package (ope…

    …n-telemetry#28856)
    
    This allows the collectdreceiver to pass checkapi.
    atoulme authored Nov 6, 2023
    Configuration menu
    Copy the full SHA
    ad6fa27 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    6696f7e View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    1955509 View commit details
    Browse the repository at this point in the history

Commits on Nov 7, 2023

  1. Bump github.com/google/cadvisor from 0.47.3 to 0.48.1 in /receiver/aw…

    …scontainerinsightreceiver (open-telemetry#28998)
    
    Second attempt after dependabot's PR open-telemetry#28974. There was a typo fixed in
    cadvisor `v0.48.1` that was a breaking change for us. This updates all
    references to correct spelling of `housekeeping`.
    
    Fixes open-telemetry#28995
    crobert-1 authored Nov 7, 2023
    Configuration menu
    Copy the full SHA
    c53db30 View commit details
    Browse the repository at this point in the history
  2. [receiver/kafkametrics] Using unique container networks and container…

    … names and attempt to fix flaky tests (open-telemetry#28903)
    
    **Description:** <Describe what has changed.>
    Using unique container networks and container names and attempt to fix
    flaky tests
    
    **Link to tracking Issue:**
    
    open-telemetry#26293
    
    **Testing:**
    **Preparation:** 
        DIR = receiver/kafkametricsreceiver
    CMD = go test -v -count=1 -race -timeout 360s -parallel 4
    -tags=integration,"" -run=Integration ./...
    
    **Tests:**
    1. If we manually modify the code(as shown below) and use invalid kafka
    broker, such as `localhost:invalid-port`, the same error as shown in the
    issue may occur.
        ```
        // receiver/kafkametricsreceiver/integration_test.go
        scraperinttest.WithCustomConfig(
    func(t *testing.T, cfg component.Config, ci
    *scraperinttest.ContainerInfo) {
                rCfg := cfg.(*Config)
                rCfg.CollectionInterval = 5 * time.Second
                rCfg.Brokers = []string{"localhost:invalid-port"}
                rCfg.Scrapers = []string{"brokers", "consumers", "topics"}
            }),
        ```
    
    2. If we execute the test commands **sequentially** , it seems that the
    execution results are all correct.
        ```
        # all result are correct
        for i in {1..100}; do echo "Run $i"; ./${CMD} ; done
        ```
    
    3. If we execute the commands in **parallel** end with **`&`**,
    sometimes the error shown in the issue may occur.
        ```
        # sometimes result occur error
        for i in {1..20}; do echo "Run $i"; ./${CMD} &; done
        ```
    
    **Inference:**
    I have found that duplicate container networks and container names can
    cause container creation to fail or result in successfully created
    containers with unavailable ports, which may lead to issues similar to
    the one shown.
    
    **Additional information:** 
    Since Kafka's startup relies on ZooKeeper (which waits for the default
    `zookeeper.connection.timeout.ms=18000`), if Kafka starts first and
    ZooKeeper fails to start properly after the timeout duration, it will
    cause the Kafka container to fail to start correctly. I found the issue
    testcontainers/testcontainers-go#1791 wants to
    support that.
    
    **Documentation:**
    
    ---------
    
    Signed-off-by: sakulali <sakulali@126.com>
    sakulali authored Nov 7, 2023
    Configuration menu
    Copy the full SHA
    dbfa694 View commit details
    Browse the repository at this point in the history

Commits on Nov 8, 2023

  1. [chore][processort/tailsamplingprocessor] Limit concurrency for certa…

    …in tests (flay test on Windows runners) (open-telemetry#29014)
    
    **Description:**
    Limit number of goroutines started during
    `processor/tailsamplingprocessor` tests. This causes very frequently
    failures on the Windows tests, see
    [here](open-telemetry#28682 (comment))
    for example.
    
    The issue is that the race detector has a hard limit on number of
    goroutines, see golang/go#23611. The fix
    limits the concurrency in two tests so this limit is not hit on GH
    Windows runners.
    
    **Link to tracking Issue:** 
    Fix open-telemetry#9126
    
    **Testing:**
    Increased the concurrency on the two changed tests caused the error and
    validated that it passed twice on my fork.
    
    **Documentation:**
    N/A
    pjanotti authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    902b1a9 View commit details
    Browse the repository at this point in the history
  2. Codesmon/exporter/azuremonitor/persistent queue (open-telemetry#26258)

    Description:
    Added a new config item to support the QueueSettings values.
    Extended the exportHelper.New[Metrics|Logs|Traces]Exporter call to pass
    in the QueueSettings config, thus enabling persistent_queue for this
    exporter.
    
    Link to tracking Issue:
    Fixes issue
    open-telemetry#25859
    
    Testing:
    Extending unit tests to check configuration changes are picked up.
    
    Documentation:
    Added sending_queue config items to README.md's configuration section.
    codesmon authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    c26444b View commit details
    Browse the repository at this point in the history
  3. [chore] update affiliation (open-telemetry#29019)

    Updated to match core
    Alex Boten authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    cec3e59 View commit details
    Browse the repository at this point in the history
  4. [receiver/collectd] move collectdreceiver to beta (open-telemetry#28997)

    Promote collectdreceiver as beta component
    
    Fixes open-telemetry#28658
    atoulme authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    a80edda View commit details
    Browse the repository at this point in the history
  5. [chore] dependabot updates Wed Nov 8 16:58:54 UTC 2023 (open-telemetr…

    …y#29028)
    
    Bump github.com/DataDog/datadog-agent/pkg/proto from 0.49.0-rc.2 to
    0.50.0-devel in /exporter/datadogexporter
    Bump github.com/IBM/sarama from 1.41.3 to 1.42.0 in
    /exporter/kafkaexporter
    Bump github.com/IBM/sarama from 1.41.3 to 1.42.0 in
    /receiver/kafkareceiver
    Bump github.com/IBM/sarama from 1.41.3 to 1.42.1 in
    /receiver/kafkametricsreceiver
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /exporter/awscloudwatchlogsexporter
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /exporter/awsemfexporter
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /exporter/awsxrayexporter
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /exporter/datadogexporter
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /extension/observer/ecsobserver
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/aws/awsutil
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/aws/cwlogs
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/aws/k8s
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/aws/proxy
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/aws/xray
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/aws/xray/testdata/sampleapp
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /internal/metadataproviders
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /processor/resourcedetectionprocessor
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /receiver/awsecscontainermetricsreceiver
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.3 in
    /receiver/awsxrayreceiver
    Bump github.com/aws/aws-sdk-go from 1.46.7 to 1.47.4 in
    /receiver/awscontainerinsightreceiver
    Bump github.com/aws/aws-sdk-go-v2 from 1.21.2 to 1.22.1 in
    /exporter/awskinesisexporter
    Bump github.com/aws/aws-sdk-go-v2 from 1.21.2 to 1.22.1 in
    /extension/sigv4authextension
    Bump github.com/aws/aws-sdk-go-v2/config from 1.19.1 to 1.22.0 in
    /exporter/awskinesisexporter
    Bump github.com/aws/aws-sdk-go-v2/config from 1.19.1 to 1.22.0 in
    /extension/sigv4authextension
    Bump github.com/aws/aws-sdk-go-v2/credentials from 1.13.43 to 1.15.1 in
    /exporter/awskinesisexporter
    Bump github.com/aws/aws-sdk-go-v2/credentials from 1.13.43 to 1.15.1 in
    /extension/sigv4authextension
    Bump github.com/aws/aws-sdk-go-v2/service/kinesis from 1.20.0 to 1.22.0
    in /exporter/awskinesisexporter
    Bump github.com/aws/aws-sdk-go-v2/service/sts from 1.23.2 to 1.25.0 in
    /exporter/awskinesisexporter
    Bump github.com/aws/aws-sdk-go-v2/service/sts from 1.23.2 to 1.25.0 in
    /extension/sigv4authextension
    Bump github.com/golangci/golangci-lint from 1.55.1 to 1.55.2 in
    /internal/tools
    Bump github.com/gorilla/mux from 1.8.0 to 1.8.1 in
    /receiver/jaegerreceiver
    Bump github.com/gorilla/mux from 1.8.0 to 1.8.1 in
    /receiver/sapmreceiver
    Bump github.com/gorilla/mux from 1.8.0 to 1.8.1 in
    /receiver/signalfxreceiver
    Bump github.com/gorilla/mux from 1.8.0 to 1.8.1 in
    /receiver/skywalkingreceiver
    Bump github.com/gorilla/mux from 1.8.0 to 1.8.1 in
    /receiver/splunkhecreceiver
    Bump github.com/gorilla/mux from 1.8.0 to 1.8.1 in
    /testbed/mockdatareceivers/mockawsxrayreceiver
    Bump github.com/influxdata/influxdb-client-go/v2 from 2.12.3 to 2.12.4
    in /receiver/influxdbreceiver
    Bump github.com/mattn/go-sqlite3 from 1.14.17 to 1.14.18 in
    /extension/storage
    Bump github.com/prometheus/procfs from 0.11.1 to 0.12.0 in
    /receiver/hostmetricsreceiver
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in
    /exporter/signalfxexporter
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in
    /extension/observer/hostobserver
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in
    /processor/resourcedetectionprocessor
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in
    /receiver/awscontainerinsightreceiver
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in
    /receiver/hostmetricsreceiver
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in
    /receiver/jmxreceiver
    Bump github.com/shirou/gopsutil/v3 from 3.23.9 to 3.23.10 in /testbed
    Bump github.com/spf13/cobra from 1.7.0 to 1.8.0 in /cmd/telemetrygen
    Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common
    from 1.0.777 to 1.0.782 in /exporter/tencentcloudlogserviceexporter
    Bump go.mongodb.org/atlas from 0.34.0 to 0.35.0 in
    /receiver/mongodbatlasreceiver
    Bump golang.org/x/mod from 0.13.0 to 0.14.0 in /cmd/configschema
    Bump golang.org/x/sys from 0.13.0 to 0.14.0 in
    /exporter/signalfxexporter
    Bump golang.org/x/sys from 0.13.0 to 0.14.0 in /pkg/stanza
    Bump golang.org/x/sys from 0.13.0 to 0.14.0 in /pkg/winperfcounters
    Bump golang.org/x/sys from 0.13.0 to 0.14.0 in
    /receiver/hostmetricsreceiver
    Bump golang.org/x/sys from 0.13.0 to 0.14.0 in
    /receiver/windowseventlogreceiver
    Bump golang.org/x/text from 0.13.0 to 0.14.0 in /cmd/configschema
    Bump golang.org/x/text from 0.13.0 to 0.14.0 in /cmd/mdatagen
    Bump golang.org/x/text from 0.13.0 to 0.14.0 in /internal/coreinternal
    Bump golang.org/x/text from 0.13.0 to 0.14.0 in /pkg/stanza
    Bump golang.org/x/text from 0.13.0 to 0.14.0 in /testbed
    Bump golang.org/x/time from 0.3.0 to 0.4.0 in /cmd/telemetrygen
    Bump google.golang.org/api from 0.148.0 to 0.149.0 in
    /exporter/f5cloudexporter
    Bump google.golang.org/api from 0.148.0 to 0.149.0 in
    /exporter/googlecloudpubsubexporter
    Bump google.golang.org/api from 0.148.0 to 0.149.0 in
    /receiver/googlecloudpubsubreceiver
    Bump google.golang.org/api from 0.148.0 to 0.149.0 in
    /receiver/googlecloudspannerreceiver
    opentelemetrybot authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    22d86de View commit details
    Browse the repository at this point in the history
  6. [chore] dependabot updates Wed Nov 8 18:29:02 UTC 2023 (open-telemetr…

    …y#29052)
    
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /exporter/awscloudwatchlogsexporter
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /exporter/awsemfexporter
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /exporter/awsxrayexporter
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /exporter/datadogexporter
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /extension/observer/ecsobserver
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/aws/awsutil
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/aws/cwlogs
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/aws/k8s
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/aws/proxy
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/aws/xray
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/aws/xray/testdata/sampleapp
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /internal/metadataproviders
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /processor/resourcedetectionprocessor
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /receiver/awscontainerinsightreceiver
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /receiver/awsecscontainermetricsreceiver
    Bump github.com/aws/aws-sdk-go from 1.47.4 to 1.47.5 in
    /receiver/awsxrayreceiver
    Bump github.com/aws/aws-sdk-go-v2/config from 1.22.0 to 1.22.2 in
    /exporter/awskinesisexporter
    Bump github.com/aws/aws-sdk-go-v2/config from 1.22.0 to 1.22.2 in
    /extension/sigv4authextension
    Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common
    from 1.0.782 to 1.0.786 in /exporter/tencentcloudlogserviceexporter
    Bump google.golang.org/api from 0.149.0 to 0.150.0 in
    /exporter/f5cloudexporter
    Bump google.golang.org/api from 0.149.0 to 0.150.0 in
    /exporter/googlecloudpubsubexporter
    Bump google.golang.org/api from 0.149.0 to 0.150.0 in
    /receiver/googlecloudpubsubreceiver
    Bump google.golang.org/api from 0.149.0 to 0.150.0 in
    /receiver/googlecloudspannerreceiver
    opentelemetrybot authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    eb320a1 View commit details
    Browse the repository at this point in the history
  7. [chore] dependabot updates Wed Nov 8 21:01:03 UTC 2023 (open-telemetr…

    …y#29071)
    
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /exporter/awscloudwatchlogsexporter
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /exporter/awsemfexporter
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /exporter/awsxrayexporter
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /exporter/datadogexporter
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /extension/observer/ecsobserver
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/aws/awsutil
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/aws/cwlogs
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/aws/k8s
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/aws/proxy
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/aws/xray
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/aws/xray/testdata/sampleapp
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /internal/metadataproviders
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /processor/resourcedetectionprocessor
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /receiver/awscontainerinsightreceiver
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /receiver/awsecscontainermetricsreceiver
    Bump github.com/aws/aws-sdk-go from 1.47.5 to 1.47.6 in
    /receiver/awsxrayreceiver
    opentelemetrybot authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    c95d613 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    b595986 View commit details
    Browse the repository at this point in the history
  9. hostmetricsreceiver: remove unused function (open-telemetry#29075)

    **Description:**
    `gopsutil` recently added the capability to pass environment vars
    through context. This is now done everywhere. This environment variable
    setting function is no longer used or necessary. This PR removes it.
    
    **Link to tracking Issue:** open-telemetry#23055
    
    Signed-off-by: Braydon Kains <braydonk@google.com>
    braydonk authored Nov 8, 2023
    Configuration menu
    Copy the full SHA
    65b745f View commit details
    Browse the repository at this point in the history

Commits on Nov 9, 2023

  1. [chore] bump go versions in workflows to 1.20.11 and 1.21.4 (open-tel…

    …emetry#29080)
    
    This fixes security vulnerabilities found via govulncheck in the
    standard library when running against the previous patch versions of
    golang. While these vulnerabilities don't actually present themselves in
    the binary, the workflows when running govuln check fail and thus taking
    in the latest patches fix the issue.
    
    
    Testing gets caught in workflow run. Noticed the issue originally when
    running workflows on this pr:
    open-telemetry#28885
    adrielp authored Nov 9, 2023
    Configuration menu
    Copy the full SHA
    330ff0a View commit details
    Browse the repository at this point in the history
  2. [all][chore] Moved from interface{} to any for all go code (open-tele…

    …metry#29072)
    
    Additionally added a golangci-lint.yaml update to automatically apply
    this change to new code going forward
    
    Fixes open-telemetry#23811
    
    ---------
    
    Co-authored-by: Alex Boten <aboten@lightstep.com>
    lucasoskorep and Alex Boten authored Nov 9, 2023
    Configuration menu
    Copy the full SHA
    f4c4485 View commit details
    Browse the repository at this point in the history
  3. [receiver/dockerstats] rename struct and function to keep expected re…

    …ceiver.Factory and pass checkapi (open-telemetry#27086)
    
    Rename struct and function to keep expected receiver.Factory and pass
    checkapi
    
    open-telemetry#26304
    
    go run cmd/checkapi/main.go .
    go test for dockerstatsreceiver
    
    Signed-off-by: sakulali <sakulali@126.com>
    sakulali authored Nov 9, 2023
    Configuration menu
    Copy the full SHA
    57b5dcc View commit details
    Browse the repository at this point in the history
  4. [receiver/mongodbatlasreceiver] add provider resource attributes (ope…

    …n-telemetry#28835)
    
    **Description:** <Describe what has changed.>
    <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
    Ex. Adding a feature - Explain what this achieves.-->
    This feature adds provider resource attributes
    `mongodb_atlas.provider.name` and `mongodb_atlas.region.name` to add
    additional context and filtering capabilities.
    
    **Link to tracking Issue:** <Issue number if applicable>
    open-telemetry#28833 
    
    **Testing:** <Describe what testing was performed and which tests were
    added.>
    Test were automatically updated. Live testing was performed and
    validated on clusters.
    **Documentation:** <Describe the documentation added.>
    Docs were automatically updated.
    JonathanWamsley authored Nov 9, 2023
    Configuration menu
    Copy the full SHA
    320eff4 View commit details
    Browse the repository at this point in the history
  5. [exporter/syslog] Enable component (open-telemetry#28902)

    **Description:** Promote syslogexporter to alpha and add it to
    otelcontribcol
    
    **Link to tracking Issue:**  related to: open-telemetry#21242, open-telemetry#21244, open-telemetry#21245
    
    **Testing:** <Describe what testing was performed and which tests were
    added.>
    Manual tests:
    Configuration:
    ```yaml
    exporters:
      syslog:
        network: tcp
        port: 514
        endpoint: 127.0.0.1
        protocol: rfc5424
    
    receivers:
      filelog:
        start_at: beginning
        include:
        - /Users/kkujawa/git/opentelemetry-collector-contrib/test.txt
        operators:
          - type: syslog_parser
            protocol: rfc5424
    
    service:
      pipelines:
        logs:
          receivers:
            - filelog
          exporters:
            - syslog
    ```
    
    Logs:
    ```
     ./bin/otelcontribcol_darwin_amd64 --config /Users/kkujawa/git/opentelemetry-collector-contrib/bin/config.yaml 
    2023-11-06T12:59:31.656+0100    info    service@v0.88.1-0.20231026220224-6405e152a2d9/telemetry.go:84   Setting up own telemetry...
    2023-11-06T12:59:31.656+0100    info    service@v0.88.1-0.20231026220224-6405e152a2d9/telemetry.go:201  Serving Prometheus metrics      {"address": ":8888", "level": "Basic"}
    2023-11-06T12:59:31.656+0100    info    exporter@v0.88.1-0.20231026220224-6405e152a2d9/exporter.go:275  Development component. May change in the future.        {"kind": "exporter", "data_type": "logs", "name": "syslog"}
    2023-11-06T12:59:31.656+0100    info    syslogexporter@v0.88.0/exporter.go:42   Syslog Exporter configured      {"kind": "exporter", "data_type": "logs", "name": "syslog", "endpoint": "127.0.0.1", "Protocol": "rfc5424", "port": 514}
    2023-11-06T12:59:31.657+0100    info    service@v0.88.1-0.20231026220224-6405e152a2d9/service.go:143    Starting otelcontribcol...      {"Version": "0.88.0-dev", "NumCPU": 16}
    2023-11-06T12:59:31.657+0100    info    extensions/extensions.go:33     Starting extensions...
    2023-11-06T12:59:31.657+0100    info    adapter/receiver.go:45  Starting stanza receiver        {"kind": "receiver", "name": "filelog", "data_type": "logs"}
    2023-11-06T12:59:31.657+0100    info    service@v0.88.1-0.20231026220224-6405e152a2d9/service.go:169    Everything is ready. Begin running and processing data.
    2023-11-06T12:59:31.858+0100    info    fileconsumer/file.go:263        Started watching file   {"kind": "receiver", "name": "filelog", "data_type": "logs", "component": "fileconsumer", "path": "/Users/kkujawa/git/opentelemetry-collector-contrib/test.txt"}
    ```
    kasia-kujawa authored Nov 9, 2023
    Configuration menu
    Copy the full SHA
    b716a4d View commit details
    Browse the repository at this point in the history

Commits on Nov 10, 2023

  1. [chore][pkg/stanza]: when found duplicate, continue from outer loop (o…

    …pen-telemetry#28889)
    
    **Description:** 
    Fix a bug when duplicate readers are added to the active list even after
    the underlying file is closed. To fix this, continue from the outer
    loop.
    This doesn't result in any duplicates, but this will keep producing the
    following annoying error every time.
    ```2023-11-05T02:34:03.530+0530       ERROR       Failed to seek  {"component": "fileconsumer", "path": "/var/folders/fs/njj5c3xx7vdcsr28n19vykw00000gn/T/TestStalePartialFingerprintDiscarded2443925830/001/1616317274.log2", "error": "seek /var/folders/fs/njj5c3xx7vdcsr28n19vykw00000gn/T/TestStalePartialFingerprintDiscarded2443925830/001/1616317274.log2: file already closed"}```
    
    **Testing:** Update the test to check the previouPollFiles
    VihasMakwana authored Nov 10, 2023
    Configuration menu
    Copy the full SHA
    e8116e6 View commit details
    Browse the repository at this point in the history
  2. udp-receiver async - fix data corruption (with buffer pools) (open-te…

    …lemetry#28898)
    
    **Description:** Fixing a bug in udp async mode only (but didn't affect
    the default non-async mode).
    Udp-receiver reuses the same buffer when each packet is processed.
    While that's working fine when running it without async config, it cause
    a significant amount of duplicate packets and corrupted packets being
    sent downstream.
    The reader-async thread is reading a packet from the udp port into the
    buffer, places that buffer in the channel, and reads another packet into
    the same buffer and pushes it to the channel.
    Let's say that the processor-async thread was a bit slow, so it only
    tries to read from the channel after the 2 items were placed in the
    channel. In that case, the processor thread will read 2 items from the
    channel, but it will be the same 2nd packet (since the 1st one was
    overwritten). In some cases, it seems the processor is reading a
    corrupted buffer (since the reader is currently writing into it).
    We can't fix it by having the reader allocate a new buffer before each
    time it reads a packet from the udp port, since that hurts performance
    significantly (reducing it up to ~50%). Instead, use a pool so the
    buffers are reused.
    Before reading a packet, the reader get a buffer from the pool. The
    processor returns it back to the pool after it has been successfully
    processed
    
    **Link to tracking Issue:** 27613
    
    **Testing:** Ran existing unitests. 
    Ran ran stress tests (sending 250k udp packets per second)
    duplicate/corruption issue didn't happen; performance wasn't hurt.
    
    **Documentation:** None
    hovavza authored Nov 10, 2023
    Configuration menu
    Copy the full SHA
    bf5a3f8 View commit details
    Browse the repository at this point in the history
  3. [chore][receiver/windowseventlog] remove duplicate function NewFactor…

    …y and pass checkapi (open-telemetry#29020)
    
    **Description:**
    Remove duplicate function NewFactory and pass checkapi.
    
    **Link to tracking Issue:**
    
    open-telemetry#26304
    
    **Testing:**
    go run cmd/checkapi/main.go .
    go test for windowseventlogreceiver
    
    **Documentation:**
    
    Signed-off-by: sakulali <sakulali@126.com>
    sakulali authored Nov 10, 2023
    Configuration menu
    Copy the full SHA
    55a7d9d View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    21469b1 View commit details
    Browse the repository at this point in the history
  5. [chore][CONTRIBUTING.md] Add triage process link (open-telemetry#29092)

    The `needs triage` label is directly related to how we define triaging.
    Added a link to the triaging definition to make the label's usage more
    clear. (Even though the triaging process paragraph is just above this
    table in the document, it's easy to miss).
    crobert-1 authored Nov 10, 2023
    Configuration menu
    Copy the full SHA
    361b070 View commit details
    Browse the repository at this point in the history

Commits on Nov 11, 2023

  1. fix(processor/k8sattributes): README was misleading/had typoes (open-…

    …telemetry#29108)
    
    **Description:**
    Fixes misleading documentation about which RBAC role is required and
    other invalid YAML I found along the way
    lizthegrey authored Nov 11, 2023
    Configuration menu
    Copy the full SHA
    cd0ffe1 View commit details
    Browse the repository at this point in the history
  2. [processor/k8sattributes] fix(docs): typo for kubernetes label (open-…

    …telemetry#29110)
    
    **Description:** typo for kubernetes label in k8sattributesprocessor
    
    **Link to tracking Issue:** n/a
    
    **Testing:** n/a docs
    
    **Documentation:** n/a
    karmingc authored Nov 11, 2023
    Configuration menu
    Copy the full SHA
    61ba7fd View commit details
    Browse the repository at this point in the history

Commits on Nov 12, 2023

  1. Configuration menu
    Copy the full SHA
    1d4cf91 View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2023

  1. [connector/datadog] Set MutatesData to true (open-telemetry#29114)

    **Description:** 
    Mark datadogconnector as `MutatesData` to prevent data race
    
    **Link to tracking Issue:**
    Fixes
    open-telemetry#29111
    songy23 authored Nov 13, 2023
    Configuration menu
    Copy the full SHA
    c9f1f25 View commit details
    Browse the repository at this point in the history
  2. cmd/telemetrygen: add HTTP export for logs (open-telemetry#29078)

    **Description:**
    
    Closes
    open-telemetry#18867
    
    **Testing:**
    
    Ran opentelemetry-collector locally with debug exporter, then used
    telemetrygen with `--otlp-http` with and without `--otlp-insecure`.
    
    **Documentation:** None
    axw authored Nov 13, 2023
    Configuration menu
    Copy the full SHA
    770f1c0 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    a573f50 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    c9ab41e View commit details
    Browse the repository at this point in the history