Skip to content

Commit e5c3ba3

Browse files
committed
Merge branch 'loadgen-5hs-tests' into multi-server-tests
2 parents 10d469e + 2c2201c commit e5c3ba3

4 files changed

Lines changed: 43 additions & 16 deletions

File tree

.github/workflows/ci-multi-server-tests.yml

Lines changed: 34 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,11 @@ name: Multi-Server CI Tests
22

33
on:
44
push:
5-
branches: [ "loadgen-5hs-tests" ]
6-
5+
branches-ignore:
6+
- coverity_scan
7+
- run-fuzzer**
8+
- debug-fuzzer-**
79
pull_request:
8-
branches: [ "loadgen-5hs-tests" ]
910
paths:
1011
- '.github/workflows/ci-multi-server-tests.yml'
1112
- 'src/tests/multi-server/**'
@@ -15,8 +16,8 @@ jobs:
1516
pre-test:
1617
runs-on: ubuntu-latest
1718
outputs:
18-
selfhosted: 0
19-
#selfhosted: ${{ github.repository_owner == 'FreeRADIUS' && '1' || '0' }}
19+
#selfhosted: 0
20+
selfhosted: ${{ (github.repository_owner == 'FreeRADIUS') && '1' || '0' }}
2021
steps:
2122
- run: echo "Pre-test job; checking if using self-hosted runners"
2223

@@ -84,10 +85,10 @@ jobs:
8485
- name: Get pre-built Docker image for self-hosted runner test
8586
if: ${{ needs.pre-test.outputs.selfhosted == '1' }}
8687
run: |
87-
docker pull docker.internal.networkradius.com/self-hosted-${MULTI_SERVER_ENV_DOCKER_BUILD_OS}:latest
88+
docker pull docker.internal.networkradius.com/self-hosted
8889
8990
# Tag freeradius build image using using a non-OS specific name to be used with the multi-server docker compose environment.
90-
docker tag docker.internal.networkradius.com/self-hosted-${MULTI_SERVER_ENV_DOCKER_BUILD_OS}:latest freeradius-build:latest
91+
docker tag docker.internal.networkradius.com/self-hosted freeradius-build:latest
9192
9293
- name: Run test-5hs-autoaccept test
9394
if: ${{ needs.pre-test.outputs.selfhosted != '1' }}
@@ -145,3 +146,29 @@ jobs:
145146
echo "TEST FAILED: No Matched line found with equal non-zero counts"
146147
exit 1
147148
fi
149+
150+
#
151+
# If the CI has failed and the branch is loadgen-5hs-tests then we start a tmate
152+
# session to provide interactive shell access to the session.
153+
#
154+
# The SSH rendezvous point will be emited continuously in the job output,
155+
# which will look something like:
156+
#
157+
# SSH: ssh VfuX8SrNuU5pGPMyZcz7TpJTa@sfo2.tmate.io
158+
#
159+
# For example:
160+
#
161+
# git push origin loadgen-5hs-tests --force
162+
#
163+
# Look at the job output in: https://github.com/FreeRADIUS/freeradius-server/actions
164+
#
165+
# ssh VfuX8SrNuU5pGPMyZcz7TpJTa@sfo2.tmate.io
166+
#
167+
# Access requires that you have the private key corresponding to the
168+
# public key of the GitHub user that initiated the job.
169+
#
170+
- name: "Debug: Start tmate"
171+
uses: mxschmitt/action-tmate@v3
172+
with:
173+
limit-access-to-actor: true
174+
if: ${{ github.ref == 'refs/heads/loadgen-5hs-tests' && failure() }}

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, (load_gen_num_of_dest_servers + 1)) %}
44-
radius radius-module-dest-server{{ i }} {
45-
$template radius-module-dest-server-tmpl
43+
{% for i in range(1, (load_gen_num_of_dst_servers + 1)) %}
44+
radius radius-module-dst-server{{ i }} {
45+
$template radius-module-dst-server-tmpl
4646
udp {
47-
ipaddr = {{ load_gen_dest_server_name }}{{ i }}
47+
ipaddr = {{ load_gen_dst_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, (load_gen_num_of_dest_servers + 1)) %}
187-
radius-module-dest-server{{ i }}
186+
{% for i in range(1, (load_gen_num_of_dst_servers + 1)) %}
187+
radius-module-dst-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-dest-server-tmpl {
5+
radius-module-dst-server-tmpl {
66
mode = client
77
transport = udp
88
type = Access-Request
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Docker compose Jinja template vars
22
compose_num_of_home_servers: 5
33
# Load generator Jinja template vars
4-
load_gen_num_of_dest_servers: 5
5-
load_gen_dest_server_name: homeserver
4+
load_gen_num_of_dst_servers: 5
5+
load_gen_dst_server_name: homeserver
66
# General Jinja template vars
77
listener_type: file

0 commit comments

Comments
 (0)