feat(isp): changes of switch for next step #517
No reviewers
Labels
No labels
awaiting
awaiting-author
awaiting
awaiting-reviewer
bot
deployed
Silenced alert
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
DGNum/infrastructure!517
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "isp-switch-change"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
36d9f60a4f649049ba36b63a6559ad175bf15ff3175bf15ff3d1a7e47ca6d1a7e47ca6afb18de939afb18de939bf0b4e8457bf0b4e8457337541cc9c337541cc9cfbf9d4d0fbfbf9d4d0fbe54176c9b7e54176c9b7f40513ef95f40513ef958327c1c90f8327c1c90f87a1ab433587a1ab4335e0fbf25d24e0fbf25d24641dd2d6321de021e3b1e2a43981b8e2a43981b800d47bf12469ceeef171e07fbd3d36a882f0ce10eda12e5265d4079d1f10d1664ed265d1664ed2653c939538243c939538243763be35383763be35384d022e9e8f4d022e9e8fd12d254b3dd12d254b3d339b0ae0e9ecc92e35b81781141307d9e0dc04d5ce443b2f93ce443b2f93f939a73427f939a734272d5880dc482d5880dc4810ce7ee48410ce7ee484249e94aabb249e94aabbd3989a24d6d3989a24d64bf00acd02e15e744e68603aef94f6603aef94f6ef3edf5830ef3edf5830d6f751ff1cd6f751ff1c237bbd697a237bbd697a5408e5448b5408e5448b0a4e65df2b0a4e65df2bcd14f00614cd14f0061483224b5fa683224b5fa659905a572f59905a572fdb4c6ed698d3d724ea351502e509c81502e509c8d14044ef13d14044ef138b799f1e998b799f1e994899170c094899170c09fe6b5bdb08fe6b5bdb08a57347c39aa57347c39a7e256cb6ac35d716baecdd8684be91dd8684be9185b96d0194669c2e41a0d8811d900f9e3f846f514dfe0d8b5ec6a291c8d2a40025a99fa40025a99f48b6b629d648b6b629d67663017fcb98dce354f11301e0aea51301e0aea516dc991df7445193e171599e888946599e888946cc8a360280cc8a36028091f6dca6b63584f3a50022e7c1aefb34457a561ddaaf1eab79daaf1eab79e7c808176ee7c808176e872185aeff872185aeffce4ed4149931af59589fa330935471a38a5e41fddbd99d4a9c86581d522afd4dfafc0693a058287ae959605e41e959605e419bd22f1f6e9bd22f1f6e6289843d106289843d10da694bfb52da694bfb523d2b21ea643d2b21ea64b71a5ac2c0b71a5ac2c08e785c49a031e488ff7828193ed3ef28193ed3ef2bff24ad7c2bff24ad7cd68c2ed8a9d68c2ed8a94d9c6af1556e5dacc4f892da15af30f25f01b6d50ec722191e0ec722191e9f01acc08c0dd0e3b462eb99f8e4aeeb99f8e4aee9c986ab6ce9c986ab6c9aaea82658885ef285c72b1ddd81ee2b1ddd81ee382c7205a5382c7205a5df6b80b73676e0724cd519095d58f319095d58f390ff3bf5dd90ff3bf5dd9c5e0497cc9c5e0497cc8f714d71db8f714d71dba6205cc2cecebb21d2ddb5aab24043b5aab2404343772938af43772938afe4c85bfd51e4c85bfd51f483644532f4836445326ffec671886ffec671883e0198d20aWIP: feat(isp): changes of switch for next stepto feat(isp): changes of switch for next step3e0198d20a265c98060c@lbailly tu pourrais marquer les json qu'il faut comme générés ?
en vrai, il y a que les aps.json qui sont généré, thurnes.json et MR-distribution.json on été généré puis adapté à la main, H1-map.json c'est mes petites main
Note: ce serait cool de factoriser la config des switchs Paccess comme tu as fait pour MR, mais ça attendra une future PR
@ -16,0 +67,4 @@${concatMapAttrsStringSep "\n" (gateway: cfg: ''<qualified-next-hop><name>${gateway}</name>${optionalString (cfg.preference != null) "<preference>${toString cfg.preference}</preference>"}nit: pour factoriser cette opération que tu fais souvent, on pourrait imaginer:
Ce qui donnerait
optionalXML cfg "preference", et avantoptionalXML route "next-hop"@ -57,0 +59,4 @@];description = "Login policy to use for root.";};allow-tcp-forwarding = mkOption {style:
@ -57,0 +66,4 @@Allow forwarding TCP connections via SSH.'';};sftp-server = mkOption {style: Pareil qu'au dessus:
@ -88,0 +90,4 @@type = listOf str;default = [ ];description = ''Interface name for this VLANtypo: il manque le
.à la fin de la description@ -68,0 +47,4 @@(_: patch:!elem patch ["016"Tu peux mettre un commentaire pour indiquer en quoi ces ports sont spéciaux ?
euh, bah juste ils existe pas ?
@ -0,0 +19,4 @@dgn-hardware.model = "EX4100-F-48P";dgn-isp = {enable = true;admin-ip = "${nodeMeta.deployment.targetHost}/64";Est-ce que ça vaudrait pas le coup de set
admin-ipà cette valeur par défaut dans la définition de l'option ? Vu que tu met ça partout@ -9,0 +19,4 @@genFuse (i: singleAttr "${fixedWidthNumber 3 (i + 1)}" "ge-0/0/${toString i}") 12// genFuse (i: singleAttr "${fixedWidthNumber 3 (i + 101)}" "ge-0/0/${toString (i + 16)}") 16// genFuse (i: singleAttr "${fixedWidthNumber 3 (i + 201)}" "ge-0/0/${toString (i + 32)}") 16) [ "009" ];Tu peux rajouter un commentaire ici aussi pour dire ce que 009 a de spécial ?
@ -0,0 +38,4 @@room-connections = listToAttrs (imap0 (idx: nameValuePair "ge-0/0/${toString idx}") ["A102"merci la compacité du formattage@ -0,0 +225,4 @@'';};this-site = mkOption {nit: On pourrait juste mettre
siteimo@ -0,0 +275,4 @@};};monitoring = {nit: On pourra split cette partie du module dans un fichier
monitoring.nix, à faire dans une future PR@ -0,0 +366,4 @@networks = listToAttrs (map mkUserBrNet userVlans);netdevs = listToAttrs (map mkUserBrDev userVlans);};services = {nit: Pareil, le module pour thurned pourra être mis dans un autre fichier (idéalement dans le module du repo thurned pour de la simplicité)
faut qu'on finisse DGNum/thurned#3 ouai
@ -0,0 +16,4 @@domain = "radius.dgnum.eu";radiusClients = {ap = {FIXME: il faudra contrôler les adresses des APs un jour
@ -0,0 +12,4 @@;cfg = config.dgn-isp;mkVlanNetwork =@ -0,0 +21,4 @@nameValuePair "10-${name}" (network // { inherit name; });mkVlanNetdev =name:@ -0,0 +58,4 @@fi'';};# environment.defaultPackages = [Ça se fait comment du coup la gestion des traces sans tes scripts ?
les scripts sont plus vraiment valide, donc si on a besoin de sortir des traces pour le moment c'est chiant, mais elles sont bien généré.
New commits pushed, approval review dismissed automatically according to repository settings
@ -12,2 +12,4 @@optionalString;inherit (lib.extra)optionalXmlDans la lib t'as mis
optionalXMLavec les majusculesohh well
6f30f82d6215e436f533b1d5c04c4ccf3b310c01cf3b310c01ea9c53e5f1ea9c53e5f14bc6baf5614bc6baf561d70ec99e6f@ -141,2 +141,4 @@netconf = {specialArgs = {nodeMeta.deployment.targetHost = ''''${nodeMeta.deployement.targetHost}'';Pas besoin de mettre un specialArg en utilisant
defaultTextc.f. le comment dans dgn-ispje me disait bien que c'était moche, j'y avait pas repenser :(
@ -58,2 +61,3 @@admin-ip = mkOption {type = str;type = nullOr str;default = "${nodeMeta.deployment.targetHost}/64";Il faut rajouter
d70ec99e6fab25a632ee