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
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 54
- Forks
- 36
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Abre exercises/Python/08-deploy-agent-to-cf.md y revisa la sección “Update requirements.txt” en las líneas 255-282. Añade el paso de instalación local y señala que es necesario antes de realizar pruebas localmente, mientras que Cloud Foundry instala los requisitos durante el despliegue. Verifica que el ejercicio renderizado distinga claramente entre las pruebas locales y cf push.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- documentation
- Tipo de issue
- Documentación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Estado de actividad
- Tranquilo
- Claridad
- Bien especificado
- Aptitud para principiantes
- 88/100