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

Improvements to the service-idler #1139

Merged
merged 8 commits into from Aug 29, 2019

Conversation

@shreddedbacon
Copy link
Member

commented Jul 5, 2019

Checklist

  • Affected Issues have been mentioned in the Closing issues section
  • Documentation has been written/updated.
  • Changelog entry has been written

This change allows the router-log query to logs-db to allow for an adjustable interval from 4h. In busy clusters it can be beneficial to idle environments sooner if they aren't being used as often.

It also adds 2 new checks

  • one for checking if there are any running builds in the namespace
  • one that checks if any deploymentconfigs notin (mariadb,postgres) have pods running greater than the default 4hr interval before
    These 2 checks need to pass before it will continue to check router-logs and do the eventual idling.

It adds the functionality to also force idling on services if required by using ./idle-services.sh force

Changelog Entry

Improvement - Allow adjustable interval in service idler (#1138)
Improvement - Check running builds and pods before idling (#1138)
Improvement - Allow services to be force idled manually ./idle-services.sh force

Closing issues

closes #1138

@shreddedbacon shreddedbacon changed the title Allow adjustable router-log interval in service idler Improvements to the service-idler Jul 22, 2019

@Schnitzel Schnitzel added this to the v1.0.0 RBAC milestone Aug 21, 2019

Schnitzel added 2 commits Aug 21, 2019
Schnitzel added a commit that referenced this pull request Aug 28, 2019
Schnitzel added a commit that referenced this pull request Aug 29, 2019

@Schnitzel Schnitzel merged commit 1540f66 into amazeeio:master Aug 29, 2019

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.