Skip to content

[receiver/azure_functions] Introduce new component#46584

Merged
andrzej-stencel merged 17 commits into
open-telemetry:mainfrom
tetianakravchenko:azurefunctionsreceiver
Mar 15, 2026
Merged

[receiver/azure_functions] Introduce new component#46584
andrzej-stencel merged 17 commits into
open-telemetry:mainfrom
tetianakravchenko:azurefunctionsreceiver

Conversation

@tetianakravchenko

@tetianakravchenko tetianakravchenko commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Description

This change includes only overall structure (including factory), readme and proposed configuration for a new component.
Implementation will follow in future PRs.

Sponsored by @jmacd

Link to tracking issue

Part 1 of #43507

Testing

Initial tests were added

Documentation

Initial README added.

@tetianakravchenko
[receiver/azurefunctionsreceiver] Introduce new component
b4f2954 
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@tetianakravchenko tetianakravchenko requested a review from a team as a code owner March 3, 2026 12:06
@github-actions

github-actions Bot commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib.

Important reminders:

A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better!

@github-actions github-actions Bot added the first-time contributor PRs made by new contributors label Mar 3, 2026
@paulojmdias

paulojmdias commented Mar 3, 2026

Copy link
Copy Markdown
Member

/workflow-approve

@edmocosta edmocosta requested a review from jmacd March 3, 2026 16:43
@tetianakravchenko
add changelog record; fix linter issues
9b9c5c8 
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@tetianakravchenko tetianakravchenko mentioned this pull request Mar 4, 2026
Open
@paulojmdias

paulojmdias commented Mar 4, 2026

Copy link
Copy Markdown
Member

/workflow-approve

constanca-m
constanca-m reviewed Mar 4, 2026
View reviewed changes

@constanca-m constanca-m left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Tanya! Looks great, just two comments

Comment thread receiver/azurefunctionsreceiver/README.md Outdated
Comment on lines +34 to +35
| `http` | confighttp.ServerConfig | **Required.** HTTP server settings (e.g. `endpoint: :9090`). Typically use `FUNCTIONS_CUSTOMHANDLER_PORT`. |
| `auth` | component.ID | **Required.** Component ID of the extension that provides Azure authentication (e.g. token credential). |

@constanca-m constanca-m Mar 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If you already put required in the docs, maybe add it to the config validate already? In the skeleton PR, I usually add the validate implementation, but if you prefer to do it later, it is also fine. In that case, I would remove the required from this README

@tetianakravchenko tetianakravchenko Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've added config validation

Comment thread receiver/azurefunctionsreceiver/receiver.go Outdated

type functionsReceiver struct{}

