Skip to content

Commit c462cda

Browse files
Merge branch 'main' into flagd-spike
2 parents e6ef65d + 0a1ad08 commit c462cda

File tree

5 files changed

+24
-47
lines changed

5 files changed

+24
-47
lines changed

.github/workflows/run-integration-tests.yml

+5-36
Original file line numberDiff line numberDiff line change
@@ -3,49 +3,18 @@
33
name: Integration Tests
44

55
on:
6-
pull_request_target:
7-
branches:
8-
- main
9-
types: [closed]
6+
pull_request_review:
7+
types:
8+
- submitted
109

1110
jobs:
12-
build_runner:
13-
runs-on: ubuntu-latest
14-
name: "Build Runner"
15-
if: github.event.pull_request.merged == true
16-
steps:
17-
- name: metal-runner-action
18-
uses: equinix-labs/[email protected]
19-
with:
20-
github_token: ${{ secrets.GH_CI_SECRET }}
21-
metal_auth_token: ${{ secrets.METAL_AUTH_TOKEN }}
22-
metal_project_id: ${{ secrets.METAL_PROJECT_ID }}
23-
metro: "da"
24-
plan: "c3.small.x86"
25-
os: "ubuntu_20_04"
2611
run_tests:
27-
needs: build_runner
28-
runs-on: self-hosted
12+
runs-on: ubuntu-latest
2913
name: "Run CI"
14+
if: github.event.review.state == 'APPROVED'
3015
steps:
3116
- name: check out code
3217
uses: actions/checkout@v4
33-
- name: install docker
34-
run: |
35-
curl -fsSL https://get.docker.com -o get-docker.sh
36-
sudo sh get-docker.sh
3718
- name: run tracetesting
3819
run: |
3920
make build && make run-tracetesting
40-
destroy:
41-
needs: [build_runner, run_tests]
42-
runs-on: ubuntu-latest
43-
name: "Cleanup"
44-
if: always()
45-
steps:
46-
- name: metal-sweeper-action
47-
uses: equinix-labs/[email protected]
48-
with:
49-
authToken: ${{ secrets.METAL_AUTH_TOKEN }}
50-
projectID: ${{ secrets.METAL_PROJECT_ID }}
51-
keepProject: true

CHANGELOG.md

