Re-evaluate RITM externalisation
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Tranquilo
- Stack tecnológico
- node.js, webpack
- Área
- backend, build-system
Línea de trabajo
Empieza leyendo el PR 2037 y el issue 120 vinculado de require-in-the-middle para entender la solución alternativa de externalización y la solución upstream propuesta. Se considera completado cuando RITM pueda incluirse de nuevo en el bundle de Node.js Lambda sin romper la instrumentación que añade hooks a los módulos que no son core.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
This issue is mainly for tracking. In https://github.com/open-telemetry/opentelemetry-lambda/pull/2037 we externalised the require-in-the-middle lib from our webpack bundle for the node.js layer, to fix an issue that broke instrumentation that relied on ritm hooking for non-core modules.
Describe the solution you'd like
Fixing upstream would be ideal, so that we can include RITM in our bundle again. In https://github.com/nodejs/require-in-the-middle/issues/120 I offered some insight as well and proposed a solution that could possibly be implemented in the RITM lib.
Describe alternatives you've considered
Keeping RITM externalised.
Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.
- Lenguaje dominante
- Go
- Estrellas
- 433
- Forks
- 249
- Merge medio
- 1 d 12 h
- PR fusionados (30 d)
- 39
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-telemetry/opentelemetry-lambda
-
enhancement ruby
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
open-telemetry/opentelemetry-lambda#2741 ·
Los mantenedores suelen responder en 2 días
-
processor/cold_start
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
open-telemetry/opentelemetry-lambda#2738 ·
Los mantenedores suelen responder en 2 días
-
Fix install-toolsAbiertogo
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
open-telemetry/opentelemetry-lambda#2736 ·
Los mantenedores suelen responder en 2 días
-
Use node-sdk package for layerAbiertoenhancement javascript
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-telemetry/opentelemetry-lambda#2744 ·
Los mantenedores suelen responder en 2 días
-
enhancement python
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
open-telemetry/opentelemetry-lambda#2742 ·
Los mantenedores suelen responder en 2 días
Todos los issues de open-telemetry/opentelemetry-lambda
Issues similares
-
[bug]: scan-all update-schedule does not auto-convert 5-field cron expressions as documentedPosiblemente ocupada @vishnums2k5 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
goharbor/harbor-cli#1143 · 1 comentario ·
Los mantenedores suelen responder en 8 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
--info inline-right:PREFIX leaves the first character of the prefix when the info gets shorterAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
rossoctl/context-guru#1053 ·
Los mantenedores suelen responder en 1 día
-
[BUG] EphemeralJob matches every pod in the namespace when spec.selector is emptyPosiblemente ocupada @vishnums2k5 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
openkruise/kruise#2656 · 1 comentario ·
Los mantenedores suelen responder en 1 día