Skip to content
This repository has been archived by the owner on Sep 12, 2022. It is now read-only.

Run monitor_machines once each night rather than every 30 mins #625

Merged
merged 2 commits into from
Jun 15, 2018

Conversation

c-mart
Copy link
Contributor

@c-mart c-mart commented Jun 15, 2018

Description

This task is taking >1 hour to run in production for each provider, doesn't make sense to run every 30 mins. Image import via this method is rare, and monitor_machines can always be run on-demand if needed. See #atmosphere_issues channel in Jetstream Slack from morning of 2018-06-15.

Checklist before merging Pull Requests

  • New test(s) included to reproduce the bug/verify the feature
  • Add an entry in the changelog
  • Documentation created/updated (include links)
  • If creating/modifying DB models which will contain secrets or sensitive information, PR to clank updating sanitation queries in roles/sanitary-sql-access/templates/sanitize-dump.sh.j2
  • Reviewed and approved by at least one other contributor.
  • New variables supported in Clank
  • New variables committed to secrets repos

@coveralls
Copy link

coveralls commented Jun 15, 2018

Coverage Status

Coverage increased (+0.006%) to 37.369% when pulling d6320dc on c-mart:celery-periodic-task-timing into 6d78e13 on cyverse:v33.

@cdosborn cdosborn changed the base branch from master to v33 June 15, 2018 16:11
@cdosborn cdosborn force-pushed the celery-periodic-task-timing branch from afc91e0 to d6320dc Compare June 15, 2018 16:15
@cdosborn
Copy link
Contributor

Just moved this branch onto v33 so that it'll be in the next release.

@cdosborn cdosborn merged commit 905f8e0 into cyverse:v33 Jun 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants