sipXecs

Scheduled Device (phone, gateway) reboot from sipxconfig

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 3.11.5
  • Component/s: None
  • Rank:
    Issue can not be ranked.   
  • Labels:
  • Description:
    Hide
    We have done quite some sipX and sipxchange presentations now and there is one request that pops up a lot from administrators. Since they cannot reboot the phones to do config changes or firmware upgrades during working hours, admins would love to have a scheduler for this that would allow them to set the reboot time to 2am for example :)
    Show
    We have done quite some sipX and sipxchange presentations now and there is one request that pops up a lot from administrators. Since they cannot reboot the phones to do config changes or firmware upgrades during working hours, admins would love to have a scheduler for this that would allow them to set the reboot time to 2am for example :)

Issue Links

Activity

Hide
Douglas Hubler added a comment - 2006-06-23 06:44
Cool feature
Show
Douglas Hubler added a comment - 2006-06-23 06:44 Cool feature
Hide
Lih-Shyng Tzeng added a comment - 2006-06-28 21:41
Assign this to Gibson for feature tracking.
Show
Lih-Shyng Tzeng added a comment - 2006-06-28 21:41 Assign this to Gibson for feature tracking.
Hide
Lih-Shyng Tzeng added a comment - 2006-06-29 16:17
Added to 3.6 list.
Show
Lih-Shyng Tzeng added a comment - 2006-06-29 16:17 Added to 3.6 list.
Hide
Lih-Shyng Tzeng added a comment - 2006-07-02 20:28
Removed this from 3.6 list per Martin's email on 7/1/06.
Show
Lih-Shyng Tzeng added a comment - 2006-07-02 20:28 Removed this from 3.6 list per Martin's email on 7/1/06.
Hide
Andrei Niculae added a comment - 2007-12-07 11:00
UI Proposal
Show
Andrei Niculae added a comment - 2007-12-07 11:00 UI Proposal
Hide
Andrei Niculae added a comment - 2007-12-07 11:00
UI Proposal
Show
Andrei Niculae added a comment - 2007-12-07 11:00 UI Proposal
Hide
Andrei Niculae added a comment - 2007-12-10 11:16
Changed the UI.
See the thread about this :
http://comments.gmane.org/gmane.comp.voip.sipx.devel/7843
Show
Andrei Niculae added a comment - 2007-12-10 11:16 Changed the UI. See the thread about this : http://comments.gmane.org/gmane.comp.voip.sipx.devel/7843
Hide
Andrei Niculae added a comment - 2008-05-07 07:33
The first patch for this issue. This is against an older revision, since it was ready before Easter, but I had some minor modifications to add to it. I see that there have been some changes with the restart mechanism. I'll post a patch against the newer revision soon.
Show
Andrei Niculae added a comment - 2008-05-07 07:33 The first patch for this issue. This is against an older revision, since it was ready before Easter, but I had some minor modifications to add to it. I see that there have been some changes with the restart mechanism. I'll post a patch against the newer revision soon.
Hide
Damian Krzeminski added a comment - 2008-05-22 14:29
I tried to rebase it but as you noticed Restart management was rewritten so there is not much that can be salvaged.
However I believe that it should be now easier to implement this feature (there is only a single restart manager not 3).

Some comments:
- the bulk of the implementation should end up in RestartManger and not in LazyRestartManger, lazy restart manager should just pass the call to real restart manager, there is no point in delaying the scheduled restart: you do not have to spin a new thread to schedule something
- please do not use @SuppressWarnings - there are rare cases where it makes sense to use it, mostly if we want to preserve unused parameter, but in most cases we want to see and get rid of all warnings
- please do not add comments in Romanian: it's a beautiful language alas not commonly known; we all agreed to just continue to butcher English







Show
Damian Krzeminski added a comment - 2008-05-22 14:29 I tried to rebase it but as you noticed Restart management was rewritten so there is not much that can be salvaged. However I believe that it should be now easier to implement this feature (there is only a single restart manager not 3). Some comments: - the bulk of the implementation should end up in RestartManger and not in LazyRestartManger, lazy restart manager should just pass the call to real restart manager, there is no point in delaying the scheduled restart: you do not have to spin a new thread to schedule something - please do not use @SuppressWarnings - there are rare cases where it makes sense to use it, mostly if we want to preserve unused parameter, but in most cases we want to see and get rid of all warnings - please do not add comments in Romanian: it's a beautiful language alas not commonly known; we all agreed to just continue to butcher English
Hide
Michael Haag added a comment - 2008-06-27 10:02
another customer request for this functionality (there have been others, too):
 salesforce case 4258, https://na4.salesforce.com/50060000004VZGu
