localstack / localstack/localstack-dagger-module

Accommodate comma-separated config values

Abierto
#4 0 comentarios 1 reacción 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Python
Estrellas
3
Forks
0
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

There's no way to provide comma-separated config values through the configuration argument.

For example, I want to specify a list of services via the SERVICES config variable (e.g., SERVICES=s3,ec2,secretsmanager,ecs,iam,sts,fis). However, the configuration parsing code naively splits the provided string on commas:
https://github.com/localstack/localstack-dagger-module/blob/6e2683e20e54e7ceab130ebe2abed050b6670ed2/src/localstack/main.py#L47-L52

As such, when specifying configuration as follows, ec2 is recognized as a separate config variable rather than another service in the list of services that comprise the value for the SERVICES config variable:

dag.Localstack().Start(dag.LocalstackStartOpts{
  Configuration: "SERVICES=s3,ec2,secretsmanager,ecs,iam,sts,fis"
})

There are a few solutions to this:

  1. use a non-comma delimiter to distinguish configuration key-value pairs (e.g., use ; as the delimiter: "SERVICES=s3,ec2,secretsmanager;AWS_DEFAULT_REGION=east-us-1")
  2. accommodate escaped quotes (e.g., "SERVICES=\"s3,ec2,secretsmanager\",AWS_DEFAULT_REGION=east-us-1")

Guía de contribución

Abrir la guía de contribución

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

Comienza leyendo el código de análisis de configuración en src/localstack/main.py en las líneas 47-52 y sigue cómo se divide el argumento de configuración. Compara los enfoques propuestos para el delimitador y el escape, y después comprueba las pruebas o los comandos de prueba existentes del repositorio. Se considera terminado cuando los valores separados por comas siguen formando parte de un único valor de configuración y las entradas de configuración independientes se siguen analizando correctamente.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
devops
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
38/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.