Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 0 additions & 21 deletions .github/workflows/ci-e2e-all.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,30 +16,9 @@ permissions:
contents: read

jobs:
badger:
uses: ./.github/workflows/ci-e2e-badger.yaml

cassandra:
uses: ./.github/workflows/ci-e2e-cassandra.yml

elasticsearch:
uses: ./.github/workflows/ci-e2e-elasticsearch.yml

grpc:
uses: ./.github/workflows/ci-e2e-grpc.yml

kafka:
uses: ./.github/workflows/ci-e2e-kafka.yml

memory:
uses: ./.github/workflows/ci-e2e-memory.yaml

opensearch:
uses: ./.github/workflows/ci-e2e-opensearch.yml

query:
uses: ./.github/workflows/ci-e2e-query.yml

upload_pr_number:
name: Save and Upload PR Number as Artifact
runs-on: ubuntu-latest
Expand Down
9 changes: 0 additions & 9 deletions .github/workflows/ci-e2e-elasticsearch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,15 +18,6 @@ jobs:
fail-fast: false
matrix:
version:
- major: 6.x
distribution: elasticsearch
jaeger: v1
- major: 7.x
distribution: elasticsearch
jaeger: v1
- major: 8.x
distribution: elasticsearch
jaeger: v1
- major: 8.x
distribution: elasticsearch
jaeger: v2
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,11 +49,11 @@ func (q *queryMetrics) emit(err error, latency time.Duration, responses int) {
func NewReaderDecorator(traceReader tracestore.Reader, metricsFactory metrics.Factory) *ReadMetricsDecorator {
return &ReadMetricsDecorator{
traceReader: traceReader,
findTracesMetrics: buildQueryMetrics("find_traces", metricsFactory),
findTraceIDsMetrics: buildQueryMetrics("find_trace_ids", metricsFactory),
getTraceMetrics: buildQueryMetrics("get_trace", metricsFactory),
getServicesMetrics: buildQueryMetrics("get_services", metricsFactory),
getOperationsMetrics: buildQueryMetrics("get_operations", metricsFactory),
findTracesMetrics: buildQueryMetrics("error", metricsFactory),
findTraceIDsMetrics: buildQueryMetrics("error", metricsFactory),
getTraceMetrics: buildQueryMetrics("error", metricsFactory),
getServicesMetrics: buildQueryMetrics("error", metricsFactory),
getOperationsMetrics: buildQueryMetrics("error", metricsFactory),
}
}

Expand Down
26 changes: 21 additions & 5 deletions scripts/e2e/metrics_summary.sh
Original file line number Diff line number Diff line change
Expand Up @@ -47,22 +47,38 @@ if $DIFF_FOUND; then
echo "Metric differences detected"
echo "DIFF_FOUND=true" >> "$GITHUB_OUTPUT"

# Calculate total changes across all files
total_changes=0
for summary_file in "${summary_files[@]}"; do
changes=$(grep -F "**Total Changes:**" "$summary_file" | awk '{print $3}')
total_changes=$((total_changes + changes))
done

# Combine all summaries into one
echo "## Metrics Comparison Summary" > "$METRICS_DIR/combined_summary.md"
echo "" >> "$METRICS_DIR/combined_summary.md"
combined_file="$METRICS_DIR/combined_summary.md"
echo "## Metrics Comparison Summary" > "$combined_file"
{
echo ""
echo "Total changes across all snapshots: $total_changes"
echo ""
echo "<details>"
echo "<summary>Detailed changes per snapshot</summary>"
echo ""
} >> "$combined_file"

if [ ${#summary_files[@]} -gt 0 ]; then
for summary_file in "${summary_files[@]}"; do
echo "Appending $summary_file to combined summary"
{
echo "### $(basename "$summary_file" .md)"
cat "$summary_file"
} >> "$METRICS_DIR/combined_summary.md"
echo "" >> "$METRICS_DIR/combined_summary.md"
} >> "$combined_file"
echo "" >> "$combined_file"
done
fi

echo -e "\n\n➡️ [View full metrics file]($LINK_TO_ARTIFACT)" >> "$METRICS_DIR/combined_summary.md"
echo "</details>" >> "$combined_file"
echo -e "\n\n➡️ [View full metrics file]($LINK_TO_ARTIFACT)" >> "$combined_file"
else
echo "No metric differences detected"
fi
Expand Down
Loading