Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Enhance Open Telemetry specific features#1663

Open
sughosneo wants to merge 80 commits intodotnet-architecture:feature/open-telemetryfrom
sughosneo:feature/enable-telemetry
Open

Enhance Open Telemetry specific features#1663
sughosneo wants to merge 80 commits intodotnet-architecture:feature/open-telemetryfrom
sughosneo:feature/enable-telemetry

Conversation

@sughosneo
Copy link
Contributor

@sughosneo sughosneo commented May 5, 2021

This PR includes following changes :

  • Refactored OpenTelemetry related configuration into a different project OpenTelemetry.Customization.csproj
  • Included Zipkin Helm chart
  • Enabled default trace with Zipkin in K8s deployment.
  • Enabled Zipkin related configuration for all the other services.

To test the PR :

docker-compose :
docker-compose -f docker-compose.yml -f docker-compose.override.yml -f docker-compose.opentelemetry.zipkin.yml up
local kubernetes :
.\deploy-all.ps1 -imageTag linux-latest -useLocalk8s $true -imagePullPolicy Never -enableTrace $true

sughosneo and others added 30 commits April 7, 2021 18:05
@sughosneo
Enabled Instrumentations with OpenTelemetry
18e39fd
@mfcallahan @IEvangelist
updated CardType and Enumeration classes (dotnet-architecture#1528)
a71b004 
* updated CardType and Enumeration classes

* Update per IEvangelist's suggestion

* Update src/Services/Ordering/Ordering.Domain/SeedWork/Enumeration.cs

Co-authored-by: David Pine <david.pine@microsoft.com>

Co-authored-by: Sumit Ghosh <sumit.ghosh@neudesic.com>
Co-authored-by: David Pine <david.pine@microsoft.com>
@sughosneo
Refactored opentelemetry module to a common project
516de21
@mseada94
Fix aggregator address (dotnet-architecture#1646)
6012fb0
@sughosneo
Included Zipkin helm chart
6093ec0
@sughosneo
Updated Ingress API version
c01ac6e
@sughosneo
Included tracing for WebMVC
ef548da
@richlander
Update sdk tag (dotnet-architecture#1647)
d0cd283
@colindembovsky
Adding deploy workflows and script
acb304c
@colindembovsky
Update checkout from @v1 to @v2
996da77
@colindembovsky
Update chart-root to CHART_ROOT
8414627
@colindembovsky
Update chart to CHART
ede057c
@colindembovsky
Update namespace to NAMESPACE
7343ceb
@colindembovsky
Remove app-name
c152b03
@colindembovsky
Update DeployToK8s to deploy-to-k8s
391de9f
@colindembovsky
Adding unit tests
9affd62
@colindembovsky
Revert
d261523
@colindembovsky
Full revert
e858b1a
@colindembovsky
Adding unit tests
53ad2d1
@colindembovsky
Add workflow_dispatch
c6763a2
@colindembovsky
Change working dir
99e8631
@colindembovsky
Adding workflow_dispatch and unit tests
47ca568
@colindembovsky
Adding unit tests
17dc8ef
@colindembovsky
Up verbosity
f388b4f
@colindembovsky
Update arg
95fc219
@colindembovsky
Change verbosity to only test
87016c3
@colindembovsky
Fixing dotnet test cmd
5ec0228
@colindembovsky
Fix file name
e10f9c1
@colindembovsky
Fix perms
e367740
@colindembovsky
Fix casing
d8c6578
Sumit Ghosh and others added 29 commits June 9, 2021 19:51
Updated package (dotnet-architecture#1696)
08d3a2b
@mvelosop
Initial eShop-Learn version
dbf782d
@mvelosop
Initial version running
bbe423c
@mvelosop
Fix catalog search error
c60a493
@mvelosop
Remove campaigns, marketing and coupon-related components
ae45ffb
@mvelosop
Initial SPA application working
7d26ef8
@mvelosop
running version without refresh page error
07e0bfe
@mvelosop
Update to .NET 5
2a0db63
@mvelosop
Update target js version
4bf3079
@mvelosop
Update to Angular 9
4bf0db6
@mvelosop
Update packages required by Angular 10
c8a8884
@mvelosop
install packages for Angular 9
c7f482d
@mvelosop
Angular 10 version running
066b055
@mvelosop
Update versions in package.json
67c248b
@mvelosop
Update other packages to latest versions
9194b2c
@mvelosop
Handle SPA/MVC branding for login and registration views
4b0c2c5
@mvelosop
Update README, catalog image files and MVC page size
7405eba
@mvelosop
Update item images
5afcd37
@mvelosop
Merge pull request dotnet-architecture#1705 from dotnet-architecture/…
e9aa525 
…feature/refresh-spa-theme

Refresh SPA theme
@mvelosop
Update Angular version to 11.2
e858ac0
@mvelosop
Merge pull request dotnet-architecture#1711 from dotnet-architecture/…
8c27163 
…update/angular-version

Update Angular version to 11.2
@kaypee90
Correct typo error in Shipping Address (dotnet-architecture#1712)
6982f0e 
Co-authored-by: kwabena <kwabena@mpharma.com>
Updates npm packages (dotnet-architecture#1710)
6381df5 
* Updated npm packages

* Updated packages
Included Envoyproxy filrewall rules in the script (dotnet-architectur…
2afc003 
…e#1704)
@sughosneo
Included feedback
6946735
Updated npm packages (dotnet-architecture#1714)
8bb15ce
@alan0428a
Binding queue using existing cosumerChannel (dotnet-architecture#1716)
5c57b8c 
Co-authored-by: Alan.Yu <alan.yu@hidglobal.com>
@sughosneo
Merge branch 'dev' into feature/enable-telemetry
8099d27
@sughosneo
Updated project files
0601707
@sughosneo sughosneo marked this pull request as ready for review July 7, 2021 10:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@cijothomas cijothomas cijothomas left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.