+6-2
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,14 @@ the release.
1717
([#1397](https://github.com/open-telemetry/opentelemetry-demo/pull/1397))
1818
* [chore] add nightly releases
1919
([#1398](https://github.com/open-telemetry/opentelemetry-demo/pull/1398))
20-
* [chore] remove unused integration test
21-
([#1406](https://github.com/open-telemetry/opentelemetry-demo/pull/1406))
20+
* [checkoutservice] add producer interceptor for tracing
21+
([#1400](https://github.com/open-telemetry/opentelemetry-demo/pull/1400))
2222
* [productcatalogservice] fix graceful shutdown issues
2323
([#1402](https://github.com/open-telemetry/opentelemetry-demo/pull/1402))
24+
* [chore] remove unused integration test
25+
([#1406](https://github.com/open-telemetry/opentelemetry-demo/pull/1406))
26+
* [CartService] - Add Host Detector
27+
([#1415](https://github.com/open-telemetry/opentelemetry-demo/pull/1415))
2428

2529
## 1.8.0
2630

README.md

+9-8
Original file line numberDiff line numberDiff line change
@@ -56,16 +56,16 @@ keeping it up to date for you.
5656

5757
| | | |
5858
|-----------------------------------------|-----------------------------|----------------------------------------------------------------|
59-
| [AlibabaCloud LogService][AlibabaCloud] | [Google Cloud][GoogleCloud] | [Sentry][Sentry] |
60-
| [AppDynamics][AppDynamics] | [Grafana Labs][GrafanaLabs] | [ServiceNow Cloud Observability][ServiceNowCloudObservability] |
61-
| [Aspecto][Aspecto] | [Guance][Guance] | [Splunk][Splunk] |
62-
| [Axiom][Axiom] | [Helios][Helios] | [Sumo Logic][SumoLogic] |
63-
| [Axoflow][Axoflow] | [Honeycomb.io][Honeycombio] | [TelemetryHub][TelemetryHub] |
64-
| [Azure Data Explorer][Azure] | [Instana][Instana] | [Teletrace][Teletrace] |
65-
| [Coralogix][Coralogix] | [Kloudfuse][Kloudfuse] | [Tracetest][Tracetest] |
59+
| [AlibabaCloud LogService][AlibabaCloud] | [Elastic][Elastic] | [OpenSearch][OpenSearch] |
60+
| [AppDynamics][AppDynamics] | [Google Cloud][GoogleCloud] | [Sentry][Sentry] |
61+
| [Aspecto][Aspecto] | [Grafana Labs][GrafanaLabs] | [ServiceNow Cloud Observability][ServiceNowCloudObservability] |
62+
| [Axiom][Axiom] | [Guance][Guance] | [Splunk][Splunk] |
63+
| [Axoflow][Axoflow] | [Helios][Helios] | [Sumo Logic][SumoLogic] |
64+
| [Azure Data Explorer][Azure] | [Honeycomb.io][Honeycombio] | [TelemetryHub][TelemetryHub] |
65+
| [Coralogix][Coralogix] | [Instana][Instana] | [Teletrace][Teletrace] |
66+
| [Dash0][Dash0] | [Kloudfuse][Kloudfuse] | [Tracetest][Tracetest] |
6667
| [Datadog][Datadog] | [Logz.io][Logzio] | [Uptrace][Uptrace] |
6768
| [Dynatrace][Dynatrace] | [New Relic][NewRelic] | |
68-
| [Elastic][Elastic] | [OpenSearch][OpenSearch] | |
6969

7070
## Contributing
7171

@@ -112,6 +112,7 @@ Emeritus:
112112
[Axoflow]: https://axoflow.com/opentelemetry-support-in-more-detail-in-axosyslog-and-syslog-ng/
113113
[Azure]: https://github.com/Azure/Azure-kusto-opentelemetry-demo
114114
[Coralogix]: https://coralogix.com/blog/configure-otel-demo-send-telemetry-data-coralogix
115+
[Dash0]: https://github.com/dash0hq/opentelemetry-demo
115116
[Datadog]: https://github.com/DataDog/opentelemetry-demo
116117
[Dynatrace]: https://www.dynatrace.com/news/blog/opentelemetry-demo-application-with-dynatrace/
117118
[Elastic]: https://github.com/elastic/opentelemetry-demo

src/cartservice/src/Program.cs

+3-1
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
using OpenTelemetry.Logs;
1515
using OpenTelemetry.Metrics;
1616
using OpenTelemetry.ResourceDetectors.Container;
17+
using OpenTelemetry.ResourceDetectors.Host;
1718
using OpenTelemetry.Resources;
1819
using OpenTelemetry.Trace;
1920
using OpenFeature;
@@ -51,7 +52,8 @@
5152

5253
Action<ResourceBuilder> appResourceBuilder =
5354
resource => resource
54-
.AddDetector(new ContainerResourceDetector());
55+
.AddDetector(new ContainerResourceDetector())
56+
.AddDetector(new HostDetector());
5557

5658
builder.Services.AddOpenTelemetry()
5759
.ConfigureResource(appResourceBuilder)

src/cartservice/src/cartservice.csproj

+1
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
<PackageReference Include="OpenTelemetry.Instrumentation.StackExchangeRedis" Version="1.0.0-rc9.13" />
2222
<PackageReference Include="OpenTelemetry.Instrumentation.Runtime" Version="1.7.0" />
2323
<PackageReference Include="OpenTelemetry.ResourceDetectors.Container" Version="1.0.0-beta.6" />
24+
<PackageReference Include="OpenTelemetry.ResourceDetectors.Host" Version="0.1.0-alpha.2" />
2425
<PackageReference Include="StackExchange.Redis" Version="2.7.17" />
2526
<PackageReference Include="OpenFeature.Contrib.Providers.Flagd" Version="0.1.8" />
2627
<PackageReference Include="OpenFeature" Version="1.4.1" />

0 commit comments

Comments
 (0)