-
Notifications
You must be signed in to change notification settings - Fork 91
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
update forecasting dag #1534
update forecasting dag #1534
Conversation
@perrymcmanis144 can you take a look at this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM, you can view DAG status for this job here:
https://workflow.telemetry.mozilla.org/tree?dag_id=kpi_forecasting
I need some help clicking the big green button on this |
@irrationalagent the new task is failing with the following error: cc: @perrymcmanis144 |
Looks like this was because CI deploy for the jobs failed due to authentication errors for your user @irrationalagent. I've triggered a redeploy and seems the job deployment worked this time. Although some of the other ones are failing due to linter checks. |
The following two jobs fail to deploy:
|
I was able to redeploy an updated version of the image using my account. The Airflow task has now completed successfully. |
Created a PR to address the CI failures: |
This is paired with mozilla/docker-etl#67
Adding the task for the mobile non-cumulative forecast and changing the desktop non-cumulative forecast to now depend on unified metrics