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

Move one more operator to use DRE #1

Merged
merged 7 commits into from
Jun 25, 2024
Merged

Move one more operator to use DRE #1

merged 7 commits into from
Jun 25, 2024

Conversation

DFINITYManu
Copy link
Collaborator

No description provided.

@DFINITYManu DFINITYManu force-pushed the onemore branch 3 times, most recently from dbfd024 to f783de1 Compare June 25, 2024 14:06
@DFINITYManu DFINITYManu force-pushed the onemore branch 14 times, most recently from 08d430e to 39e0dda Compare June 25, 2024 15:13
Copy link
Member

@sasa-tomic sasa-tomic left a comment

Choose a reason for hiding this comment

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

LGTM!

@DFINITYManu DFINITYManu merged commit 2905e75 into main Jun 25, 2024
3 checks passed
@DFINITYManu DFINITYManu deleted the onemore branch June 25, 2024 15:27
DFINITYManu added a commit that referenced this pull request Sep 30, 2024
Two things happen in this PR:

1. An endpoint to fetch the internal cache is added, in order to help diagnose any logic errors that may be causing the API to return "not yet computed rollout" for rollouts started during the weekend.
2. The logic of when to fetch the rollout plan is changed to the task evaluation loop.

I cannot reproduce this locally with Airflow so the hope is that item #1 in this PR will help diagnose next week if the problem is not resolved by #2.
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