[BUG] Restarting a run leaves GcpInstance, so CI never re-queues it
Les mainteneurs répondent en général sous 2 jours
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 30/100
Piste de recherche
Read the restart_run API endpoint and the restart_test UI path — both clear TestProgress/results but leave the GcpInstance row behind. Then look at the gcp_instance / start_test logic that skips any test already holding a GcpInstance, and confirm that is why cron never re-queues after a restart. The fix is deleting GcpInstance rows for the test_id in both restart paths, with regression tests covering restart on API and UI when an instance row exists. Note an open linked pull request (#1225) already addresses this, so coordinate before starting.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
restart_run (API) and restart_test (UI) clear TestProgress / results so the run looks pending again. They do not delete the GcpInstance row.
gcp_instance / start_test treat any test with a GcpInstance as already running and skip it. After restarting a running or stuck run, cron therefore never starts a new VM.
Expected
Restart clears the instance reservation so the same test id can be picked up again. The old VM, if still present, becomes orphaned and can be cleaned up by the existing orphan scanner.
Actual
The GcpInstance row stays. start_test logs that the test already has a GCP instance and skips starting it.
Fix
Delete GcpInstance rows for that test_id in both restart paths.
Tests
Regression coverage on API and UI restart when an instance row exists.
- Langage dominant
- Python
- Étoiles
- 30
- Forks
- 90
- Merge moyen
- 4 j 18 h
- PR mergées (30 j)
- 21
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de CCExtractor/sample-platform
-
[BUG] restart_test and stop_test raise 500 / 403 when test_id does not existPeut-être pris @youknowme19 l’a pris il y a 7 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
CCExtractor/sample-platform#1248 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
[BUG] Missing timeouts on 5 outbound HTTP calls outside mod_ci (follow-up to #943)Peut-être pris @devnishantt l’a pris il y a 34 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 84/100
CCExtractor/sample-platform#1192 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
[Feature Request] Check Console DiffPeut-être pris @Anurag-S1ngh l’a pris il y a 2 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 42/100
CCExtractor/sample-platform#1082 · 3 commentaires ·
Les mainteneurs répondent en général sous 2 jours
-
[Feature Request] Improve comparison algorithm to handle frame offsets intelligently (GSoC Qualification)Peut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
CCExtractor/sample-platform#1079 ·
Les mainteneurs répondent en général sous 2 jours
-
[BUG] never-worked baseline handling is not fully integrated into trusted CI reportingPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 42/100
CCExtractor/sample-platform#1057 ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de CCExtractor/sample-platform
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
huggingface/smolagents#3106 ·
Les mainteneurs répondent en général sous 2 jours
-
area:scripts area:tests bug priority:later
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Ikalus1988/MisakaNet#4358 ·
Les mainteneurs répondent en général sous 1 jour
-
Discovered ports from service discovery are never propagated to modules in Module.load()Peut-être pris @vishalmore90 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
OWASP/Nettacker#1828 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
aws-samples/sample-per-model-bedrock#708 ·
Les mainteneurs répondent en général sous 1 jour
-
maintenance
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100