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

Open
ljf wants to merge 14 commits from ARN/enh-quick-creation-remake into FFDN/master
ljf commented 6 years ago

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

Note: chez ARN on a ce commit en plus 5f9851e
jocelyn commented 6 years ago
Owner

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 commented 6 years ago
Collaborator

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 commented 6 years ago
Owner

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… ?
This pull request can't be merged automatically because there are conflicts.
Please merge manually in order to resolve the conflicts.
Sign in to join this conversation.
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.