Hello everyone,

I'm trying to make a startup script to run and automatically change some router settings and this script works perfectly if I run it from the terminal, but if the GUI "run script" button is pressed or the scheduler runs the script, the run count increases, but the script does not complete.

I have the following script:

{
{/interface wireless
set [ find default-name=wlan1 ] band=2ghz-b/g/n channel-width=20/40mhz-Ce \
disabled=no distance=indoors frequency=auto mode=ap-bridge ssid=\
CHANGEME wireless-protocol=802.11
set [ find default-name=wlan2 ] band=5ghz-a/n/ac channel-width=20/40/80mhz-Ceee \
disabled=no distance=indoors frequency=auto mode=ap-bridge ssid=\
CHANGEME_5G_AC_Only wireless-protocol=802.11}
{/interface wireless security-profiles
set [ find default=yes ] authentication-types=wpa-psk,wpa2-psk mode=dynamic-keys supplicant-identity=\
MikroTik wpa-pre-shared-key=CHANGEME wpa2-pre-shared-key=CHANGEME}}


{/user group set 1 skin=restech
/user group set 0 skin=customer}

{/int wireless
set 0 name=wlan1
set 1 name=wlan2} <------------------------------- With some tests, I have determined that all settings above this line apply correctly, so this is where it breaks

{/int bridge port
remove 4,5
add bridge=bridge interface=wlan1
add bridge=bridge interface=wlan2}

{/system ident set name=CHANGEME}

{/int eth reset-mac-address 0,1,2,3,4}

{/sys script rem program}
}


Any idea what I'm doing wrong here?

Thanks!

I actually solved my issue. It was due to the nature of the dynamic numbering system. You need print statements to cache your numbers as referable variables.

The working script was as follows:

{

{/interface wireless
set [ find default-name=wlan1 ] band=2ghz-b/g/n channel-width=20/40mhz-Ce \
disabled=no distance=indoors frequency=auto mode=ap-bridge ssid=\
CHANGEME wireless-protocol=802.11
set [ find default-name=wlan2 ] band=5ghz-a/n/ac channel-width=20/40/80mhz-Ceee \
disabled=no distance=indoors frequency=auto mode=ap-bridge ssid=\
CHANGEME_5G_AC_Only wireless-protocol=802.11}
{/interface wireless security-profiles
set [ find default=yes ] authentication-types=wpa-psk,wpa2-psk mode=dynamic-keys supplicant-identity=\
MikroTik wpa-pre-shared-key=CHANGEME wpa2-pre-shared-key=CHANGEME}


{/user group
print
set 1 skin=restech
set 0 skin=customer}

{/int wireless
print
set 0 name=wlan1
set 1 name=wlan2}



{/int bridge port
print
remove 4,5
add bridge=bridge interface=wlan1
add bridge=bridge interface=wlan2}

{/system ident set name=CHANGEMETESTINGSTUFF}

{/int eth reset-mac-address 0,1,2,3,4}

{/sys script
pr
rem 0}
}


Thanks

Thanks for following up with solution - thats actually a very peculiar behaviour which would be easy to overlook.

You should not (never ever!) use any print and index in your scripts. Things will break badly if items are numbered different for what ever reason. Instead of this bad code:
You should use something like this:
I guess that meets your needs? But it will not break if wlan1 and wlan2 are not numbered 4 and 5 - in contrast to your code.

You should not (never ever!) use any print and index in your scripts. Things will break badly if items are numbered different for what ever reason. Instead of this bad code:

Code: Select all

/int bridge port
print
remove 4,5

You should use something like this:

Code: Select all

/int bridge port
remove [ find where interface=wlan1 ]
remove [ find where interface=wlan2 ]

I guess that meets your needs? But it will not break if wlan1 and wlan2 are not numbered 4 and 5 - in contrast to your code.

That indeed meets my needs, though in my case, the script is to fix some things that result from using a backup config for the base unit on other units (trying to simplify configuration for customer deployment). As a result of the backup file always being the same, I was not in danger of the numbers changing, but in most cases I would have been. Those commands are super useful and I'll definitely use those going forward though. Thanks for the help!