Repository navigation
Fix/docker ci test audit - #1
Merged
Merged
Conversation
…ency pins, test fixtures Core bugs fixed: - app/pipeline/predict.py: version.isdigit() crashed because MLflow 3.13.0 returns ModelVersion.version as int, not str — now casts to str first. This was hitting the real inference server, not just tests. - docker-compose.yml: added missing mlflow/inference-server/metrics-exporter services; fixed anomaly-detector's broken Dockerfile/port wiring. - app/serving/Dockerfile: fixed EXPOSE/healthcheck port mismatch (8000 vs 8006), added curl, added missing mlflow/scipy to requirements.txt. - requirements.txt: removed nonexistent starlette==1.2.0 pin. - pyproject.toml: fixed invalid build-backend, realigned dependency versions with requirements.txt (previously two divergent version sets). - monitoring/prometheus/alert_rules.yml: reconciled with the K8s ConfigMap's alert set (ModelDriftDetected, PredictionConfidenceLow, etc.) — local and K8s environments previously alerted K8s environments previously alerted K8s environments previously alerted K8s environments previously a training-job (pre training-job (pre training-job (pre training-job (pre training-job r fixes (new infra/docker/mlflow/Dockerfile): - mlflow now baked into an image- mlflow now baked into an image- mlflow now baked into anrtup- mlflow now baked into an image- mlflow now baked into an image- mlflow nopo- mlflow now baked into an image- mlflow now baked into an image- huey scheduler. - --allowed-hosts includes mlflow,mlflow:5000 — MLflow's D- --allowed-hosts includes mlflow,mlflow:5000 — MLflow's D- --allnot strip the port. - --artifacts-destination + --serve-artifact- --artifacts-destination + --serve-artifact- --artifacts-destination + --she- --artifacts-destination + --serve-artifact- --artifacts-desties- --artifacts-destination + --serve-artifact- --artifacts-destination + --s r- --artifacts-destination + --serve-artifact- --artifacts-destination + --seba- --artifacts-destination + --serve-artifact- --artifacts-destination + --serve-artifact- --artifacts-destination + --she- --artifacts-destinati0 tests in this file silently skipped in CI. Verified: full docker-compose stack builds and runs end-to-end (mlflow, inference-server, metrics-exporter all healthy), training completes and registers a model, /predict returns valid results, all 86 unit tests pass.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.