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

Bugfixes #1411

Merged
merged 8 commits into from
Dec 11, 2021
Merged

Bugfixes #1411

merged 8 commits into from
Dec 11, 2021

Conversation

jimrandomh
Copy link
Contributor

Significant changes:

  • Check for bad (super slow) RPi nodejs versions, install nvm version if found. This fixes excessive CPU usage and slow looping interval on Raspberry Pi Zero.
  • Add a pair of config settings, pi_battery_shutdown_percent and edison_battery_shutdown_voltage, for controlling when the rig shuts down due to low battery. This config setting is needed for USB-powered Pi rigs, where the battery level indicator just returns a random number, which might occasionally be zero.

Minor changes:

  • If unable to update with git pull, prompt before aborting oref0-runagain.sh. This is mainly for convenience during development; a checkout in some weird branch doesn't necessarily return success when you "git pull".
  • Fix bug that would clobber sshd_config in some circumstances
  • Extend timeout on JS syntax check unit tests
  • Clean up require() usage in IOB unit test, speeding it up enough to pass without timing out on Pi Zero
  • Add bash-unit-test-temp to gitignore

Adds a pair of config settings for low-battery shutdown thresholds.
One config setting is for Edison (denominated in millivolts), the
other is for Pi (given as percentage). This config setting is needed
for USB-powered Pi rigs, where the battery level indicator just
returns a random number, which might occasionally be zero.
This is mainly for convenience during development; a checkout in
some weird branch doesn't necessarily return success when you
"git pull", but that doesn't necessarily mean you want to abort
oref0-runagain.sh.
Depending on the initial contents of sshd_config, this would sometimes
try to add a line to it, but instead completely overwrite it to contain
only that line.
On Pi Zero hardware, these tests would sometimes time out because
starting the nodejs interpreter is slow (even with a fixed nodejs
interpreter). Extend the timeout to 4s, from the default of 2s, so they
pass.
Copy link
Contributor

@scottleibrand scottleibrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a couple of minor comments

bin/oref0-cron-every-15min.sh Outdated Show resolved Hide resolved
lib/profile/index.js Show resolved Hide resolved
@tzachi-dar
Copy link
Contributor

I have run the part that updates the node manually, and indeed start up time is much shorter.

@scottleibrand scottleibrand merged commit 1ec3528 into openaps:dev Dec 11, 2021
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

Successfully merging this pull request may close these issues.

3 participants