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

Patch Augur-new's task scheduling mechanism #1975

Merged
merged 2 commits into from Sep 8, 2022
Merged

Conversation

IsaacMilarky
Copy link
Contributor

Description

  • Add patches to AugurTaskRoutine to prevent one phase's changes not being reflected in subsequent phases
  • Patch contributor resolution so it can parse string data better
  • Phases are now defined in python functions instead of messily through an enum. The name of the phase is the name of the function and the function just returns what you want celery to run; ezpz hopefully
  • Revert change to some tasks to take a repo_id instead of the repo_git string as it raised concerns about log readability.

ABrain7710 and others added 2 commits September 7, 2022 14:18
Signed-off-by: Andrew Brain <61482022+ABrain7710@users.noreply.github.com>
Signed-off-by: Isaac Milarsky <krabs@tilde.team>
@IsaacMilarky IsaacMilarky changed the title Augur new dev Patch Augur-new's task scheduling mechanism Sep 7, 2022
@ABrain7710 ABrain7710 merged commit 08d08db into augur-new Sep 8, 2022
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.

None yet

3 participants