Signed-off-by: Naman-B-Parlecha <[email protected]>

feat(histogram): Refactor ConvertNHCBToClassicHistogram and add unit tests

Signed-off-by: Naman-B-Parlecha <[email protected]>

feat(histogram): Add ConvertNHCBToClassic flag and refactor related types and tests

Signed-off-by: Naman-B-Parlecha <[email protected]>

Replace gopkg.in/yaml.v2 with go.yaml.in/yaml/v2 (prometheus#17151)

* Replace gopkg.in/yaml.v2 with go.yaml.in/yaml/v2
* Upgrade to [email protected]

---------

Signed-off-by: Arve Knudsen <[email protected]>

OTLP to directly write to an interface which can hide storage details (prometheus#16951)

* OTLP writer writes directly to appender

Do not convert to Remote-Write 1.0 protocol. Convert to TSDB Appender interface instead.

For downstream projects that still convert OTLP to something else (e.g. Mimir using
its own RW 1.0+2.0 compatible protocol), introduce a compatibility layer between
OTLP decoding and TSDB Appender. This is the CombinedAppender that hides the
implementation. Name is subject to change.

---------

Signed-off-by: David Ashpole <[email protected]>
Signed-off-by: György Krajcsovits <[email protected]>
Signed-off-by: George Krajcsovits <[email protected]>
Co-authored-by: David Ashpole <[email protected]>
Co-authored-by: Jesus Vazquez <[email protected]>
Co-authored-by: Arve Knudsen <[email protected]>

fix(textparse/protobuf): metric family name corrupted by NHCB parser (prometheus#17156)

* fix(textparse): implement NHCB parsing in ProtoBuf parser directly

The NHCB conversion does some validation, but we can only return error
from Parser.Next() not Parser.Histogram(). So the conversion needs to
happen in Next().

There are 2 cases:
1. "always_scrape_classic_histograms" is enabled, in which case we
convert after returning the classic series. This is to be consistent
with the PromParser text parser, which collects NHCB while spitting out
classic series; then returns the NHCB.
2. "always_scrape_classic_histograms" is disabled. In which case we never
return the classic series.

Signed-off-by: György Krajcsovits <[email protected]>

* refactor(textparse): skip classic series instead of adding NHCB around

Do not return the first classic series from the EntryType state,
switch to EntrySeries. This means we need to start the histogram
field state from -3 , not -2.

In EntrySeries, skip classic series if needed.

Signed-off-by: György Krajcsovits <[email protected]>

* reuse nhcb converter

Signed-off-by: György Krajcsovits <[email protected]>

* test(textparse/nhcb): test corrupting metric family name

NHCB parse doesn't always copy the metric name from the underlying
parser. When called via HELP, UNIT, the string is directly referenced
which means that the read-ahead of NHCB can corrupt it.

Signed-off-by: György Krajcsovits <[email protected]>

Handle error gracefully for the `desymbolizeLabels` function in prompb/io/prometheus/write/v2/symbols.go (prometheus#17160)

Signed-off-by: pipiland <[email protected]>

---------

Signed-off-by: pipiland <[email protected]>
Co-authored-by: pipiland <[email protected]>

feat(histogram): Refactor NHCB to classic conversion with enhanced validation and update tests

Signed-off-by: Naman-B-Parlecha <[email protected]>

feat(histogram): Update NHCB to classic conversion logic and remove tests

Signed-off-by: Naman-B-Parlecha <[email protected]>

refactor(histogram): Update ConvertNHCBToClassicHistogram

Signed-off-by: Naman-B-Parlecha <[email protected]>

test cases for new conversion

Signed-off-by: Naman-B-Parlecha <[email protected]>

refactor(histogram): Simplify label building in ConvertNHCBToClassicHistogram

Signed-off-by: Naman-B-Parlecha <[email protected]>

Both k8s and prometheus libraries have to be updated because they depend
on each other and both have breaking changes.

The following files changed to adopt to the API breaking changes:
- processor/k8sattributesprocessor/internal/kube/fake_informer.go:
  - kubernetes/kubernetes#126387
- receiver/prometheusreceiver/metrics_receiver.go:
  - prometheus/prometheus#16257
- receiver/prometheusremotewritereceiver/receiver.go:
  - prometheus/prometheus#17160

I haven't touched any usages of deprecated k8s API to keep the changeset
small. Created a separate issue for that instead
#43891

Some prometheus tests are failing due to behavioral change in prometheus
library. I skipped them for now and reported
#43893

Both k8s and prometheus libraries have to be updated because they depend
on each other and both have breaking changes.

The following files changed to adopt to the API breaking changes:
- processor/k8sattributesprocessor/internal/kube/fake_informer.go:
  - kubernetes/kubernetes#126387
- receiver/prometheusreceiver/metrics_receiver.go:
  - prometheus/prometheus#16257
- receiver/prometheusremotewritereceiver/receiver.go:
  - prometheus/prometheus#17160

I haven't touched any usages of deprecated k8s API to keep the changeset
small. Created a separate issue for that instead
open-telemetry#43891

Some prometheus tests are failing due to behavioral change in prometheus
library. I skipped them for now and reported
open-telemetry#43893

Both k8s and prometheus libraries have to be updated because they depend
on each other and both have breaking changes.

The following files changed to adopt to the API breaking changes:
- processor/k8sattributesprocessor/internal/kube/fake_informer.go:
  - kubernetes/kubernetes#126387
- receiver/prometheusreceiver/metrics_receiver.go:
  - prometheus/prometheus#16257
- receiver/prometheusremotewritereceiver/receiver.go:
  - prometheus/prometheus#17160

I haven't touched any usages of deprecated k8s API to keep the changeset
small. Created a separate issue for that instead
open-telemetry#43891

Some prometheus tests are failing due to behavioral change in prometheus
library. I skipped them for now and reported
open-telemetry#43893