profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/bogdandrutu/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Bogdan Drutu bogdandrutu @Splunk San Francisco Lazy Software Engineer - When in doubt leave it out

bogdandrutu/appengine-try-java 0

App Engine Getting Started with Java runtime

bogdandrutu/ApplicationInsights-Go 0

Microsoft Application Insights SDK for Go

bogdandrutu/auto-assign-action 0

An action which adds reviewers to the pull request when the pull request is opened.

bogdandrutu/b3-propagation 0

Repository that describes and sometimes implements B3 propagation

bogdandrutu/build-tools 0

Building tools provided by OpenTelemetry

bogdandrutu/cloud-bigtable-client 0

Java libraries and HBase client extensions for accessing Google Cloud Bigtable

bogdandrutu/cloud-bigtable-examples 0

Examples of how to use Cloud Bigtable both with GCE map/reduce as well as stand alone applications.

pull request commentopen-telemetry/opentelemetry-collector

wire up obsreport settings

@codeboten thanks for the offer, looking forward to see that happening :)

Aneurysm9

comment created time in 20 minutes

push eventbogdandrutu/opentelemetry-collector

Bogdan Drutu

commit sha a05ce070a6539e6e3cc98b12b6f2ba443c836f0d

Bump go.opentelemetry.io/contrib/* from 0.23.0 to 0.24.0 (#4097) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

Bogdan Drutu

commit sha 8971643be1eb247a458a01b4523da488b49f6270

Fix TLS config in readme (#4095) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

Bogdan Drutu

commit sha 53c25987f8b5035602375dad392cfcbb11049707

Rename ParserProvider to MapProvider, preparation to move to config (#4096) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

Bogdan Drutu

commit sha 77f42c394c2df86ea24da8216f123bfb9888af10

Rename parserprovider.Default to parserprovider.NewDefaultMapProvider (#4099) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

alrex

commit sha 10f543a9a43f41027b6da94c74a37727e84fa2bb

obsreporttest now returns an error (#4080)

view details

Bogdan Drutu

commit sha 2333b10fa58585093a69ee8c12530542ac57a5d2

Remove tests from CircleCI, nothing that is not covered in github actions (#4100) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

Bogdan Drutu

commit sha 18c3cef877f73f23aa843441c766a7a836d4563c

Remove dependency between command line flags and providers Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

push time in 22 minutes

Pull request review commentopen-telemetry/opentelemetry-collector

Remove dependency between command line flags and providers

 import ( 	"go.opentelemetry.io/collector/config" ) -type setFlagProvider struct{}+type propertiesProvider struct {+	props []string+} -// NewSetFlag returns a ParserProvider, that provides a config.Map from set flag(s) properties.+// NewPropertiesProvider returns a ParserProvider, that provides a config.Map from set flag(s) properties. //-// The implementation reads set flag(s) from the cmd and concatenates them as a "properties" file.-// Then the properties file is read and properties are set to the loaded Parser.-func NewSetFlag() ParserProvider {-	return &setFlagProvider{}+// The implementation parses the properties and concatenates them as a "properties" file.+// Then the properties file is read and properties are set to the returned config.Map.+func NewPropertiesProvider(props []string) ParserProvider {

Done.

bogdandrutu

comment created time in 23 minutes

PullRequestReviewEvent

Pull request review commentopen-telemetry/opentelemetry-collector

Remove dependency between command line flags and providers

 func (s *WindowsService) start(elog *eventlog.Log, colErrorChannel chan error) e 	// col.Run blocks until receiving a SIGTERM signal, so needs to be started 	// asynchronously, but it will exit early if an error occurs on startup 	go func() {-		cmd := NewCommand(s.col)-		colErrorChannel <- cmd.Execute()+		colErrorChannel <- col.Run(context.Background())

Done.

bogdandrutu

comment created time in 30 minutes

PullRequestReviewEvent

Pull request review commentopen-telemetry/opentelemetry-collector

Remove dependency between command line flags and providers

 import ( 	"go.opentelemetry.io/collector/config" ) -type fileProvider struct{}+type fileProvider struct {+	fileName string+} -// NewFile returns a new ParserProvider that reads the configuration from a file configured+// NewFileProvider returns a new ParserProvider that reads the configuration from a file configured // via the --config command line flag.

Done.

bogdandrutu

comment created time in 31 minutes

PullRequestReviewEvent
PullRequestReviewEvent

push eventdmitryax/opentelemetry-collector-contrib

alrex

commit sha 5870e74e24205b075f8aec2a17950220c494f52b

make update-otel + required fixes (#5424) * make update-otel * Rename `configparser.ConfigMap` to `config.Map` * rename ParserProvider -> ConfigMapProvider * update CheckExporterTraces, CheckExporterMetrics * update CheckReceiverMetrics, CheckReceiverTraces

view details

Bogdan Drutu

commit sha d63cc733abffce3727f321a76a80be8b07ae0ddc

Merge branch 'main' into change-logfile-undefined-severitytext

view details

push time in an hour

PullRequestReviewEvent

push eventopen-telemetry/opentelemetry-collector-contrib

alrex

commit sha 5870e74e24205b075f8aec2a17950220c494f52b

make update-otel + required fixes (#5424) * make update-otel * Rename `configparser.ConfigMap` to `config.Map` * rename ParserProvider -> ConfigMapProvider * update CheckExporterTraces, CheckExporterMetrics * update CheckReceiverMetrics, CheckReceiverTraces

view details

push time in an hour

pull request commentopen-telemetry/opentelemetry-collector

Move Persistent Queue to exporterhelper/internal

/cc @tigrannajaryan as you reviewed merged the previous PR 🗡️

pmm-sumo

comment created time in an hour

pull request commentopen-telemetry/opentelemetry-collector

Move Persistent Queue to exporterhelper/internal

@pmm-sumo we discussed that this needs to be split into separate PRs, is this ready for review?

pmm-sumo

comment created time in an hour

push eventbogdandrutu/opentelemetry-collector

alrex

commit sha 10f543a9a43f41027b6da94c74a37727e84fa2bb

obsreporttest now returns an error (#4080)

view details

Bogdan Drutu

commit sha 2333b10fa58585093a69ee8c12530542ac57a5d2

Remove tests from CircleCI, nothing that is not covered in github actions (#4100) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

Bogdan Drutu

commit sha 8c0a79296f67092d45ffa012a86e8c0657280ed0

Add an expand provider, currently expands only env variables Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

push time in an hour

push eventbogdandrutu/opentelemetry-collector

alrex

commit sha 10f543a9a43f41027b6da94c74a37727e84fa2bb

obsreporttest now returns an error (#4080)

view details

Bogdan Drutu

commit sha 2333b10fa58585093a69ee8c12530542ac57a5d2

Remove tests from CircleCI, nothing that is not covered in github actions (#4100) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

push time in an hour

delete branch bogdandrutu/opentelemetry-collector

delete branch : rmcircleci

delete time in an hour

push eventopen-telemetry/opentelemetry-collector

Bogdan Drutu

commit sha 2333b10fa58585093a69ee8c12530542ac57a5d2

Remove tests from CircleCI, nothing that is not covered in github actions (#4100) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

push time in an hour

PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentopen-telemetry/opentelemetry-specification

Add optional min and max fields to histogram data model.

 release.  ### OpenTelemetry Protocol -- Add environment variables for configuring the OTLP exporter protocol (`grpc`, `http/protobuf`, `http/json`) ([#1880](https://github.com/open-telemetry/opentelemetry-specification/pull/1880))+- Add environment variables for configuring the OTLP exporter protocol

Why this change? Consider a separate PR

jack-berg

comment created time in 18 hours

PullRequestReviewEvent
PullRequestReviewEvent

push eventopen-telemetry/opentelemetry-collector

alrex

commit sha 10f543a9a43f41027b6da94c74a37727e84fa2bb

obsreporttest now returns an error (#4080)

view details

push time in 19 hours

PR merged open-telemetry/opentelemetry-collector

Reviewers
obsreporttest now returns an error

Description: First part of the work for https://github.com/open-telemetry/opentelemetry-collector/issues/4038, removing the dependency on testing.T from obsreporttest.

Link to tracking Issue: Part of https://github.com/open-telemetry/opentelemetry-collector/issues/4038

+94 -66

1 comment

8 changed files

codeboten

pr closed time in 19 hours

push eventbogdandrutu/opentelemetry-collector

Bogdan Drutu

commit sha 77f42c394c2df86ea24da8216f123bfb9888af10

Rename parserprovider.Default to parserprovider.NewDefaultMapProvider (#4099) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

Bogdan Drutu

commit sha 490869b338386604e37e345390a48f82beab468b

Add an expand provider, currently expands only env variables Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

push time in 20 hours

push eventbogdandrutu/opentelemetry-collector

Bogdan Drutu

commit sha 77f42c394c2df86ea24da8216f123bfb9888af10

Rename parserprovider.Default to parserprovider.NewDefaultMapProvider (#4099) Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>

view details

push time in 20 hours

delete branch bogdandrutu/opentelemetry-collector

delete branch : newdefault

delete time in 20 hours