Skip to content

Commit 6efc4a8

Browse files
Validate Jaeger demo configurations in CI workflow (#7464)
## Which problem is this PR solving? - Part of #7115 - We needed validation workflow of jaeger demo configurations , Instead of creating workflow from scratch we could reuse already created workflow with minor changes present in .github/workflows/ci-docker-hotrod.yml ## Description of the changes - Update the build hotrod script to use oci example's deployment script . ## How was this change tested? - ## Checklist - [ ] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [ ] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `npm run lint` and `npm run test` --------- Signed-off-by: chahat sagar <109112505+chahatsagarmain@users.noreply.github.com>
1 parent 47ddfe5 commit 6efc4a8

File tree

14 files changed

+168
-173
lines changed

14 files changed

+168
-173
lines changed

.github/workflows/ci-deploy-demo.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,10 @@ jobs:
3333

3434
- name: Deploy using appropriate script
3535
run: |
36-
cd ./examples/oci
3736
if [ "${{ github.event_name }}" = "schedule" ]; then
3837
echo "🕒 Scheduled run - using deploy-all.sh (upgrade mode)"
39-
bash ./deploy-all.sh
38+
bash ./examples/oci/deploy-all.sh
4039
else
4140
echo "🔄 Manual run - using deploy-all.sh with clean mode (uninstall/install)"
42-
bash ./deploy-all.sh clean
41+
bash ./examples/oci/deploy-all.sh clean
4342
fi

.github/workflows/ci-docker-hotrod.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,9 @@ jobs:
2424
matrix:
2525
runtime: [docker, k8s]
2626
jaeger-version: [v1, v2]
27+
exclude:
28+
- runtime: k8s
29+
jaeger-version: v1
2730

2831
steps:
2932
- uses: step-security/harden-runner@ec9f2d5744a09debf3a187a3f4f675c53b671911 # v2.13.0
@@ -64,10 +67,6 @@ jobs:
6467
with:
6568
version: 'latest'
6669

67-
- name: Install Kustomize
68-
if: matrix.runtime == 'k8s'
69-
uses: imranismail/setup-kustomize@2ba527d4d055ab63514ba50a99456fc35684947f # v2
70-
7170
- name: Create k8s Kind Cluster
7271
if: matrix.runtime == 'k8s'
7372
uses: helm/kind-action@a1b0e391336a6ee6713a0583f8c6240d70863de3 # v1
Lines changed: 60 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,69 @@
11
# Hot R.O.D. - Rides on Demand on Kubernetes
22

3-
Example k8s manifests for deploying the [hotrod app](..) to your k8s environment of choice. e.g. minikube, k3s, EKS, GKE
3+
Example deployment for the [hotrod app](..) using Helm charts in your Kubernetes environment (Kind, minikube, k3s, EKS, GKE).
4+
5+
## Prerequisites
6+
7+
- Kubernetes cluster
8+
- Helm 3.x installed
9+
- kubectl configured
410

511
## Usage
612

13+
### Deploy with Default Configuration
14+
715
```bash
8-
kustomize build . | kubectl apply -f -
9-
kubectl port-forward -n example-hotrod service/example-hotrod 8080:frontend
10-
# In another terminal
11-
kubectl port-forward -n example-hotrod service/jaeger 16686:frontend
16+
cd examples/oci
17+
./deploy-all.sh clean
18+
```
19+
20+
### Deploy with Local Images (for development)
1221

13-
# To cleanup
14-
kustomize build . | kubectl delete -f -
22+
```bash
23+
# For Jaeger v2 with local images
24+
cd examples/oci
25+
./deploy-all.sh local <image-tag>
1526
```
1627

17-
Access Jaeger UI at <http://localhost:16686> and HotROD app at <http://localhost:8080>
28+
### Deploy Modes
29+
30+
- **`upgrade`** (default): Upgrade existing deployment or install if not present
31+
- **`local`**: Deploy using local registry images (localhost:5000)
32+
- **`clean`**: Clean install (removes existing deployment first)
33+
34+
### Access Applications
35+
36+
After deployment completes, use port-forwarding:
37+
38+
```bash
39+
# Jaeger UI
40+
kubectl port-forward svc/jaeger-query 16686:16686
41+
42+
# HotROD application
43+
kubectl port-forward svc/jaeger-hotrod 8080:80
44+
45+
# Prometheus (optional)
46+
kubectl port-forward svc/prometheus 9090:9090
47+
48+
# Grafana (optional)
49+
kubectl port-forward svc/prometheus-grafana 9091:80
50+
```
51+
52+
Then access:
53+
- 🔍 **Jaeger UI**: http://localhost:16686/jaeger
54+
- 🚕 **HotROD App**: http://localhost:8080/hotrod
55+
- 📈 **Prometheus**: http://localhost:9090
56+
- 📊 **Grafana**: http://localhost:9091
57+
58+
## Configuration
59+
60+
The deployment uses:
61+
- **Helm charts** from [jaeger-helm-charts](https://github.com/jaegertracing/helm-charts)
62+
- **Prometheus** for metrics collection
63+
- **Load generator** for creating sample traces
64+
65+
Configuration files:
66+
- `jaeger-values.yaml` - Jaeger Helm chart values
67+
- `config.yaml` - Jaeger configuration
68+
- `ui-config.json` - Jaeger UI configuration
69+
- `monitoring-values.yaml` - Prometheus configuration

examples/hotrod/kubernetes/base/hotrod/deployment.yaml

Lines changed: 0 additions & 40 deletions
This file was deleted.

examples/hotrod/kubernetes/base/hotrod/kustomization.yaml

Lines changed: 0 additions & 6 deletions
This file was deleted.

examples/hotrod/kubernetes/base/hotrod/service.yaml

Lines changed: 0 additions & 12 deletions
This file was deleted.

examples/hotrod/kubernetes/base/jaeger-all-in-one/kustomization.yaml

Lines changed: 0 additions & 33 deletions
This file was deleted.

examples/hotrod/kubernetes/base/jaeger-all-in-one/service.yaml

Lines changed: 0 additions & 16 deletions
This file was deleted.

examples/hotrod/kubernetes/kustomization.yaml

Lines changed: 0 additions & 13 deletions
This file was deleted.

examples/hotrod/kubernetes/namespace.yaml

Lines changed: 0 additions & 4 deletions
This file was deleted.

0 commit comments

Comments
 (0)