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

parker mode does not work #257

Open
dneise opened this issue Jan 22, 2018 · 0 comments
Open

parker mode does not work #257

dneise opened this issue Jan 22, 2018 · 0 comments

Comments

@dneise
Copy link
Member

dneise commented Jan 22, 2018

today we wanted to test the "parker mode", which goes like this:

The shifter performs the normal startup and is available during the night for calls. In the morning, the "parker" clicks the "Ready for shutdown!" button on the SH website (https://shifthelper.app.tu-dortmund.de/) in the time window between 30min and 20min before shutdown.

This way the SH would not call the shifter, since it assumes the shifter is already awake and "ready for shutdown". The parker would then observe/supervise the shutdown and fill the checklist. Any problems during the shutdown, which normally would lead to a call to the shifter, still lead to a call to the shifter. E.g. if during shutdown main.js throws an exception due to a drive problem, then the shifter is still called.

We believe the parker mode is especially important for the short nights in summer, where startup is very late and shutdown is very early.

The test failed, because ...

... the SH code is too strict. It does not only check if anybody is "ready for shutdown" but that the shifter is ready for shutdown. This was done to avoid that somebody in the FACT collaboration clicks this button in the morning accidentally and this way a SH-call is not send out.

So for parker mode, we have to change the SH code here:

if not awake:
return True
else:
return get_current_shifter().username not in awake

to simply:

   return not awake

This way anybody in the collaboration can press the "Ready for Shutdown!" button in the time window between 30min and 20min before shutdown and thus take the role of the parker.

To assert everything still works after this minor change, the SH will still undergo the manual test suite here:

https://github.com/fact-project/shifthelper_smartfact_mockup_test

The modification in the SH and the following manual test will be conducted today around 9 UTC.

So tomorrow we can test the parker mode again.

@dneise dneise mentioned this issue Jan 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant