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 ?
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.
Note: chez ARN on a ce commit en plus
5f9851e
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 ?Cette PR dépend de housing et vps.
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.
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 en inclut d'autres ; je ne peux pas trop fusionner en l'état ; à nettoyer/rebase, etc… ?