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

[receiver/cloudflarereceiver] New Alpha Logs Component cloudflarereceiver #17993

Closed
Closed
Prev Previous commit
Next Next commit
add otel requirements
  • Loading branch information
JonathanWamsley committed Feb 14, 2023
commit a2d8ce4961e268707f5f6a2992d7fab4eb256433
16 changes: 16 additions & 0 deletions .chloggen/add-cloudflarereceiver.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: new_component

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: cloudflarereceiver

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Adds the 'cloudflarereceiver' in alpha state

# One or more tracking issues related to the change
issues: [17376]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:
1 change: 1 addition & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,7 @@ receiver/azureblobreceiver/ @open-telemetry/collect
receiver/bigipreceiver/ @open-telemetry/collector-contrib-approvers @djaglowski @StefanKurek
receiver/carbonreceiver/ @open-telemetry/collector-contrib-approvers @pjanotti
receiver/chronyreceiver/ @open-telemetry/collector-contrib-approvers @MovieStoreGuy @jamesmoessis
receiver/cloudflarereceiver/ @open-telemetry/collector-contrib-approvers @djaglowski @JonathanWamsley
receiver/cloudfoundryreceiver/ @open-telemetry/collector-contrib-approvers @agoallikmaa @pellared @crobert-1
receiver/collectdreceiver/ @open-telemetry/collector-contrib-approvers @atoulme
receiver/couchdbreceiver/ @open-telemetry/collector-contrib-approvers @djaglowski
Expand Down
3 changes: 3 additions & 0 deletions cmd/configschema/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -430,6 +430,7 @@ require (
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/bigipreceiver v0.71.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/carbonreceiver v0.71.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/chronyreceiver v0.71.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/cloudflarereceiver v0.71.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/cloudfoundryreceiver v0.71.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/collectdreceiver v0.71.0 // indirect
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/couchdbreceiver v0.71.0 // indirect
Expand Down Expand Up @@ -894,6 +895,8 @@ replace github.com/open-telemetry/opentelemetry-collector-contrib/receiver/carbo

replace github.com/open-telemetry/opentelemetry-collector-contrib/receiver/chronyreceiver => ../../receiver/chronyreceiver

replace github.com/open-telemetry/opentelemetry-collector-contrib/receiver/cloudflarereceiver => ../../receiver/cloudflarereceiver

replace github.com/open-telemetry/opentelemetry-collector-contrib/receiver/cloudfoundryreceiver => ../../receiver/cloudfoundryreceiver

replace github.com/open-telemetry/opentelemetry-collector-contrib/receiver/collectdreceiver => ../../receiver/collectdreceiver
Expand Down
2 changes: 2 additions & 0 deletions internal/components/components.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,7 @@ import (
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/bigipreceiver"
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/carbonreceiver"
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/chronyreceiver"
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/cloudflarereceiver"
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/cloudfoundryreceiver"
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/collectdreceiver"
"github.com/open-telemetry/opentelemetry-collector-contrib/receiver/couchdbreceiver"
Expand Down Expand Up @@ -231,6 +232,7 @@ func Components() (otelcol.Factories, error) {
bigipreceiver.NewFactory(),
carbonreceiver.NewFactory(),
chronyreceiver.NewFactory(),
cloudflarereceiver.NewFactory(),
Copy link
Contributor

Choose a reason for hiding this comment

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

you should check out how to add your receiver now to this list, this has changed.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

cloudfoundryreceiver.NewFactory(),
collectdreceiver.NewFactory(),
couchdbreceiver.NewFactory(),
Expand Down
4 changes: 4 additions & 0 deletions internal/components/receivers_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,10 @@ func TestDefaultReceivers(t *testing.T) {
},
skipLifecyle: true, // Panics after test have completed, requires a wait group
},
{
receiver: "cloudflare",
skipLifecyle: true, // Requires auth cred to run
},
{
receiver: "cloudfoundry",
skipLifecyle: true, // Requires UAA (auth) endpoint to run
Expand Down
2 changes: 0 additions & 2 deletions receiver/cloudflarereceiver/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.