SAP-samples / SAP-samples/codejam-code-based-agents
Python track: exercise 08 doesn't mention re-running pip install after updating requirements.txt for local testing
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 54
- Fork
- 36
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
Summary
The "Update requirements.txt" section (08-deploy-agent-to-cf.md:255-282) instructs learners to add crewai[a2a] and a2a-sdk[http-server] to requirements.txt, then moves straight into creating manifest.yml for Cloud Foundry deployment. There's no pip install -r requirements.txt (or equivalent) command shown anywhere in the exercise.
Effect
This works fine for the actual CF deployment, since CF's Python buildpack automatically runs pip install -r requirements.txt during cf push — the deployed app gets the new dependencies without any local action needed.
However, if a learner wants to test the A2A server locally before deploying (a reasonable thing to want to do, and not explicitly discouraged anywhere), editing requirements.txt alone doesn't install anything into their existing venv. Running the server locally at that point fails with ModuleNotFoundError: No module named 'a2a', since the new packages were never actually installed — only added to a text file.
Suggested fix
Add an explicit step after the requirements.txt update, e.g.:
bash pip install -r requirements.txt
with a short note that this is needed if testing locally before cf push (even though CF's buildpack handles it automatically during deployment).
Environment
Reproduced on a personal SAP AI Core tenant, Python (CrewAI + LiteLLM) track, testing the A2A server locally before Cloud Foundry deployment.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Apri exercises/Python/08-deploy-agent-to-cf.md e rivedi la sezione “Update requirements.txt” alle righe 255-282. Aggiungi il passaggio per l’installazione locale e specifica che è necessario prima di eseguire i test localmente, mentre Cloud Foundry installa i requisiti durante il deployment. Verifica che l’esercizio renderizzato distingua chiaramente tra i test locali e cf push.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- documentation
- Tipo di issue
- Documentazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Stato di attività
- Tranquilla
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 88/100