func newFunctionsReceiver(_ *Config, _ receiver.Settings, _ consumer.Logs) *functionsReceiver {

@constanca-m constanca-m Mar 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
func newFunctionsReceiver(_ *Config, _ receiver.Settings, _ consumer.Logs) *functionsReceiver {
func newFunctionsReceiver(_ *Config, _ receiver.Settings, _ consumer.Logs) receiver.Receiver {

@tetianakravchenko
add config validation; address review comments
e5abc46 
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
zmoog
zmoog approved these changes Mar 4, 2026
View reviewed changes

@zmoog zmoog left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Added a few minor comments, but this LGTM!

Comment thread receiver/azurefunctionsreceiver/README.md Outdated
Comment thread receiver/azurefunctionsreceiver/README.md
Comment thread receiver/azurefunctionsreceiver/README.md Outdated
Comment thread receiver/azurefunctionsreceiver/config.go Outdated
Comment on lines +41 to +43
if cfg.Auth == (component.ID{}) {
errs = append(errs, errors.New("auth must be set"))
}

@zmoog zmoog Mar 4, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We can make this optional—or remove it for now—since the azureauth extension is needed for blob storage, something we're going to contribute a little later.

@tetianakravchenko tetianakravchenko Mar 4, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good point! I made it optional

tetianakravchenko and others added 3 commits March 4, 2026 16:01
@tetianakravchenko @zmoog
Apply suggestions from code review
e21355b 
Co-authored-by: Maurizio Branca <maurizio.branca@elastic.co>
@tetianakravchenko
Merge branch 'main' into azurefunctionsreceiver
4b2a9a0
@tetianakravchenko
make auth configuration optional; use azure_encoding instead of azure…
f7eb49e 
…resourcelogs_encoding in config tests

Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@paulojmdias

paulojmdias commented Mar 4, 2026

Copy link
Copy Markdown
Member

/workflow-approve

@songy23 songy23 added the Accepted Component New component has been sponsored label Mar 4, 2026
andrzej-stencel
andrzej-stencel requested changes Mar 6, 2026
View reviewed changes
Comment thread receiver/azurefunctionsreceiver/metadata.yaml Outdated
@tetianakravchenko
follow naming convention for the receiver: rename to azure_functions
97e532e 
Signed-off-by: Tetiana Kravchenko <tetiana.kravchenko@elastic.co>
@paulojmdias

paulojmdias commented Mar 6, 2026

Copy link
Copy Markdown
Member

/workflow-approve

andrzej-stencel
andrzej-stencel reviewed Mar 6, 2026
View reviewed changes

@andrzej-stencel andrzej-stencel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Needs some more updates adding the underscore in name - those mentioned below and more.

Comment thread .chloggen/config.yaml Outdated
Comment thread .chloggen/feat_new-component-azurefunctionsreceiver.yaml Outdated
jmacd
jmacd approved these changes Mar 13, 2026
View reviewed changes

@jmacd jmacd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

@otelbot

otelbot Bot commented Mar 15, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution @tetianakravchenko! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help.

@tetianakravchenko tetianakravchenko mentioned this pull request Mar 19, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmacd jmacd jmacd approved these changes

@andrzej-stencel andrzej-stencel andrzej-stencel approved these changes

@MovieStoreGuy MovieStoreGuy Awaiting requested review from MovieStoreGuy

@crobert-1 crobert-1 Awaiting requested review from crobert-1

@atoulme atoulme Awaiting requested review from atoulme

@pjanotti pjanotti Awaiting requested review from pjanotti

@dashpole dashpole Awaiting requested review from dashpole

@VihasMakwana VihasMakwana Awaiting requested review from VihasMakwana

@tigrannajaryan tigrannajaryan Awaiting requested review from tigrannajaryan

@dmitryax dmitryax Awaiting requested review from dmitryax

@fatsheep9146 fatsheep9146 Awaiting requested review from fatsheep9146

@dehaansa dehaansa Awaiting requested review from dehaansa

@ChrsMark ChrsMark Awaiting requested review from ChrsMark

@ArthurSens ArthurSens Awaiting requested review from ArthurSens

@evan-bradley evan-bradley Awaiting requested review from evan-bradley

@TylerHelmuth TylerHelmuth Awaiting requested review from TylerHelmuth

@axw axw Awaiting requested review from axw

@braydonk braydonk Awaiting requested review from braydonk

@mx-psi mx-psi Awaiting requested review from mx-psi

@edmocosta edmocosta Awaiting requested review from edmocosta

@bogdandrutu bogdandrutu Awaiting requested review from bogdandrutu

@songy23 songy23 Awaiting requested review from songy23

@jsuereth jsuereth Awaiting requested review from jsuereth

@codeboten codeboten Awaiting requested review from codeboten

+3 more reviewers

@zmoog zmoog zmoog approved these changes

@MichaelKatsoulis MichaelKatsoulis MichaelKatsoulis approved these changes

@constanca-m constanca-m constanca-m approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jmacd jmacd

@edmocosta edmocosta

Labels

Accepted Component New component has been sponsored cmd/golden cmd/opampsupervisor cmd/telemetrygen telemetrygen command confmap/provider/aesprovider confmap/provider/googlesecretmanagerprovider confmap/provider/s3provider confmap/provider/secretsmanagerprovider connector/count connector/datadog connector/exceptions connector/failover connector/grafanacloud connector/metricsaslogs connector/otlpjson connector/roundrobin connector/routing connector/servicegraph connector/signaltometrics connector/slowsql connector/spanmetrics connector/sum exporter/alertmanager exporter/alibabacloudlogservice Alibaba components exporter/awscloudwatchlogs awscloudwatchlogs exporter exporter/awsemf awsemf exporter exporter/awskinesis exporter/awss3 exporter/awsxray exporter/azureblob exporter/azuredataexplorer exporter/azuremonitor exporter/bmchelix exporter/cassandra exporter/clickhouse exporter/coralogix exporter/datadog Datadog components exporter/dataset exporter/doris exporter/elasticsearch exporter/faro exporter/file exporter/googlecloud exporter/googlecloudpubsub exporter/googlecloudstorage exporter/googlemanagedprometheus Google Managed Prometheus exporter exporter/honeycombmarker exporter/influxdb exporter/kafka exporter/loadbalancing exporter/logicmonitor exporter/logzio exporter/mezmo exporter/opensearch exporter/otelarrow exporter/prometheus exporter/prometheusremotewrite exporter/pulsar exporter/rabbitmq exporter/sematext exporter/sentry exporter/signalfx exporter/splunkhec exporter/stef exporter/sumologic exporter/syslog exporter/tencentcloudlogservice exporter/tinybird exporter/zipkin extension/ack extension/asapauth extension/awsproxy extension/azureauth extension/basicauth extension/bearertokenauth extension/cgroupruntime extension/datadog extension/encoding/avrologencoding extension/encoding/awscloudwatchmetricstreams extension/encoding/awslogsencoding extension/encoding/azureencoding extension/encoding/googlecloudlogentryencoding extension/encoding/jaegerencoding extension/encoding/jsonlogencoding extension/encoding/otlpencoding extension/encoding/skywalkingencoding extension/encoding/textencoding extension/encoding first-time contributor PRs made by new contributors

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@tetianakravchenko @paulojmdias @zmoog @jmacd @MichaelKatsoulis @andrzej-stencel @constanca-m @songy23 @edmocosta