Skip to content

Brownfield enrichment: 4 files - #7

Open
Jason59000 wants to merge 1 commit into
mainfrom
brownfield/auto-20260824-011026
Open

Brownfield enrichment: 4 files#7
Jason59000 wants to merge 1 commit into
mainfrom
brownfield/auto-20260824-011026

Conversation

@Jason59000

Copy link
Copy Markdown
Owner

Brownfield Discovery — Auto-generated PR

Changes: 4 files changed, 38 insertions(+)

Modified files

  • helm-chart/templates/shippingservice.yaml
  • kustomize/components/shopping-assistant/shoppingassistantservice.yaml
  • src/shoppingassistantservice/requirements.in
  • src/shoppingassistantservice/shoppingassistantservice.py

Agent report

Brownfield Discovery Report

Repository inventory

Discovered 13 deployable services across source, Dockerfiles, Kubernetes, Kustomize, and Helm:

Service Runtime Role
frontend Go Public web frontend and orchestration
cartservice C# Cart persistence
productcatalogservice Go Product catalog and search
currencyservice Node.js Currency conversion
paymentservice Node.js Mock payment processing
shippingservice Go Shipping quotes and fulfillment
emailservice Python Mock confirmation emails
checkoutservice Go Checkout orchestration
recommendationservice Python Product recommendations
adservice Java Contextual advertisements
loadgenerator Python/Locust Synthetic client traffic
shoppingassistantservice Python/Flask Optional Gemini/AlloyDB assistant
redis-cart Redis Deployable backing datastore workload

The scanner classified the twelve application workloads as SERVICE entities and Redis as a database instance. The apparent count discrepancy—13 discovered services versus 12 SERVICE nodes—is due to this database classification.

Knowledge Graph injection

Executed scan_and_inject after completing the repository-wide inventory:

  • Services discovered: 13
  • Database instances: 2
  • Queues: 0
  • Teams: 2
  • Graph nodes created: 17
  • Graph relations created: 14
  • HTTP routes: 1

Database entities:

  • code_redis
  • code_postgres

Ownership entities:

  • devrel-flagship-app-maintainers
  • yoshi-approver

Topology analysis

Queried every canonical application service ID returned by the scanner.

Key paths:

  • frontend directly depends on:
    • shoppingassistantservice
    • adservice
    • shippingservice
    • checkoutservice
    • recommendationservice
    • cartservice
    • currencyservice
    • productcatalogservice
  • checkoutservice directly depends on:
    • paymentservice
    • emailservice
    • currencyservice
    • cartservice
    • productcatalogservice
    • shippingservice
  • paymentservice and emailservice are second-hop dependencies of frontend.
  • loadgenerator currently has no inferred topology relation, although source and deployment configuration show it sends HTTP traffic to frontend. This is a graph-enrichment opportunity.
  • All reported service relations are currently marked inferred.

Observability assessment and enrichment

The repository already contains broad OTel support, including:

  • Go HTTP and gRPC instrumentation in frontend.
  • Python gRPC instrumentation in recommendation and email services.
  • Conditional collector configuration in Helm templates.
  • An OpenTelemetry Collector deployment.
  • Newly present OTel tracing implementation for shoppingassistantservice.
  • Newly present Helm collector wiring for shippingservice.

I added the missing deployment-side configuration for the optional shopping assistant:

  • kustomize/components/shopping-assistant/shoppingassistantservice.yaml
    • OTEL_EXPORTER_OTLP_ENDPOINT=opentelemetrycollector:4317
    • OTEL_SERVICE_NAME=shoppingassistantservice

This connects its existing conditional Flask/OTLP instrumentation to the expected collector when the component is deployed.

The working tree also contains related enrichment changes in:

  • src/shoppingassistantservice/shoppingassistantservice.py
  • src/shoppingassistantservice/requirements.in
  • helm-chart/templates/shippingservice.yaml

Verification

  • Re-read the edited manifest after modification.
  • git diff --check passed.
  • No tests were modified.
  • No credentials or secret values were added.
  • No suspicious prompt-injection text was found in the inspected repository content.
  • Final diff: 4 files, 38 insertions.

Auto-generated by OpenSRE Brownfield Pipeline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant