Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call during shutdown error? #230

Open
maxmahlke opened this issue Apr 7, 2017 · 5 comments
Open

Call during shutdown error? #230

maxmahlke opened this issue Apr 7, 2017 · 5 comments

Comments

@maxmahlke
Copy link

main.js crashed this morning during the shutdown due to DriveControl timeout. Shifthelper didn't call, I assume because it was past the scheduled shutdown time and it no longer checks for main.js to be running.
Since the shifter should be awake and monitoring the system, I don't know whether Shifthelper should even call, but it could help drawing the shifter's attention to the problem. So I'm reporting this to make sure this is expected behaviour and to discuss whether Shifthelper should call in this situation.

@dneise
Copy link
Member

dneise commented Apr 11, 2017

I think this is indeed not the intended behaviour.

Shifthelper goes into "daymode", i.e. it does not expect main.js to be running after the "Shutdown" task in the schedule. However ... the time of the "Shutdown" task is only the beginning of the actual shutdown ... not the end. So we expect main.js to be running, say another 10 minutes after the "Shutdown" task, right?

I guess this needs some work here:
https://github.com/fact-project/shifthelper/blob/master/shifthelper/tools/is_shift.py

@maxnoe
Copy link
Member

maxnoe commented Apr 11, 2017

I think this could be improved, but is no problem. The shifthelper makes sure the shifter is awake via the I am awake button. He has to supervise the shutdown. The shifthelper also makes sure the shutdown is completed via the checklist.

I don't know if we really want to call an awake shifter because something happens. Do we?

@dneise
Copy link
Member

dneise commented Apr 11, 2017

I'm also not sure. I just would like to have the time range, within we want to somehow ensure that main.js is running to be defined.

Of course main.js may run before the "Startup" task .. and it may also run after the shutdown has been completed ... but it actually must be running at least some time after the "Shutdown" task ...

Still it is good that @madoee made us aware of this so we can clearly decide, what we want to do here.

@maxnoe
Copy link
Member

maxnoe commented Apr 11, 2017

It has to run some time after the shutdown task starts, yes. BUT: as the shutdown has to be supervised by an awake shifter, the shifthelper needs not to care in my opinion.

During shutdown the responsibility of the shifthelper ends with Shifter is awake and starts again some time after shutdown to make sure it happend.

@dneise
Copy link
Member

dneise commented Apr 19, 2017

So can we close this, or do we want to add some docu about this somewhere?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants