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

Backend tweaks #818

Merged
merged 9 commits into from
Apr 19, 2021
Merged

Backend tweaks #818

merged 9 commits into from
Apr 19, 2021

Conversation

LachlanStuart
Copy link
Contributor

A bunch of small changes, mainly dev experience improvements.

  • Disabled sci-test as it has just been continually failing on master branch
  • Remove auto-publishing of python-client, as we always manually publish it
  • Increase email verification tokens' expiry time (fixes Raise email verification timeout to 60 minutes #780)
  • Turn off slack notifications for successful INDEX/UPDATE/CLASSIFY_OFF_SAMPLE messages. Failures are still reported, though these processes virtually never fail.
  • Change the Lithops retry logic to always retry, not just for stalls. I did this because I've seen ~3 datasets that failed on their first attempt but succeeded on retry since the Lithops deployment.
  • Add killasgroup to the sm-lithops-daemon supervisor file. documented here. This is a small mitigation for Fix Lithops zombie processes issue #795 , which should hopefully mean that stalled background processes spawned by multiprocessing don't need to be manually removed

@LachlanStuart LachlanStuart merged commit ba1e60f into master Apr 19, 2021
@LachlanStuart LachlanStuart deleted the feat/disable-broken-ci branch April 19, 2021 09:50
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.

Raise email verification timeout to 60 minutes
2 participants