SG-1100 black diamond led always off in 23.01

stephenw10

The patch to fix it is in the recommend patches list in the System Patches package now. Though AFAIK it's identical to the patch shown here.
So the LED works as expected until a notification is triggered?

flepti

@stephenw10 it would seem that way the other day yes, though today, I logged into the dahsboard, saw a notification and the diamond led is still lit, the moment I click on clearing all the notifications, the diamond led turned off, this might be the case the other day too, I'll keep observing though.

flepti

@stephenw10 yep, that's the case, when I click on "Mark All as Read" from the notifications, the diamond led turns off.

RobbieTT

We have an LED that lights-up black?

Mr Desiato is very appreciative.

️

dk98174003

Same in version 23.05 ;O)

stephenw10

Just confirm running the LED script manually sets the LED as expected?

[23.05-RELEASE][admin@1100-2.stevew.lan]/root: pfSense-led.sh
usage:
pfSense-LED booting
pfSense-LED ready
pfSense-LED update [1|0]
pfSense-LED updating

Only cancelling notifications sets it unexpectedly?

stephenw10

Ok, I think this is happening because the LED happens to be named the same as one of the LEDs was on the old ALIX device and there is still some old code using that.
Testing....

stephenw10

Confirmed: https://redmine.pfsense.org/issues/14482

stephenw10

Here's a patch for this issue. Works on an 1100 here as expected:

1044.diff.txt

Let me know if that solves it.

Steve

stephenw10

Here's a better patch from @jimp. This is now in new snapshots and will be in the next pfSense release.

led_check.diff

Steve