Show
Michael Haag added a comment - 2008-06-27 10:02 another customer request for this functionality (there have been others, too):  salesforce case 4258, https://na4.salesforce.com/50060000004VZGu
Hide
Andrei Niculae added a comment - 2008-07-18 08:53
This is the patch that implements XCF-1098
It introduces a new TimerTask in RestartManagerImpl.java which triggers a restart at a specified time.
It also introduces a new column to all the devices (phones, gateways, sbcDevices): information. This column
displays the restart date.
I've named this column information, because the restart date ends up in the database only after a profile
is generated, so the user might have been confused if he scheduled a restart and the scheduled restart date
didn't appear immediately. That's why I introduced a new class: DeviceStatus which keeps track whether or not a device is generating its profile. Also, now, when a profile is generated, a device cannot be deleted (this was a bug,
I believe). This behavior can be extended so a device can report different statuses. Right now it only reports if it
is generating its profile or not (updating).

Please review.
Show
Andrei Niculae added a comment - 2008-07-18 08:53 This is the patch that implements XCF-1098 It introduces a new TimerTask in RestartManagerImpl.java which triggers a restart at a specified time. It also introduces a new column to all the devices (phones, gateways, sbcDevices): information. This column displays the restart date. I've named this column information, because the restart date ends up in the database only after a profile is generated, so the user might have been confused if he scheduled a restart and the scheduled restart date didn't appear immediately. That's why I introduced a new class: DeviceStatus which keeps track whether or not a device is generating its profile. Also, now, when a profile is generated, a device cannot be deleted (this was a bug, I believe). This behavior can be extended so a device can report different statuses. Right now it only reports if it is generating its profile or not (updating). Please review.
Hide
Damian Krzeminski added a comment - 2008-08-28 08:25
http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=13264

XCF-1098 rewrite restart manager to handle scheduled restarts

RestartManager interface now takes timestamp of the restart allowing for delayed restart. The
implementation relies on ScheduledExecutorService from concurrent package. Throttling restarts is
still supported (if you restart more than one device there is a minimum throttle period to separate
http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=
persistent: if sipXconfig is closed or dies devices will not get restarted. This is not a new
problem, but it may become more serious now that restarting can be delayed by more than just a
couple of minutes.

------------------------------------------------------------------------
http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=13265

XCF-1098 ProfileManager fixes to allow for delayed restart

Profile Manager interface allows for specifying the restart timestamp. Profiles are still generated
asynchronously, timestamp applies to restart only (so specifying restart in couple of days will not
stop sipXconfig from pushing profile as soon as it has time to do that). The restart will always be
scheduled *after* the profile is generated.

------------------------------------------------------------------------
http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=13266

XCF-1098 ConfirmProfileGeneration UI updated to allow for delaying device restart.
Show
Damian Krzeminski added a comment - 2008-08-28 08:25 http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=13264 XCF-1098 rewrite restart manager to handle scheduled restarts RestartManager interface now takes timestamp of the restart allowing for delayed restart. The implementation relies on ScheduledExecutorService from concurrent package. Throttling restarts is still supported (if you restart more than one device there is a minimum throttle period to separate http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev= persistent: if sipXconfig is closed or dies devices will not get restarted. This is not a new problem, but it may become more serious now that restarting can be delayed by more than just a couple of minutes. ------------------------------------------------------------------------ http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=13265 XCF-1098 ProfileManager fixes to allow for delayed restart Profile Manager interface allows for specifying the restart timestamp. Profiles are still generated asynchronously, timestamp applies to restart only (so specifying restart in couple of days will not stop sipXconfig from pushing profile as soon as it has time to do that). The restart will always be scheduled *after* the profile is generated. ------------------------------------------------------------------------ http://sipxecs.sipfoundry.org/ViewVC/sipXecs?view=rev&rev=13266 XCF-1098 ConfirmProfileGeneration UI updated to allow for delaying device restart.
Hide
Ananda Teertha added a comment - 2009-01-09 00:50
This feature has been implemented and the devices indeed reboot after setting the schedule. Currently we have,

sipxpublisher 3.11.9-014409 2009-01-07T03:09:27 oem-centos5
  sipxproxy 3.11.9-014409 2009-01-07T03:05:24 oem-centos5

Any bugs found will be reported separately.
Hence this issue may be closed.


Show
Ananda Teertha added a comment - 2009-01-09 00:50 This feature has been implemented and the devices indeed reboot after setting the schedule. Currently we have, sipxpublisher 3.11.9-014409 2009-01-07T03:09:27 oem-centos5   sipxproxy 3.11.9-014409 2009-01-07T03:05:24 oem-centos5 Any bugs found will be reported separately. Hence this issue may be closed.

People

Dates

  • Created:
    2006-06-23 04:03
    Updated:
    2009-05-06 16:05
    Resolved:
    2008-08-28 08:25