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

Ouverte Adaptée aux débutants
#38 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
Python
Étoiles
54
Forks
36
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Ouvrez exercises/Python/08-deploy-agent-to-cf.md et examinez la section « Update requirements.txt » aux lignes 255-282. Ajoutez l’étape d’installation locale et précisez qu’elle est nécessaire avant les tests en local, tandis que Cloud Foundry installe les dépendances lors du déploiement. Vérifiez que l’exercice rendu distingue clairement les tests en local de cf push.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
documentation
Type d'issue
Documentation
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
Calme
Clarté
Clairement spécifiée
Accessibilité débutants
88/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.