Skip to content

Conversation

@tigrannajaryan
Copy link
Member

Batch processor is typical part of any reasonable configuration.
Added it to the configuration of E2E tests so they better reflect
real world usage.

@tigrannajaryan tigrannajaryan changed the title Add batch processor to E2E tests [WIP] Add batch processor to E2E tests Dec 13, 2019
@tigrannajaryan tigrannajaryan force-pushed the feature/tigran/add-batch branch from 6c8ba70 to ff957d2 Compare December 13, 2019 14:53
@codecov-io
Copy link

Codecov Report

Merging #464 into master will increase coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #464      +/-   ##
==========================================
+ Coverage   75.72%   75.77%   +0.05%     
==========================================
  Files         120      120              
  Lines        7345     7345              
==========================================
+ Hits         5562     5566       +4     
+ Misses       1520     1517       -3     
+ Partials      263      262       -1
Impacted Files Coverage Δ
exporter/opencensusexporter/opencensus.go 51.57% <0%> (+4.21%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4e99ac7...ff957d2. Read the comment docs.

@tigrannajaryan tigrannajaryan changed the title [WIP] Add batch processor to E2E tests Add batch processor to E2E tests Dec 13, 2019
Copy link
Contributor

@flands flands left a comment

Choose a reason for hiding this comment

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

LGTM

Batch processor is typical part of any reasonable configuration.
Added it to the configuration of E2E tests so they better reflect
real world usage.
@tigrannajaryan tigrannajaryan force-pushed the feature/tigran/add-batch branch from ff957d2 to 57fce2b Compare December 13, 2019 16:21
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

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

LGTM

@tigrannajaryan tigrannajaryan merged commit be399dc into open-telemetry:master Dec 13, 2019
@tigrannajaryan tigrannajaryan deleted the feature/tigran/add-batch branch December 13, 2019 22:04
MovieStoreGuy pushed a commit to atlassian-forks/opentelemetry-collector that referenced this pull request Nov 11, 2021
Looks like a fallout from adding the `otel` part to the package
names. Spotted by Matej Gera.
hughesjj pushed a commit to hughesjj/opentelemetry-collector that referenced this pull request Apr 27, 2023
…y#464)

Bumps [boto3](https://github.com/boto/boto3) from 1.17.90 to 1.17.91.
- [Release notes](https://github.com/boto/boto3/releases)
- [Changelog](https://github.com/boto/boto3/blob/develop/CHANGELOG.rst)
- [Commits](boto/boto3@1.17.90...1.17.91)

---
updated-dependencies:
- dependency-name: boto3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
swiatekm pushed a commit to swiatekm/opentelemetry-collector that referenced this pull request Oct 9, 2024
* add frontendproxy

* add frontendproxy

* add grafana routing config

* fix grafana routing config

* use port 80 for Grafana

* add exposed services paths
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants