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?
36d9f60a4fto649049ba36b63a6559adto175bf15ff3175bf15ff3tod1a7e47ca6d1a7e47ca6toafb18de939afb18de939tobf0b4e8457bf0b4e8457to337541cc9c337541cc9ctofbf9d4d0fbfbf9d4d0fbtoe54176c9b7e54176c9b7tof40513ef95f40513ef95to8327c1c90f8327c1c90fto87a1ab433587a1ab4335toe0fbf25d24e0fbf25d24to641dd2d6321de021e3b1toe2a43981b8e2a43981b8to00d47bf12469ceeef171toe07fbd3d36a882f0ce10toeda12e5265d4079d1f10tod1664ed265d1664ed265to3c939538243c93953824to3763be35383763be3538to4d022e9e8f4d022e9e8ftod12d254b3dd12d254b3dto339b0ae0e9ecc92e35b8to1781141307d9e0dc04d5toce443b2f93ce443b2f93tof939a73427f939a73427to2d5880dc482d5880dc48to10ce7ee48410ce7ee484to249e94aabb249e94aabbtod3989a24d6d3989a24d6to4bf00acd02e15e744e68to603aef94f6603aef94f6toef3edf5830ef3edf5830tod6f751ff1cd6f751ff1cto237bbd697a237bbd697ato5408e5448b5408e5448bto0a4e65df2b0a4e65df2btocd14f00614cd14f00614to83224b5fa683224b5fa6to59905a572f59905a572ftodb4c6ed698d3d724ea35to1502e509c81502e509c8tod14044ef13d14044ef13to8b799f1e998b799f1e99to4899170c094899170c09tofe6b5bdb08fe6b5bdb08toa57347c39aa57347c39ato7e256cb6ac35d716baectodd8684be91dd8684be91to85b96d0194669c2e41a0tod8811d900f9e3f846f51to4dfe0d8b5ec6a291c8d2toa40025a99fa40025a99fto48b6b629d648b6b629d6to7663017fcb98dce354f1to1301e0aea51301e0aea5to16dc991df7445193e171to599e888946599e888946tocc8a360280cc8a360280to91f6dca6b63584f3a500to22e7c1aefb34457a561dtodaaf1eab79daaf1eab79toe7c808176ee7c808176eto872185aeff872185aefftoce4ed4149931af59589ftoa330935471a38a5e41fdtodbd99d4a9c86581d522atofd4dfafc0693a058287atoe959605e41e959605e41to9bd22f1f6e9bd22f1f6eto6289843d106289843d10toda694bfb52da694bfb52to3d2b21ea643d2b21ea64tob71a5ac2c0b71a5ac2c0to8e785c49a031e488ff78to28193ed3ef28193ed3efto2bff24ad7c2bff24ad7ctod68c2ed8a9d68c2ed8a9to4d9c6af1556e5dacc4f8to92da15af30f25f01b6d5to0ec722191e0ec722191eto9f01acc08c0dd0e3b462toeb99f8e4aeeb99f8e4aetoe9c986ab6ce9c986ab6cto9aaea82658885ef285c7to2b1ddd81ee2b1ddd81eeto382c7205a5382c7205a5todf6b80b73676e0724cd5to19095d58f319095d58f3to90ff3bf5dd90ff3bf5ddto9c5e0497cc9c5e0497ccto8f714d71db8f714d71dbtoa6205cc2cecebb21d2ddtob5aab24043b5aab24043to43772938af43772938aftoe4c85bfd51e4c85bfd51tof483644532f483644532to6ffec671886ffec67188to3e0198d20aWIP: feat(isp): changes of switch for next stepto feat(isp): changes of switch for next step3e0198d20ato265c98060c@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
6f30f82d62to15e436f533b1d5c04c4ctocf3b310c01cf3b310c01toea9c53e5f1ea9c53e5f1to4bc6baf5614bc6baf561tod70ec99e6f@ -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
d70ec99e6ftoab25a632ee