Skip to content

Commit

Permalink
calling eulenfunk-hostapdcheck after -HUP
Browse files Browse the repository at this point in the history
  • Loading branch information
@Adorfer
Adorfer committed Jul 15, 2020
1 parent c274380 commit 1893c69
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions gluon-ssid-changer/shsrc/ssid-changer.sh
View file
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,10 @@ fi
if [ $HUP_NEEDED = 1 ]; then
# send HUP to all hostapd to load the new SSID
killall -HUP hostapd
## check for nonmachting hotapd-pidfiles
if [ -f /lib/gluon/eulenfunk-hotfix/check_hostapd.sh ] ; then
sleep 2 # settle down
ps|grep hostapd|grep .pid|xargs -n 10 /lib/gluon/eulenfunk-hotfix/check_hostapd.sh
HUP_NEEDED=0
echo "HUP!"
fi
Expand Down

7 comments on commit 1893c69

@rubo77
Copy link

@rubo77 rubo77 commented on 1893c69 Jul 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you add this check?

@Adorfer
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in order to have working client-wifi.

@rubo77
Copy link

@rubo77 rubo77 commented on 1893c69 Jul 21, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what was the problem and on which devices? I had no problems so far in kiel

@Adorfer
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it looks like hostapd may enter undefined/non operational status after sigHUP, e.g. having mismatching PID and value in PID-files.
Or claiming in syslog about derrivating PID-values.
see. https://forum.freifunk.net/t/outdoor-mode-in-gluon2020-1-x-dfs-pre-cac-expired/22128

@rubo77
Copy link

@rubo77 rubo77 commented on 1893c69 Jul 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then this is just a dirty fix! we should better solve the reason, it fails instead

@rubo77
Copy link

@rubo77 rubo77 commented on 1893c69 Jul 21, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bitte bei sowas wichtigen immer ein issue eröffnen.

Let's discuss it further here
Freifunk-Nord#26

@Adorfer
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then this is just a dirty fix!

It was never stated to be something different.

Please sign in to comment.