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

Merge 'main' into 'collector_logging' #370

Merged
merged 22 commits into from
Apr 29, 2022

Conversation

damemi
Copy link
Member

@damemi damemi commented Apr 29, 2022

Similar to #362 and #350, merge changes from main into collector_logging to resolve conflicts in preparation for merging the feature branch back into main

dashpole and others added 21 commits March 28, 2022 15:48
…orm#336)

* do not use IsZero on timestamps obtained from pdata
…oogleCloudPlatform#346)

* update fixtures based on upstream changes to the prometheus receiver
…m#343)

* feat(propagator): create a new one-way propagator

CloudTraceOneWayPropagator moves trace info from X-cloud-trace-context
into 'traceparent'.

* Apply suggestions from code review

Co-authored-by: David Ashpole <dashpole@google.com>

* Update propagator to extract both, emit only traceparent

Users will no longer need to create a composite propagator, and there's
a clear precedence order between trace headers.

Co-authored-by: David Ashpole <dashpole@google.com>
…oudPlatform#349)

* demonstrate that integration tests aren't catching failures
* send timeseries syncronously

* update docs, and fix existing broken test
* update opentelemetry collector to v0.49.0

* drop histograms without a sum
…loudPlatform#365)

* Add instrumentation library to span attributes.

* Add test to ensure resource + instrumentation-scope labels show up in spans.

* Expand testing.

* Move resource mapping into shared module, and add monitored resource labels to trace spans.

* Do the go mod tidy shuffle.

* Fix docker build to include new shared module.

* Bump back to go 1.17

* Migrate to SDK version of semantic convention constants instead of Collector's.

* Clean up documents.
@damemi damemi requested a review from dashpole April 29, 2022 18:33
Copy link
Contributor

@dashpole dashpole left a comment

Choose a reason for hiding this comment

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

lgtm after it passes tests...

@damemi damemi merged commit 77fb7e2 into GoogleCloudPlatform:collector_logging Apr 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants