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

Abierto Apto para principiantes
#38 0 comentarios 0 reacciones 0 asignados Ver en GitHub

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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.