Openstack provider clean-up

Abierto
#691 3 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
35/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
go
Área
backend, cloud

Línea de trabajo

Comienza en pkg/cloudprovider/provider/openstack/helper.go, en la búsqueda de red alrededor de la línea 217, la búsqueda de grupos de seguridad alrededor de la línea 184 y la lógica de la subred predeterminada alrededor de la línea 416. Rastrea el uso de la OpenStack API y el razonamiento detrás del comportamiento de la subred. Se considera terminado cuando las rutas de búsqueda son claras y eficientes, y el comportamiento de la subred predeterminada está justificado en comentarios o se ha hecho coherente con el comportamiento previsto del provider.

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

Descripción

kind/cleanup priority/low sig/cluster-management

This issue is to do some clean-up in the openstack provider logic, that in some cases is not very clear or optimal. A few examples:

In order to get a network by name or ID we iterate through all networks instead of getting it directly with the proper API call:
https://github.com/kubermatic/machine-controller/blob/master/pkg/cloudprovider/provider/openstack/helper.go#L217

Same applies for security groups:
https://github.com/kubermatic/machine-controller/blob/master/pkg/cloudprovider/provider/openstack/helper.go#L184

The logic we use to get the default subset is a bit obscure: if we have just one subnet in the network provided in input we return it otherwise we obtain the subnets from the API and we return the first if any or we throw an error. If there is a reason behind that, I have some doubts, we should at least comment it properly.
https://github.com/kubermatic/machine-controller/blob/master/pkg/cloudprovider/provider/openstack/helper.go#L416

Lenguaje dominante
Go
Estrellas
325
Forks
140
Merge medio
4 h 3 min
PR fusionados (30 d)
13

Preparar el entorno

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.

Más de kubermatic/machine-controller

Todos los issues de kubermatic/machine-controller

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.