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.

Sign up for GitHub

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

Jump to bottom

[receiver/googlecloudmonitoringreceiver] Adding Google Cloud monitoring receiver #33886

Closed
wants to merge 2 commits into from
Closed

[receiver/googlecloudmonitoringreceiver] Adding Google Cloud monitoring receiver #33886

wants to merge 2 commits into from

Conversation

abhishek-at-cloudwerx
Copy link
Contributor

Description: Adding Google Cloud Monitoring receiver

Link to tracking Issue: #33762

Testing: Test cases for configuration added

Documentation: README file added for describing the component

@abhishek-at-cloudwerx abhishek-at-cloudwerx force-pushed the googlecloudmonitoringreceiver-skeleton branch from 33bd99e to 143dee0 Compare July 3, 2024 16:13
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Jul 3, 2024 •
edited
Loading

CLA Not Signed

@crobert-1
Copy link
Member

It looks like a lot of changes from this PR may be due to not being up to date with main, so I'm going to update the branch.

@crobert-1
Copy link
Member

@abhishek-at-cloudwerx, can you please sign the EasyCLA? This PR can't proceed until it's signed.

crobert-1
crobert-1 reviewed Jul 3, 2024
View reviewed changes
Copy link
Member

@crobert-1 crobert-1 left a comment

Choose a reason for hiding this comment

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

It looks like this PR has deleted a lot of changelog entries, and includes quite a few unrelated changes. Please remove unrelated changes.

@abhishek-at-cloudwerx
Copy link
Contributor Author

Closing in favour of Org CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crobert-1 crobert-1 crobert-1 left review comments

@MovieStoreGuy MovieStoreGuy Awaiting requested review from MovieStoreGuy

@djaglowski djaglowski Awaiting requested review from djaglowski djaglowski is a code owner

@TylerHelmuth TylerHelmuth Awaiting requested review from TylerHelmuth TylerHelmuth is a code owner

@bogdandrutu bogdandrutu Awaiting requested review from bogdandrutu bogdandrutu is a code owner

@evan-bradley evan-bradley Awaiting requested review from evan-bradley evan-bradley is a code owner

@dmitryax dmitryax Awaiting requested review from dmitryax dmitryax is a code owner

@jpkrohling jpkrohling Awaiting requested review from jpkrohling jpkrohling is a code owner

@dashpole dashpole Awaiting requested review from dashpole dashpole is a code owner

@carsonip carsonip Awaiting requested review from carsonip

@Frapschen Frapschen Awaiting requested review from Frapschen

@greatestusername greatestusername Awaiting requested review from greatestusername

@hanjm hanjm Awaiting requested review from hanjm

@JaredTan95 JaredTan95 Awaiting requested review from JaredTan95

@kentquirk kentquirk Awaiting requested review from kentquirk

@mapno mapno Awaiting requested review from mapno

@shalper2 shalper2 Awaiting requested review from shalper2

@SpencerTorres SpencerTorres Awaiting requested review from SpencerTorres

@srprash srprash Awaiting requested review from srprash

@wangzlei wangzlei Awaiting requested review from wangzlei

@ycombinator ycombinator Awaiting requested review from ycombinator

Assignees

@crobert-1 crobert-1

Labels
cmd/otelcontribcol otelcontribcol command cmd/oteltestbedcol connector/servicegraph connector/sum exporter/awsxray exporter/clickhouse exporter/elasticsearch pkg/ottl pkg/stanza
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@abhishek-at-cloudwerx @crobert-1