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

[exporter/debug] change behavior of "normal" verbosity to be different from "basic" #7806

Closed
andrzej-stencel opened this issue Jun 1, 2023 · 8 comments
Assignees
Labels
exporter/debug Issues related to the Debug exporter

Comments

@andrzej-stencel
Copy link
Member

andrzej-stencel commented Jun 1, 2023

EDIT: Updated the issue to change the debug exporter instead of logging exporter.

Collector version: v0.88.0

Is your feature request related to a problem? Please describe.

Currently there are three verbosity levels available for the Debug exporter: basic, normal, detailed. The basic and normal level result in the same output, which only displays the number of records:

exporters:
  logging:
    verbosity: normal
2023-06-01T13:26:16.044+0200    info    MetricsExporter {"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 1, "data points": 1}

On the other hand, the detailed verbosity displays all the details about telemetry, with multiple lines per span/data point/log record:

2023-06-01T13:30:22.669+0200    info    MetricsExporter {"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 1, "data points": 1}
2023-06-01T13:30:22.669+0200    info    ResourceMetrics #0
Resource SchemaURL: 
ScopeMetrics #0
ScopeMetrics SchemaURL: 
InstrumentationScope otelcol/httpcheckreceiver 0.78.0
Metric #0
Descriptor:
     -> Name: httpcheck.duration
     -> Description: Measures the duration of the HTTP check.
     -> Unit: ms
     -> DataType: Gauge
NumberDataPoints #0
Data point attributes:
     -> http.url: Str(https://opentelemetry.io)
StartTimestamp: 2023-06-01 11:30:18.60847008 +0000 UTC
Timestamp: 2023-06-01 11:30:21.630471795 +0000 UTC
Value: 1038

Describe the solution you'd like

I propose to change the behavior for the normal verbosity to display telemetry details one line per span/data point/log record:

exporters:
  logging:
    verbosity: normal
2023-06-01T13:30:22.669+0200    info    MetricsExporter {"kind": "exporter", "data_type": "metrics", "name": "logging", "resource metrics": 1, "metrics": 1, "data points": 1}
2023-06-01T13:30:22.669+0200    info    MetricDataPoint httpcheck.duration{http.url=Str(https://opentelemetry.io)} 1038

The format is up for discussion, but I believe having one line per telemetry record might be a good "middle ground". The resource and scope details might or might not also be displayed, I'm open to suggestions.

Describe alternatives you've considered

I considered using the File exporter, which outputs JSON with full information on the telemetry to a file.

  • It only outputs to a file. I don't think it's possible to make it output to the collector's console output?
  • The output is too verbose to my taste, also it puts eeeverything in a single line. It would be much more readable to have one line per telemetry record (span/data point/log record).

Another alternative for metrics is to use Prometheus exporter, but it modifies data (replaces dots with underscores etc.).

@perk-sumo
Copy link

I also find logging exporter output a bit too verbose.

It would be awesome if I could provide a "template" for output - so that I can choose which metadata I want to see.
Eg. for logs sometimes I might be interested only in logs, sometimes in logs and some metadata and sometimes in metadata only.

Another name for this mode could be oneline (like in git log output).

@andrzej-stencel
Copy link
Member Author

It would be awesome if I could provide a "template" for output

I agree this might be very useful. A bit more complex (need to define the "templating language", even if very basic), but very useful.

@herrbpl
Copy link

herrbpl commented Aug 31, 2023

Also, what happened to log level ? Why cannot you filter out errors only?

@andrzej-stencel
Copy link
Member Author

Also, what happened to log level ? Why cannot you filter out errors only?

@herrbpl, the logging exporter does not log errors. The exporter's name is in fact misleading, it should be named debug exporter (See #7769), as it is meant for sending telemetry to console for debugging purposes. All output from this exporter is considered "info" level. This is why the loglevel configuration property has been renamed to verbosity, as you do not filter on log levels, but rather you tell the exporter to be more or less verbose.

@andrzej-stencel andrzej-stencel added the exporter/debug Issues related to the Debug exporter label Nov 11, 2023
@andrzej-stencel andrzej-stencel changed the title [exporter/logging] change behavior of "normal" verbosity to be different from "basic" [exporter/debug] change behavior of "normal" verbosity to be different from "basic" Nov 11, 2023
@andrzej-stencel
Copy link
Member Author

andrzej-stencel commented Nov 11, 2023

Related issue:

I believe #8844 should be done first.

codeboten pushed a commit that referenced this issue Nov 15, 2023
…8846)

This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).

**Link to tracking Issue:**

- #8844
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Nov 28, 2023
….90.0 (#30)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/receiver](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/receiver)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Nov 29, 2023
….90.0 (#29)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/exporter](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/exporter)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Nov 29, 2023
….90.0 (#28)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/consumer](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/consumer)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Dec 1, 2023
….90.0 (#30)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/receiver](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/receiver)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Dec 1, 2023
….90.0 (#29)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/exporter](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/exporter)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Dec 1, 2023
….90.0 (#28)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/consumer](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/consumer)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@atoulme
Copy link
Contributor

atoulme commented Dec 19, 2023

I support this proposal.

ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Dec 22, 2023
….90.0 (#28)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/consumer](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/consumer)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
ymotongpoo pushed a commit to ymotongpoo/opentelemetry-collector-extra that referenced this issue Dec 22, 2023
….90.0 (#28)

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[go.opentelemetry.io/collector/consumer](https://togithub.com/open-telemetry/opentelemetry-collector)
| require | minor | `v0.89.0` -> `v0.90.0` |

---

### Release Notes

<details>
<summary>open-telemetry/opentelemetry-collector
(go.opentelemetry.io/collector/consumer)</summary>

###
[`v0.90.0`](https://togithub.com/open-telemetry/opentelemetry-collector/blob/HEAD/CHANGELOG.md#v100v0900)

[Compare
Source](https://togithub.com/open-telemetry/opentelemetry-collector/compare/v0.89.0...v0.90.0)

##### 🛑 Breaking changes 🛑

- `service`: To remain backwards compatible w/ the metrics generated
today, otel generated metrics will be generated without the `_total`
suffix
([#&#8203;7454](https://togithub.com/open-telemetry/opentelemetry-collector/issues/7454))
- `service`: use WithNamespace instead of WrapRegistererWithPrefix
([#&#8203;8988](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8988))
Using this functionality in the otel prom exporter fixes a bug where the
    target_info was prefixed as otelcol_target_info previously.

##### 💡 Enhancements 💡

- `exporter/debug`: Change default `verbosity` from `normal` to `basic`
([#&#8203;8844](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8844))
This change has currently no effect, as `basic` and `normal` verbosity
share the same behavior. This might change in the future though, with
the `normal` verbosity being more verbose than it currently is (see
[open-telemetry/opentelemetry-collector#7806).
This is why we are changing the default to `basic`, which is expected to
stay at the current level of verbosity (one line per batch).
- `exporterhelper`: Fix shutdown logic in persistent queue to not
require consumers to be closed first
([#&#8203;8899](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8899))
- `confighttp`: Support proxy configuration field in all exporters that
support confighttp
([#&#8203;5761](https://togithub.com/open-telemetry/opentelemetry-collector/issues/5761))

##### 🧰 Bug fixes 🧰

- `exporterhelper`: Fix invalid write index updates in the persistent
queue
([#&#8203;8115](https://togithub.com/open-telemetry/opentelemetry-collector/issues/8115))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/ymotongpoo/opentelemetry-collector-extra).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy41OS44IiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@andrzej-stencel andrzej-stencel self-assigned this Jan 17, 2024
andrzej-stencel added a commit to andrzej-stencel/opentelemetry-collector that referenced this issue Apr 9, 2024
mx-psi pushed a commit that referenced this issue Apr 16, 2024
…g exporter (#9922)

This PR is the first part of this draft PR:
#9298.

This refactoring is needed to implement [[exporter/debug] change
behavior of "normal" verbosity to be different from "basic"
#7806](#7806).
I want to change the behavior of the Debug exporter, but leave the
behavior of the Logging exporter unchanged.

**Link to tracking Issue:**

- #7806
mx-psi added a commit that referenced this issue Jun 21, 2024
…ty (#10225)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description

Changes the behavior of `normal` verbosity of the Debug exporter for
logs to display each log record in one line of text.

Note that if the body of the log record contains newlines, the output
will be displayed in multiple lines.

This pull request is part of
#7806;
it implements the change for logs. The changes for metrics and
[traces](#10280)
will be proposed in separate pull requests.

The implementation in this pull request does not display any details on
the resource or the scope of the logs. I would like to propose
displaying the resource and the scope as separate lines in a separate
pull request.

This change applies to the Debug exporter only. The behavior of the
Logging exporter remains unchanged. To use this behavior, switch from
the deprecated Logging exporter to Debug exporter.

#### Link to tracking issue

- #7806

#### Testing

Added unit tests for the formatter.

#### Documentation

Described the formatting in the Debug exporter's README.

---------

Co-authored-by: Roger Coll <roger.coll@elastic.co>
Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
mx-psi added a commit that referenced this issue Jun 24, 2024
…0280)

#### Description

This is an initial barebones implementation that only outputs the span's
name, trace ID and span ID. Other useful fields like duration etc. can
be added in follow-up enhancements.

This pull request is part of
#7806;
it implements the change for traces. The changes for
[logs](#10225)
and metrics will be proposed in separate pull requests.

This change applies to the Debug exporter only. The behavior of the
Logging exporter remains unchanged. To use this behavior, switch from
the deprecated Logging exporter to Debug exporter.

#### Link to tracking issue

- #7806

#### Testing

Added unit tests for the formatter.

#### Documentation

Described the formatting in the Debug exporter's README.

---------

Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com>
mx-psi pushed a commit that referenced this issue Jun 24, 2024
…verbosity (#10462)

#### Description

This pull request is part of
#7806;
it implements the change for metrics. The changes for
[logs](#10225)
and
[traces](#10280)
have been proposed in separate pull requests.

This change applies to the Debug exporter only. The behavior of the
Logging exporter remains unchanged. To use this behavior, switch from
the deprecated Logging exporter to Debug exporter.

#### Link to tracking issue

- #7806

#### Testing

Added unit tests for the formatter.

#### Documentation

Described the formatting in the Debug exporter's README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/debug Issues related to the Debug exporter
Projects
None yet
Development

No branches or pull requests

4 participants