Skip to content

21-22-23 Temmuz yapılanlar: API gateway async job akışı, gerçek gRPC … - #26

Closed
ElifFCebeci wants to merge 1 commit into
mainfrom
backend/feature
Closed

ElifFCebeci wants to merge 1 commit into
mainfrom
backend/feature

Conversation

@ElifFCebeci

Copy link
Copy Markdown
Collaborator

Açıklama:

Özet

  • /enhance ve /enhance/batch artık sahte 202 yanıtı yerine gerçek async job akışı çalıştırıyor: Redis'te QUEUED -> RUNNING -> COMPLETED/FAILED, sonuç GET /enhance/{job_id} ile sorgulanıyor.
  • /health ve /models artık gerçek Health/ListModels gRPC çağrılarına bağlı (statik mock kaldırıldı).
  • gRPC client: 4 bağlantılı round-robin pool + sadece UNAVAILABLE için retry/backoff.
  • CORS, Prometheus /metrics, route bazlı upload boyutu sınırı (413), graceful shutdown (SIGINT/SIGTERM'de arka plandaki job'lar bitmeden Redis/gRPC kapanmıyor).
  • api/openapi.yaml üretildi; common.proto'daki JobStatus yorumu async modele göre güncellendi.
  • Go 1.25.12 + quic-go v0.59.1 — govulncheck'teki 17 açığı kapatıyor.

Kapsam dışı

  • services/ml/inference_server (Python gRPC sunucusu) bu PR'a dahil değil, hâlâ yazılmadı — gateway şu an karşı taraf olmadığı için /health, /models, job'lar "unreachable/failed" dönecek, bu beklenen.

Test planı

  • go build ./..., gofmt -l ., go vet ./... temiz
  • Redis + gerçek sunucu ile manuel smoke test: /enhance → 202 → job kaydı, olmayan job için 404, 25MB üstü dosya için 413, /health+/models gRPC yokken doğru hata mesajı, CORS izinli/izinsiz origin
  • Graceful shutdown kod incelemesiyle doğrulandı; Windows/local sandbox'ta gerçek SIGTERM sinyali canlı test edilemedi (Linux/k8s'te sorunsuz çalışması beklenir)

@Enskc05

Enskc05 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the work here — the changes themselves look solid, but I have to request changes before this can be merged.

PR description must be in English. Please rewrite the summary, scope, and test plan sections in English so the whole team can review consistently.
Branch naming convention: please rename/re-create the branch to follow our feature/** naming format (e.g. feature/enhance-async-job-flow) before re-submitting.

Once these are updated, happy to take another pass. Thanks

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.

2 participants