Skip to content

Commit 538730b

Browse files
committed
Jinja template and variable updates to use variable names that make it easier to know where they are used during Jinja template rendering.
1 parent e16c151 commit 538730b

4 files changed

Lines changed: 15 additions & 11 deletions

File tree

src/tests/multi-server/environments/configs/freeradius/load-generator/radiusd.conf.j2

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,11 @@ modules {
4040
{% include "environments/configs/freeradius/common/mods-available/stats" %}
4141

4242
# radius module instance configuration
43-
{% for i in range(1, (num_of_dst_servers + 1)) %}
44-
radius radius-module-dst-server{{ i }} {
45-
$template radius-module-dst-server-tmpl
43+
{% for i in range(1, (load_gen_num_of_dest_servers + 1)) %}
44+
radius radius-module-dest-server{{ i }} {
45+
$template radius-module-dest-server-tmpl
4646
udp {
47-
ipaddr = homeserver{{ i }}
47+
ipaddr = {{ load_gen_dest_server_name }}{{ i }}
4848
}
4949
}
5050
{% endfor %}
@@ -183,8 +183,8 @@ server load-generator {
183183
authenticate load-generator-proxy {
184184
# Use redundant-load-balance to distribute requests to multiple home servers
185185
redundant-load-balance {
186-
{% for i in range(1, (num_of_dst_servers + 1)) %}
187-
radius-module-dst-server{{ i }}
186+
{% for i in range(1, (load_gen_num_of_dest_servers + 1)) %}
187+
radius-module-dest-server{{ i }}
188188
{% endfor %}
189189
}
190190
}

src/tests/multi-server/environments/configs/freeradius/load-generator/template.d/load-generator-templates

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
#
33
# radius module template for load-generator
44
#
5-
radius-module-dst-server-tmpl {
5+
radius-module-dest-server-tmpl {
66
mode = client
77
transport = udp
88
type = Access-Request

src/tests/multi-server/environments/docker-compose/env-5hs-autoaccept.yml.j2

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ x-common-config: &id001
1717
- NET_ADMIN
1818
- SYS_PTRACE
1919
services:
20-
{% for i in range(1, (num_of_dst_servers + 1)) %}
20+
{% for i in range(1, (compose_num_of_home_servers + 1)) %}
2121
homeserver{{ i }}:
2222
image: freeradius-build:latest
2323
volumes:
@@ -48,7 +48,7 @@ services:
4848
- "1813:1813/udp"
4949
- "1820:1820/udp"
5050
depends_on:
51-
{% for i in range(1, (num_of_dst_servers + 1)) %}
51+
{% for i in range(1, (compose_num_of_home_servers + 1)) %}
5252
- homeserver{{ i }}
5353
{% endfor %}
5454
volumes:
Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1-
num_of_dst_servers: 5
2-
dst_server_name: homeserver
1+
# Docker compose Jinja template vars
2+
compose_num_of_home_servers: 5
3+
# Load generator Jinja template vars
4+
load_gen_num_of_dest_servers: 5
5+
load_gen_dest_server_name: homeserver
6+
# General Jinja template vars
37
listener_type: file

0 commit comments

Comments
 (0)