Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .chloggen/codeboten_dep-gcloudstorage.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
change_type: 'deprecation'
component: exporter/google_cloud_storage
note: Introduce new snake case compliant name `google_cloud_storage`
issues: [46733]
2 changes: 1 addition & 1 deletion .chloggen/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,9 @@ components:
- exporter/elasticsearch
- exporter/faro
- exporter/file
- exporter/google_cloud_storage
- exporter/googlecloud
- exporter/googlecloudpubsub
- exporter/googlecloudstorage
- exporter/googlemanagedprometheus
- exporter/honeycombmarker
- exporter/influxdb
Expand Down
2 changes: 1 addition & 1 deletion .chloggen/feature_gcs-use-existing-bucket.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
change_type: breaking

# The name of the component, or a single word describing the area of concern, (e.g. receiver/filelog)
component: exporter/googlecloudstorage
component: exporter/google_cloud_storage

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: "`reuse_if_exists` behavior changed: now checks bucket existence instead of attempting creation"
Expand Down
6 changes: 3 additions & 3 deletions exporter/googlecloudstorageexporter/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ Here is an example configuration for this exporter:

```yaml
exporters:
googlecloudstorage:
google_cloud_storage:
encoding: text_encoding
bucket:
name: bucket-test
Expand All @@ -65,7 +65,7 @@ extensions:

```yaml
exporters:
googlecloudstorage:
google_cloud_storage:
bucket:
name: compressed-logs-bucket
project_id: my-project
Expand All @@ -82,7 +82,7 @@ When the service account lacks project-level bucket creation permissions but has

```yaml
exporters:
googlecloudstorage:
google_cloud_storage:
bucket:
name: existing-bucket
project_id: my-project
Expand Down
8 changes: 5 additions & 3 deletions exporter/googlecloudstorageexporter/factory.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,18 @@ import (

"go.opentelemetry.io/collector/component"
"go.opentelemetry.io/collector/exporter"
"go.opentelemetry.io/collector/exporter/xexporter"

"github.com/open-telemetry/opentelemetry-collector-contrib/exporter/googlecloudstorageexporter/internal/metadata"
)

func NewFactory() exporter.Factory {
return exporter.NewFactory(
return xexporter.NewFactory(
metadata.Type,
createDefaultConfig,
exporter.WithLogs(createLogsExporter, metadata.LogsStability),
exporter.WithTraces(createTracesExporter, metadata.TracesStability),
xexporter.WithLogs(createLogsExporter, metadata.LogsStability),
xexporter.WithTraces(createTracesExporter, metadata.TracesStability),
xexporter.WithDeprecatedTypeAlias(metadata.DeprecatedType),
)
}

Expand Down

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

3 changes: 3 additions & 0 deletions exporter/googlecloudstorageexporter/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ require (
go.opentelemetry.io/collector/confmap/xconfmap v0.147.1-0.20260312222452-c212d203a110
go.opentelemetry.io/collector/consumer v1.53.1-0.20260312222452-c212d203a110
go.opentelemetry.io/collector/exporter v1.53.1-0.20260312222452-c212d203a110
go.opentelemetry.io/collector/exporter/xexporter v0.147.1-0.20260312222452-c212d203a110
go.opentelemetry.io/collector/extension v1.53.1-0.20260312222452-c212d203a110
go.opentelemetry.io/collector/pdata v1.53.1-0.20260312222452-c212d203a110
go.uber.org/goleak v1.3.0
Expand Down Expand Up @@ -60,8 +61,10 @@ require (
github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
github.com/spiffe/go-spiffe/v2 v2.6.0 // indirect
go.opentelemetry.io/auto/sdk v1.2.1 // indirect
go.opentelemetry.io/collector/consumer/xconsumer v0.147.1-0.20260312222452-c212d203a110 // indirect
go.opentelemetry.io/collector/featuregate v1.53.1-0.20260312222452-c212d203a110 // indirect
go.opentelemetry.io/collector/internal/componentalias v0.147.1-0.20260312222452-c212d203a110 // indirect
go.opentelemetry.io/collector/pdata/pprofile v0.147.1-0.20260312222452-c212d203a110 // indirect
go.opentelemetry.io/collector/pipeline v1.53.1-0.20260312222452-c212d203a110 // indirect
go.opentelemetry.io/contrib/detectors/gcp v1.39.0 // indirect
go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc v0.63.0 // indirect
Expand Down
10 changes: 6 additions & 4 deletions exporter/googlecloudstorageexporter/go.sum

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

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

3 changes: 2 additions & 1 deletion exporter/googlecloudstorageexporter/metadata.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
type: googlecloudstorage
type: google_cloud_storage
deprecated_type: googlecloudstorage
display_name: Google Cloud Storage Exporter

status:
Expand Down
14 changes: 7 additions & 7 deletions exporter/googlecloudstorageexporter/testdata/config.yaml
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
googlecloudstorage:
google_cloud_storage:
encoding: test
bucket:
region: test-region
name: test-bucket
project_id: test-project-id

googlecloudstorage/with_partition:
google_cloud_storage/with_partition:
encoding: test
bucket:
region: test-region
Expand All @@ -15,13 +15,13 @@ googlecloudstorage/with_partition:
format: year=%Y
prefix: my-logs

googlecloudstorage/empty_bucket_name:
google_cloud_storage/empty_bucket_name:
encoding: test
bucket:
region: test-region
project_id: test-project-id

googlecloudstorage/invalid_partition_format:
google_cloud_storage/invalid_partition_format:
encoding: test
bucket:
region: test-region
Expand All @@ -30,23 +30,23 @@ googlecloudstorage/invalid_partition_format:
partition:
format: year=%invalid

googlecloudstorage/with_gzip_compression:
google_cloud_storage/with_gzip_compression:
encoding: test
bucket:
region: test-region
name: test-bucket
project_id: test-project-id
compression: gzip

googlecloudstorage/with_zstd_compression:
google_cloud_storage/with_zstd_compression:
encoding: test
bucket:
region: test-region
name: test-bucket
project_id: test-project-id
compression: zstd

googlecloudstorage/unsupported_compression:
google_cloud_storage/unsupported_compression:
encoding: test
bucket:
region: test-region
Expand Down
2 changes: 1 addition & 1 deletion reports/distributions/contrib.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,9 @@ components:
- elasticsearch
- faro
- file
- google_cloud_storage
- googlecloud
- googlecloudpubsub
- googlecloudstorage
- googlemanagedprometheus
- honeycombmarker
- influxdb
Expand Down
Loading