test(flagd): drop the per-scenario /stop in e2e teardown once the testbed includes flagd-testbed#396
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Empieza en providers/flagd/src/test/java/.../e2e/steps/ProviderSteps.java y lee ProviderSteps.tearDown para encontrar el bloque POST de /stop, además de ContainerPool.release y el intercambio del proveedor NoOp, que debe conservarse. Confirma primero que la dependencia flagd-testbed se ha actualizado a una versión que incluye open-feature/flagd-testbed#396 (comprueba la versión de testbed en los archivos de build del proveedor de flagd), ya que eliminar /stop antes de eso no aporta nada. Listo significa que el bloque /stop ya no está en tearDown mientras ContainerPool.release y el intercambio del proveedor permanecen, y que toda la suite e2e (in-process, RPC, proveedores de archivos) sigue pasando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
ProviderSteps.tearDown (providers/flagd/src/test/java/.../e2e/steps/ProviderSteps.java) POSTs /stop to the launchpad after every scenario. That makes every /start a cold flagd start.
open-feature/flagd-testbed#396 makes /start reuse the running flagd and restore the baseline flag state in place. Isolation no longer needs the /stop, and keeping it means the suite never gets the speedup.
Proposal
After bumping the testbed to a release that includes open-feature/flagd-testbed#396, remove the /stop block from tearDown. Keep ContainerPool.release and the NoOp provider swap.
Evidence
Tested locally with the /stop removed, against an image built from flagd-testbed#396 and one built from testbed main. All 590 tests (in-process 224, RPC 214, file 152) passed in every run. Wall time (in-process / RPC / file):
with /stop |
without /stop |
|
|---|---|---|
| flagd-testbed#396 | 102-107s / 73-79s / 48-65s | 96s / 66s / 41s |
| testbed main | 100s / 75s / 50s | 120s / 79s / 49s |
The machine was shared with other test runs, so treat the times as indicative. Without flagd-testbed#396, dropping /stop gains nothing, which is why this should wait for the bump.
- Lenguaje dominante
- Java
- Estrellas
- 50
- Forks
- 87
- Merge medio
- 1 d 5 h
- PR fusionados (30 d)
- 51
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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.
Más de open-feature/java-sdk-contrib
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
open-feature/java-sdk-contrib#1861 ·
Los mantenedores suelen responder en 1 día
-
fix!: opentelemetry-semconv-incubating bump renames feature_flag.provider_name attributePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
open-feature/java-sdk-contrib#1843 ·
Los mantenedores suelen responder en 1 día
-
[ConfigCat] Evaluation errors are returned as a successful default with no error codePosiblemente ocupada @RaphaelFakhri la tomó hace 5 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
open-feature/java-sdk-contrib#1963 ·
Los mantenedores suelen responder en 1 día
-
[go-feature-flag] Should remote evaluation emit events and fail init on an unreachable relay proxy?Posiblemente ocupada @thomaspoignant la tomó hace 19 días. Abierto
open-feature/java-sdk-contrib#1948 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 53/100
open-feature/java-sdk-contrib#1924 ·
Los mantenedores suelen responder en 1 día
Todos los issues de open-feature/java-sdk-contrib
Issues similares
-
hermes-queue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
chrisalunlloyd2-sudo/mindpalace#349 ·
Los mantenedores suelen responder en 1 día
-
Retry-After parser accepts invalid delay-seconds valuesPosiblemente ocupada @Dhruvraj-Rana la tomó hoy. AbiertoBug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
open-telemetry/opentelemetry-java#9018 ·
Los mantenedores suelen responder en 2 días
-
bug priority: medium Review effort [1-5]: 2 status: needs-triage type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
carlos-emr/carlos#5818 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue help wanted tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
appmonster007/kmip-adapter#100 ·
-
[Documentation]: Remove non-existent CHANGELOG reference from StateBackendProperties JavadocAbiertoaccepted good first issue java
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día