[SCHEDULER-1225]  Bullet-proof Sky Condition Server Input
Type Enhancement
Priority Medium
Severity Minor
Component Dispatch Scheduler Engine
Fixed In Version [8.0] 8.0
Versions Affected [3.7] 3.7
Severity Closed
Resolution Complete
Reported By Bob Denny
Resources Bob Denny
Start Date 11/12/2014

Description 
See this Comm Center thread with Danny Flippo and Mark Williams. I postulated that this might be problems with their Sky Condition servers. It looks like I was right.






The Sky Condition server input should be able to suffer all sorts of catastrophic problems with the Sky Condition server without taking out the Scheduler. Innstead, the Scheduler should switch to the Dead (Operator Intervention Required) state.

Comments
12/15/2014 3:24:10 PM   Bob Denny 
OK, this is going to go out to the pre-release for user testing.
11/12/2014 8:26:10 PM   Bob Denny 

 

  SVN Comment
 

 

  Author
  rbdenny
 

 

  Repository
  svn+ssh://rbdenny@a2_svn_dc3/home/rbdenny/svn/astro/scheduler
 

 

  SVN Revision
  416
 

 

  Affected files
  /trunk/Help/RelNotes.htm (Modified)
/trunk/Scheduler/ConfigForm.cs (Modified)
/trunk/Scheduler/Engine.cs (Modified)
/trunk/Scheduler/MainForm.cs (Modified)
/trunk/Scheduler/SkyCondition.cs (Modified)

 

 

  Check-in comment
  Bullet-proof Sky Condition Server Input. This ended up uncovering several weird things. Still needs more testing.
GEM:1225
11/12/2014 6:26:16 PM   Bob Denny 
Holy cow... there are so many little tricky things going on here... The ACP sequencer is being released on one thread yet the DispatcherEnabled value is being changed on the main thread.






The Engine is releasing the Sequencer, and at the same time the main thread tmrDisplay_Tick() is polling Engine.Enabled. Much work needed here, including some critical sections. 
11/12/2014 4:29:42 PM   Bob Denny 
Add a notation to the config form that a change to the sky server requires a restart.
11/12/2014 3:49:58 PM   Bob Denny 
Very interesting. The Scheduler got into a state where it transitioned from Startup state to Dead state, and that left the Exit menu item disabled, which also locked it from allowing closing with the [X] box, and popped that "Supervisor" "Wait a bit while..." balloon. Had to kill the program.
11/12/2014 3:10:02 PM   Bob Denny 
Having trouble reproducing this ... there is already a trap around the retrieval of sky condition value...