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?
36d9f60a4f
to649049ba36
b63a6559ad
to175bf15ff3
175bf15ff3
tod1a7e47ca6
d1a7e47ca6
toafb18de939
afb18de939
tobf0b4e8457
bf0b4e8457
to337541cc9c
337541cc9c
tofbf9d4d0fb
fbf9d4d0fb
toe54176c9b7
e54176c9b7
tof40513ef95
f40513ef95
to8327c1c90f
8327c1c90f
to87a1ab4335
87a1ab4335
toe0fbf25d24
e0fbf25d24
to641dd2d632
1de021e3b1
toe2a43981b8
e2a43981b8
to00d47bf124
69ceeef171
toe07fbd3d36
a882f0ce10
toeda12e5265
d4079d1f10
tod1664ed265
d1664ed265
to3c93953824
3c93953824
to3763be3538
3763be3538
to4d022e9e8f
4d022e9e8f
tod12d254b3d
d12d254b3d
to339b0ae0e9
ecc92e35b8
to1781141307
d9e0dc04d5
toce443b2f93
ce443b2f93
tof939a73427
f939a73427
to2d5880dc48
2d5880dc48
to10ce7ee484
10ce7ee484
to249e94aabb
249e94aabb
tod3989a24d6
d3989a24d6
to4bf00acd02
e15e744e68
to603aef94f6
603aef94f6
toef3edf5830
ef3edf5830
tod6f751ff1c
d6f751ff1c
to237bbd697a
237bbd697a
to5408e5448b
5408e5448b
to0a4e65df2b
0a4e65df2b
tocd14f00614
cd14f00614
to83224b5fa6
83224b5fa6
to59905a572f
59905a572f
todb4c6ed698
d3d724ea35
to1502e509c8
1502e509c8
tod14044ef13
d14044ef13
to8b799f1e99
8b799f1e99
to4899170c09
4899170c09
tofe6b5bdb08
fe6b5bdb08
toa57347c39a
a57347c39a
to7e256cb6ac
35d716baec
todd8684be91
dd8684be91
to85b96d0194
669c2e41a0
tod8811d900f
9e3f846f51
to4dfe0d8b5e
c6a291c8d2
toa40025a99f
a40025a99f
to48b6b629d6
48b6b629d6
to7663017fcb
98dce354f1
to1301e0aea5
1301e0aea5
to16dc991df7
445193e171
to599e888946
599e888946
tocc8a360280
cc8a360280
to91f6dca6b6
3584f3a500
to22e7c1aefb
34457a561d
todaaf1eab79
daaf1eab79
toe7c808176e
e7c808176e
to872185aeff
872185aeff
toce4ed41499
31af59589f
toa330935471
a38a5e41fd
todbd99d4a9c
86581d522a
tofd4dfafc06
93a058287a
toe959605e41
e959605e41
to9bd22f1f6e
9bd22f1f6e
to6289843d10
6289843d10
toda694bfb52
da694bfb52
to3d2b21ea64
3d2b21ea64
tob71a5ac2c0
b71a5ac2c0
to8e785c49a0
31e488ff78
to28193ed3ef
28193ed3ef
to2bff24ad7c
2bff24ad7c
tod68c2ed8a9
d68c2ed8a9
to4d9c6af155
6e5dacc4f8
to92da15af30
f25f01b6d5
to0ec722191e
0ec722191e
to9f01acc08c
0dd0e3b462
toeb99f8e4ae
eb99f8e4ae
toe9c986ab6c
e9c986ab6c
to9aaea82658
885ef285c7
to2b1ddd81ee
2b1ddd81ee
to382c7205a5
382c7205a5
todf6b80b736
76e0724cd5
to19095d58f3
19095d58f3
to90ff3bf5dd
90ff3bf5dd
to9c5e0497cc
9c5e0497cc
to8f714d71db
8f714d71db
toa6205cc2ce
cebb21d2dd
tob5aab24043
b5aab24043
to43772938af
43772938af
toe4c85bfd51
e4c85bfd51
tof483644532
f483644532
to6ffec67188
6ffec67188
to3e0198d20a
WIP: feat(isp): changes of switch for next stepto feat(isp): changes of switch for next step3e0198d20a
to265c98060c
@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 VLAN
typo: 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
site
imo@ -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)
optionalXml
Dans la lib t'as mis
optionalXML
avec les majusculesohh well
6f30f82d62
to15e436f533
b1d5c04c4c
tocf3b310c01
cf3b310c01
toea9c53e5f1
ea9c53e5f1
to4bc6baf561
4bc6baf561
tod70ec99e6f
@ -141,2 +141,4 @@
netconf = {
specialArgs = {
nodeMeta.deployment.targetHost = ''''${nodeMeta.deployement.targetHost}'';
Pas besoin de mettre un specialArg en utilisant
defaultText
c.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
d70ec99e6f
toab25a632ee