Bugzilla – Bug 16230
Backup Alarm fails if SBRadio is on Battery and it automatically turned itself OFF
Last modified: 2010-06-01 13:00:54 UTC
FW 7.5; SBRadio connected to Touch TinySC; SBRadio has Battery accessory pack. During a power failure (network also down) , the SBRadio is on Battery, and it automatically turned itself OFF after 20 minutes. At the scheduled time of the Alarm, the SBRadio is properly turned ON, however, the Backup Alarm fails to sound. Note that if the power failure occurs within 20 minutes of the scheduled alarm and the Radio (on battery) is not automatically turned off that the backup alarm will sound. Tony
confirmed. tagging with bug_meeting for discussion at next week's meeting
== Auto-comment from SVN commit #8805 to the jive repo by bklaas == == http://svn.slimdevices.com/jive?view=revision&revision=8805 == Fixed Bug: 16230 Description: timeToAlarm after auto wakeup is the difference between the epoch seconds in the settings file and now, not just the epoch seconds.
FYI, QA will qualify another 7.5.1 beta firmware this week and push it out to all 7.5.1 beta SBS users. The change will include the fix for this bug, as well as the WOL fix for sending a packet before a snoozed alarm refires. the version needs to be 7.5.1 and >=r8805 to contain the fixes
(In reply to comment #3) > FYI, QA will qualify another 7.5.1 beta firmware this week and push it out to > all 7.5.1 beta SBS users. > > The change will include the fix for this bug, as well as the WOL fix for > sending a packet before a snoozed alarm refires. > > the version needs to be 7.5.1 and >=r8805 to contain the fixes Thanks Ben. Tony
(In reply to comment #3) > FYI, QA will qualify another 7.5.1 beta firmware this week and push it out to > all 7.5.1 beta SBS users. > > The change will include the fix for this bug, as well as the WOL fix for > sending a packet before a snoozed alarm refires. > > the version needs to be 7.5.1 and >=r8805 to contain the fixes Ben, Was this fix pushed out with r8826? Still not working. Backup alarm does not sound if the SBRadio wakes up from sleep and cannot connect to TinySC (network powered down). SBRadio wakes up 3 minutes before alarm is scheduled to sound, (menu on the Radio shows only "My Music" and "Settings") but backup alarm does not sound. Tony
Very curious...I was very confident with the fix and tested it as working before checking it in, and I've now retested with r8826 and it worked there too...something must be different in your setup causing the failure. Since you are using SB Touch w/ tinySC, my first question is whether the Touch (and by extension the radio that connects to it), have a correctly set clock. If the radio powers up with a badly set clock the fallback alarm won't work.
(In reply to comment #6) > Very curious...I was very confident with the fix and tested it as working > before checking it in, and I've now retested with r8826 and it worked there > too...something must be different in your setup causing the failure. > > Since you are using SB Touch w/ tinySC, my first question is whether the Touch > (and by extension the radio that connects to it), have a correctly set clock. > If the radio powers up with a badly set clock the fallback alarm won't work. Yes, the clocks are correctly set. When the radio powers-up it displays the correct time. If the alarm is scheduled for a time before the Radio is shut down, the backup alarm does sound (when power to the network is off). The only thing that I notice is that before the radio shuts down (and with power to the network off) I have a full menu, but when it powers up from a shut-down, without power to the network, all I get on the Menu is "My Music" and "Settings" -- Do you get the same Menu when you test?
Yes, I do. The reduced home menu is consistent with a radio that has not received the server-driven menu items.
(In reply to comment #8) > Yes, I do. The reduced home menu is consistent with a radio that has not > received the server-driven menu items. Is there anything else I can do to try to resolve this? Is there a log file that I can email to you?
Appreciate the offer for assistance...one big challenge however is that SB Radio rotates its syslog message file too often to typically be useful. If you'd like to try though, go to Settings->Advanced->Logging and set applet.AlarmSnooze to debug. Run through the test where you see the failure, then scp off the file /var/log/messages and attach it to the bug. Because of the aforementioned log rotate issue, don't delay too long before grabbing it. Alongside the messages file, let me know the time when the alarm was supposed to go off so I can sync that up with what's being seen in the messages file.
Created attachment 6868 [details] AlarmSnoozeApplet Log attached. Alarm set for 17:00 AlarmSnoozeApplet Log attached. Alarm set for 17:00
May 29 17:00:20 squeezeplay: WARN applet.AlarmSnooze - AlarmSnoozeApplet.lua:64 RTC ALARM FIRING May 29 17:00:20 squeezeplay: WARN squeezeplay.timer - Timer.lua:193 timer error: ...share/jive/applets/AlarmSnooze/AlarmSnoozeApplet.lua:412: attempt to index field 'localPlayer' (a nil value) smoking gun appears to be here...Fallback alarm code is definitely being hit, but self.localPlayer appears to be nil. The alarm can't sound without access to the local player object. What appears to be happening in this situation is that self.localPlayer is attempted to be set in init() and fails, returning nil (I have to guess that for some reason this works in my environment, which would explain the difference we're seeing). The only other place self.localPlayer is currently set is in notify_serverConnected() which obviously won't be hit since the server isn't connected! :) I have a proposed patch for this...Tony, since you didn't have any trouble getting the messages file off your Radio, would you mind giving the updated AlarmSnoozeApplet a try to confirm the fix (I'll give you instructions)? I only ask because I can't reproduce it here...
Tony's trouble indicates this bug isn't fixed yet. Reopening...
Created attachment 6869 [details] go get self.localPlayer if we don't have it when alarm hits
Created attachment 6870 [details] full text of the patched AlarmSnoozeApplet Tony, if you're game, this is how you can put this patched copy of AlarmSnoozeApplet.lua on your radio to test the fix. 1. download the attachment and save it as AlarmSnoozeApplet.lua 2. scp it to the radio to the location /usr/share/jive/applets/AlarmSnooze/AlarmSnoozeApplet.lua 3. reboot the radio or alternatively restart squeezeplay by ssh'ing to the radio and issuing this command: /etc/init.d/squeezeplay stopwdog && /etc/init.d/squeezeplay restart following those steps will reload squeezeplay with the new alarm applet in place. from there you can attempt to reproduce the bug, which (knock in wood) should be confirmed as fixed now.
Created attachment 6871 [details] log file attached Success!
== Auto-comment from SVN commit #8837 to the jive repo by bklaas == == http://svn.slimdevices.com/jive?view=revision&revision=8837 == Fixed Bug: 16230 Description: initialize self.localPlayer for local players that call notify_playerCurrent() if self.localPlayer is not available when openAlarmWindow() is called, try to get it then as well
Many thanks for your patience and assistance with the fix, Tony this has been checked in at r8837 and after some pleading should make it into 7.5.1
(In reply to comment #18) > Many thanks for your patience and assistance with the fix, Tony > > this has been checked in at r8837 and after some pleading should make it into > 7.5.1 And thank you!