Skip to content

Commit 3231c52

Browse files
author
root
committed
Fix fasterdata-tuning 1.3.10: ensure state operations return exit code 0 on success
1 parent 5a069e7 commit 3231c52

File tree

4 files changed

+14
-4
lines changed

4 files changed

+14
-4
lines changed

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,12 @@
55

66
All notable changes to this repository will be documented in this file.
77

8+
## [1.3.10] - 2026-01-27
9+
10+
### Fixed
11+
12+
- `fasterdata-tuning.sh` v1.3.10: Ensure all state management functions (`--restore-state`, `--diff-state`, `--list-states`) explicitly return 0 on successful completion so they exit with code 0 instead of inheriting non-zero exit codes from sub-commands.
13+
814
## [1.3.9] - 2026-01-27
915

1016
### Fixed

docs/perfsonar/tools_scripts/fasterdata-tuning.sh

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/usr/bin/env bash
22
# fasterdata-tuning.sh
33
# --------------------
4-
# Version: 1.3.9
4+
# Version: 1.3.10
55
# Author: Shawn McKee, University of Michigan
66
# Acknowledgements: Supported by IRIS-HEP and OSG-LHC
77
#
@@ -26,6 +26,7 @@
2626
# NEW in v1.3.7: Fix summary display of packet pacing status to correctly detect applied qdisc.
2727
# NEW in v1.3.8: Validate required option arguments up front to avoid unbound-variable errors and provide clearer CLI feedback.
2828
# NEW in v1.3.9: Fix qdisc state restoration: properly reset interface qdisc to pfifo_fast when saved state lacks qdisc or has unknown qdisc (fixes packet pacing not being disabled after restore).
29+
# NEW in v1.3.10: Ensure all state management functions return 0 on successful completion so --restore-state/--diff-state/--list-states exit cleanly with code 0.
2930
#
3031
# Sources: https://fasterdata.es.net/host-tuning/ , /network-tuning/ , /DTN/
3132
#
@@ -2432,6 +2433,7 @@ do_list_states() {
24322433
done <<< "$states"
24332434

24342435
echo ""
2436+
return 0
24352437
}
24362438

24372439
do_delete_state() {
@@ -2573,6 +2575,7 @@ PY
25732575
echo ""
25742576
echo "Use --restore-state to restore the saved configuration"
25752577
echo ""
2578+
return 0
25762579
}
25772580

25782581
do_restore_state() {
@@ -2890,6 +2893,7 @@ PY
28902893
echo "Note: Some settings may require a reboot to take full effect."
28912894
echo "Run with --mode audit to verify the restored state."
28922895
echo ""
2896+
return 0
28932897
}
28942898

28952899
print_host_info() {
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
8c47574d2dc228cb65affe6330eb2dede21ae31f3ec8e3a0eb0c1e2ae57db5e0 fasterdata-tuning.sh
1+
dfbdd6abe4f99a9196376de16d92648a0327d1667ddbb1d5d4a354f04e33ce8f fasterdata-tuning.sh

docs/perfsonar/tools_scripts/scripts.sha256

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
166d52478315bf67e63e69e3c9272734434f6ad2a3fa30b30b37c80c94d97034 certbot-deploy-hook.sh
22
2812b78534e8268751250b271cf0ac1868a6a8b420f9e6b4f96f715459d0eaf5 check-deps.sh
33
f0bf15b7223447878b00260d33f1db41995f47d4b56216911bbabe7f1b8435a9 check-perfsonar-dns.sh
4-
d1f100e2e5eba58007bf89455edb1b7065e8e7124c9a4238f18ce13c60a2f2e5 configure-toolkit-letsencrypt.sh
5-
8c47574d2dc228cb65affe6330eb2dede21ae31f3ec8e3a0eb0c1e2ae57db5e0 fasterdata-tuning.sh
4+
8c47574d2dc228cb65affe6330eb2dede21ae31f3ec8e3a0eb0c1e2ae57db5e0 configure-toolkit-letsencrypt.sh
5+
dfbdd6abe4f99a9196376de16d92648a0327d1667ddbb1d5d4a354f04e33ce8f fasterdata-tuning.sh
66
14d88a50bcbc606b21b00b4bcfab779c2a2f70f1576593f66502611719620df0 install-systemd-service.sh
77
4b2b91593a3ceb3c8159cc2d6ca749aeeb7a558a766227c46493feb84655f04a install-systemd-units.sh
88
9d9ff6bab281c800c923ac9616c9f88502e9677d6902d3df243fa5faf50a02fe install_tools_scripts.sh

0 commit comments

Comments
 (0)