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] Align with OTel Demo v1.11.0 #37

Closed
wants to merge 53 commits into from

Conversation

bpschmitt
Copy link

@bpschmitt bpschmitt commented Jul 8, 2024

Changes

  • Reworked Helm values.yaml and brought it up to date with the upstream v1.11.0 updates.
  • Added a commented out imageOverride section to the recommendationservice so users can optionally enable a NR APM instrumented version.
  • Updated newrelic-bundle install docs to include kafka, redis, and postgres app values only.
  • Update Postgres and Kafka exporters to include standard app.kubernetes.io/name labels so Prom agent will discover the endpoints

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]>
rogercoll and others added 23 commits June 26, 2024 11:00
…s in use (open-telemetry#1609)

* fix(frontend): Use port 443 as default to connect to flagd if https is in use

* chore: Update CHANGELOG.md

---------

Co-authored-by: Pierre Tessier <[email protected]>
* Change AccountService from go to dotnet (auto)

* fix path

* fix folder name

* dockerfile and other fixes

* add copyright

* fix encoding and cleanup

* Cleanup dockerfile

* Update OTel Auto

* fix kafka processing issues and otel export

* remove eof

* Update changelog

* Use default CancellationDelayMaxMs

* update packages

* fix merge failure

* Fix tracetest 'accountingservice' is not part of the trace anymore

---------

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

Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 6.1.0 to 6.2.0.
- [Release notes](https://github.com/docker/build-push-action/releases)
- [Commits](docker/build-push-action@v6.1.0...v6.2.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: Pierre Tessier <[email protected]>
* 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.
2 out of 12 committers have signed the CLA.

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

@bpschmitt
Copy link
Author

Closing. Will reopen a new PR after I cleanup this mess :)

@bpschmitt bpschmitt closed this Jul 11, 2024
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.