#179 Assigne automatiquement les ips lors de la création d'un abonnement

Ouvert
ljf veut fusionner 14 commits à partir de ARN/enh-quick-creation-remake vers FFDN/master
ljf a commenté il y a 6 ans

Note: chez ARN on a ce commit en plus 5f9851e

Note: chez ARN on a ce commit en plus 5f9851e
jocelyn a commenté il y a 6 ans
Propriétaire

Très cool ça :-).

Review en cours, petites notes d'ores et déjà :

  • Je ne comprends pas trop la logique du to_assign : c'est pour indiquer les blocs qui peuvent être assignés automatiquement ? Dans ce cas pourquoi faire un order_by('-to_assign') et pas un filter(to_assign=True)

  • J'ai l'impression que cette PR en inclut d'autres :x Si c'est volontaire pour des raisons de dépendances, nerci d'indiquer les dépendances de la PR <3

  • au sujet de 5f9851e, pour bien comprendre, pour le futur, la manière propre de faire serait d'avoir un moyen de marquer une IP comme réservée sans l'assigner, c'est ça ?

Très cool ça :-). Review en cours, petites notes d'ores et déjà : - Je ne comprends pas trop la logique du to_assign : c'est pour indiquer les blocs qui peuvent être assignés automatiquement ? Dans ce cas pourquoi faire un order_by('-to_assign') et pas un filter(to_assign=True) - J'ai l'impression que cette PR en inclut d'autres :x Si c'est volontaire pour des raisons de dépendances, nerci d'indiquer les dépendances de la PR <3 - au sujet de 5f9851e, pour bien comprendre, pour le futur, la manière propre de faire serait d'avoir un moyen de marquer une IP comme réservée sans l'assigner, c'est ça ?
ljf a commenté il y a 6 ans
Collaborateur

Cette PR dépend de housing et vps.

Je ne comprends pas trop la logique du to_assign

Il y a 2 choses: l'allocation automatique d'un subnet de taille 'defaul_subnetsize' issue d'un pool d'ip (IPPool). Là on peut vouloir par exemple assigner 4 subnet, par exemple 1 IPV4/32 + un bloc IPv4 /30 + 1 IPv6 /128 + un bloc IPv6 /56

Cependant, ce n'est pas parce qu'on route ces ips, que celles-ci seront configurées comme l'ip par défaut. Typiquement chez ARN sur les VPN on a par VPN 1 IPV4/32 + 1 IPv6 /128 + un bloc IPv6 /56 en sous réseaux, MAIS seul l'IPv4 et l'IPv6/128 sont utilisé pour configurer la connexion par défaut.

Chez d'autres, une Ipv6 est choisie dans le /56 et il n'y a pas d'IPv6 /128 dans un bloc séparé du prefix IPv6...

Du coup, il y a 2 choses, allouer des prefixs à une configuration ET dans ces prefix choisir les ips qui vont être configurées par défaut. Ainsi l'utilisateur peut savoir comment accéder à son VPS/Housing.

J'espère avoir été clair.

au sujet de 5f9851e, pour bien comprendre, pour le futur, la manière propre de faire serait d'avoir un moyen de marquer une IP comme réservée sans l'assigner, c'est ça ?

Oui c'est ça, car nous avions eu besoin de corriger rapidement ça... En effet, notre infra se réserve ces ips. Il ne semblait pas possible de le configurer via coin, d'où ce quick fix sur notre prod.

Cette PR dépend de housing et vps. > Je ne comprends pas trop la logique du to_assign Il y a 2 choses: l'allocation automatique d'un subnet de taille 'defaul_subnetsize' issue d'un pool d'ip (IPPool). Là on peut vouloir par exemple assigner 4 subnet, par exemple 1 IPV4/32 + un bloc IPv4 /30 + 1 IPv6 /128 + un bloc IPv6 /56 Cependant, ce n'est pas parce qu'on route ces ips, que celles-ci seront configurées comme l'ip par défaut. Typiquement chez ARN sur les VPN on a par VPN 1 IPV4/32 + 1 IPv6 /128 + un bloc IPv6 /56 en sous réseaux, MAIS seul l'IPv4 et l'IPv6/128 sont utilisé pour configurer la connexion par défaut. Chez d'autres, une Ipv6 est choisie dans le /56 et il n'y a pas d'IPv6 /128 dans un bloc séparé du prefix IPv6... Du coup, il y a 2 choses, allouer des prefixs à une configuration ET dans ces prefix choisir les ips qui vont être configurées par défaut. Ainsi l'utilisateur peut savoir comment accéder à son VPS/Housing. J'espère avoir été clair. > au sujet de 5f9851e, pour bien comprendre, pour le futur, la manière propre de faire serait d'avoir un moyen de marquer une IP comme réservée sans l'assigner, c'est ça ? Oui c'est ça, car nous avions eu besoin de corriger rapidement ça... En effet, notre infra se réserve ces ips. Il ne semblait pas possible de le configurer via coin, d'où ce quick fix sur notre prod.
jocelyn a commenté il y a 6 ans
Propriétaire

Cette PR en inclut d'autres ; je ne peux pas trop fusionner en l'état ; à nettoyer/rebase, etc… ?

Cette PR en inclut d'autres ; je ne peux pas trop fusionner en l'état ; à nettoyer/rebase, etc… ?
Cette pull request ne peut être fusionnée automatiquement à cause de conflits.
Fusionner manuellement afin de résoudre les conflits.
Connectez-vous pour rejoindre cette conversation.
Aucun jalon
Pas d'assignataire
2 Participants
Chargement…
Annuler
Enregistrer
Il n'existe pas encore de contenu.