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

[refactor] OpenTelemetry Demo 1.11.0 updates #39

Merged
merged 57 commits into from
Jul 12, 2024

Conversation

bpschmitt
Copy link

@bpschmitt bpschmitt commented Jul 11, 2024

Changes

  • Pulled latest 1.11.0 updates from upstream
  • Updated Docker install instructions, standardized NR env vars, and tested
  • Updated K8s install instructions, values file, and tested
  • Added labels to Prometheus Exporter values files for Prometheus Agent service discovery
  • Reverted recommendationservice to the default OTel version
  • Added instructions for enabling the NR-instrumented recommendationService and associated values file
  • Added default flagd.json override values file (enables the productcatalogfailure)

To-Do

  • Add option for enabling the NR-instrumented recommendationService to Docker instructions and config

zzh8829 and others added 30 commits May 31, 2024 22:24
* Fix kafka permission in dockerfile

* update changelog
…metry#1579)

This allows the solutio to be opened in any .NET editor and compile right away.

Removes the requirement to manually copy the proto files.

This also now copies the protofiles into the docker container using the original folder name.

Co-authored-by: Pierre Tessier <[email protected]>
* [cartservice] bump Otel .NET SDK to 1.8.1

* chore: add changelog entry for .NET 1.8.1

* docs: merge .NET changelog entries

---------

Co-authored-by: Pierre Tessier <[email protected]>
* add flagd service to minimal docker compose

* chore: add changelog entry

---------

Co-authored-by: Pierre Tessier <[email protected]>
* increase kafka memory limits

* increase kafka memory limits
* reduce image size

* reduce image size

---------

Co-authored-by: Juliano Costa <[email protected]>
* ci: reusable container image generation

* ci: parametrize container repositories name

---------

Co-authored-by: Austin Parker <[email protected]>
* chore: introduce .env.override for docker-compose

* parametrize docker compose cmd and env files

---------

Co-authored-by: Austin Parker <[email protected]>
…metry#1591)

* update dependencies

* Adjust Rust dependency

* Restore flagd env var

---------

Co-authored-by: Juliano Costa <[email protected]>
Co-authored-by: Juliano Costa <[email protected]>
…y#1594)

* chore: Add service version to OTEL_RESOURCE_ATTRIBUTES

* changelog
* fix kafka acks

* Update src/checkoutservice/main.go

Co-authored-by: Juliano Costa <[email protected]>

* align attribute names with semantic conventions

* checkout: recover from kafka restart

---------

Co-authored-by: Juliano Costa <[email protected]>
* Update image version

* latest otel col contrib

* Update minor version
Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5.0.0 to 5.4.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v5.0.0...v5.4.0)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: configure renovate to bump image deps

* chore: add tracetest and opensearch to renovate
…telemetry#1607)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 5.4.0 to 6.0.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v5.4.0...v6.0.0)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
open-telemetry#1611)

Bumps [gradle/wrapper-validation-action](https://github.com/gradle/wrapper-validation-action) from 3.3.2 to 3.4.2.
- [Release notes](https://github.com/gradle/wrapper-validation-action/releases)
- [Commits](gradle/wrapper-validation-action@v3.3.2...v3.4.2)

---
updated-dependencies:
- dependency-name: gradle/wrapper-validation-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* run on native runner

* fix lint and remove unneeded options

* add force build

* fix lint
* [cartservice] Bump OTel .NET SDK to 1.9.0

together with other dependencies

* Fix changelog

* Update CHANGELOG.md

* Lint

---------

Co-authored-by: Juliano Costa <[email protected]>
* chore: parametrize services Dockerfile path

* chore: add ad service relative path

* chore: add services path as environment variable

* ci: fix emailservice context build

* remove unused *_PATH environment variable

---------

Co-authored-by: Juliano Costa <[email protected]>
…telemetry#1618)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.0.0 to 6.0.1.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v6.0.0...v6.0.1)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  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>
Co-authored-by: Juliano Costa <[email protected]>
Co-authored-by: Pierre Tessier <[email protected]>
* replace redis with valkey

* changelog

---------

Co-authored-by: Mikko Viitanen <[email protected]>
…telemetry#1620)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.0.1 to 6.1.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v6.0.1...v6.1.0)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mikko Viitanen <[email protected]>
puckpuck and others added 15 commits June 28, 2024 10:23
* clarify complete release process

* clarify complete release process

* clarify complete release process

* clarify complete release process

* clarify complete release process
* update demo role membership

* update demo role membership
* Add docker stats receiver

* changelog

* Nit: prevent write access to the Docker socket

Co-authored-by: Roger Coll <[email protected]>

* Add compose minimal

---------

Co-authored-by: Roger Coll <[email protected]>
…telemetry#1656)

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.2.0 to 6.3.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v6.2.0...v6.3.0)

---
updated-dependencies:
- dependency-name: docker/build-push-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* remove fractional config from rule definition

Signed-off-by: Michael Beemer <[email protected]>

* add random failure logic to ad service

Signed-off-by: Michael Beemer <[email protected]>

* add random failure logic to cart service

Signed-off-by: Michael Beemer <[email protected]>

---------

Signed-off-by: Michael Beemer <[email protected]>
* optimize trace testing

* run trace based tests concurrently

* fix lint failures

---------

Co-authored-by: Juliano Costa <[email protected]>
* fix build-images workflow

* use consistent naming for workflows
@CLAassistant
Copy link

CLAassistant commented Jul 11, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 13 committers have signed the CLA.

✅ rogercoll
✅ bpschmitt
✅ lazyplatypus
❌ zzh8829
❌ Mpdreamz
❌ julianocosta89
❌ puckpuck
❌ flands
❌ SvenKirschbaum
❌ Kielek
❌ austinlparker
❌ beeme1mr
❌ RassK
You have signed the CLA already but the status is still pending? Let us recheck it.

@lazyplatypus lazyplatypus self-assigned this Jul 12, 2024
lazyplatypus
lazyplatypus previously approved these changes Jul 12, 2024
@lazyplatypus lazyplatypus merged commit bf2a990 into newrelic:main Jul 12, 2024
26 of 33 checks passed
@reese-lee reese-lee mentioned this pull request Jul 16, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.