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

Updated SLA values for dbt eho related tasks #384

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

edualvess
Copy link
Contributor

This PR is hotfix for the SLA alerts related to dbt EHO DAG (enriched_history_operations and elementary_dbt_enriched_base_tables), by increasing the values to avoid sending alerts in heavy load time frames (16h UTC).

@edualvess edualvess requested a review from a team as a code owner June 13, 2024 16:35
@edualvess edualvess merged commit 12a807b into master Jun 13, 2024
4 checks passed
@edualvess edualvess deleted the hotfix/increase_dbt_eho_sla branch June 13, 2024 16:47
@edualvess edualvess mentioned this pull request Jun 14, 2024
5 tasks
sydneynotthecity added a commit that referenced this pull request Jun 17, 2024
* HUBBLE-409 Update/delete airflow variable names (#377)

* test time task without affinity

* delete affinity from repo

* dbt_

* testing config related to stellar_marts

* documented

* test without var dbt_dataset

* with dbt_dataset again

* dbt_dataset_for_test

* comment excluded

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

* Created alert_sla_miss callback funcion

* Defined test value for export task SLA

* Defined sla_miss_callback for DAG level parameter

* Increased time task timeout value

* Added dev Airflow variable for tasks SLA values

* Added slack notification logic for the SLA miss callback

* Added SLA parameter to the relavant task building functions

* Removed SLA miss callback function and related logic

* Added refactored logic for sla miss callback function

* Added sla miss callback to desired dags

* Adjusted callback logic for sentry integration

* Added task SLA values for prod Airflow variables

* Changed alert message to better distinguish SLA miss from task fail alerts

* Update dev tasks sla and timeout values

* Update prod tasks sla and timeout values

* Added default sla value to dag default args

* Added missing SLA param to build task functions

* Standardizing logic for fetching SLA param value

* Added  sla miss callback reference

* Remove default SLA from default args

* Set missing sla params to tasks

* Updated values for sla variables

* Updated build_dbt_task timeout value

* Changed variable names to match new naming reference

---------

Co-authored-by: chowbao <[email protected]>

* Update stellar-etl image (#381)

* Update stellar-etl image (#383)

* Updated SLA values for dbt eho related tasks (#384)

* Patch/merge conflict resolution (#386)

* [PRODUCTION] Update production Airflow environment (#370)

* requirements updated

* sqlfluff

* Use the correct gcs bucket (#372)

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

* [PRODUCTION] Update production Airflow environment (#375)

* requirements updated

* Update READMEs

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

* [PRODUCTION] Update production Airflow environment (#378)

* HUBBLE-409 Update/delete airflow variable names (#377)

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

---------

Co-authored-by: chowbao <[email protected]>

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

* [PRODUCTION] Update production Airflow environment (#382)

* HUBBLE-409 Update/delete airflow variable names (#377)

* test time task without affinity

* delete affinity from repo

* dbt_

* testing config related to stellar_marts

* documented

* test without var dbt_dataset

* with dbt_dataset again

* dbt_dataset_for_test

* comment excluded

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

* Created alert_sla_miss callback funcion

* Defined test value for export task SLA

* Defined sla_miss_callback for DAG level parameter

* Increased time task timeout value

* Added dev Airflow variable for tasks SLA values

* Added slack notification logic for the SLA miss callback

* Added SLA parameter to the relavant task building functions

* Removed SLA miss callback function and related logic

* Added refactored logic for sla miss callback function

* Added sla miss callback to desired dags

* Adjusted callback logic for sentry integration

* Added task SLA values for prod Airflow variables

* Changed alert message to better distinguish SLA miss from task fail alerts

* Update dev tasks sla and timeout values

* Update prod tasks sla and timeout values

* Added default sla value to dag default args

* Added missing SLA param to build task functions

* Standardizing logic for fetching SLA param value

* Added  sla miss callback reference

* Remove default SLA from default args

* Set missing sla params to tasks

* Updated values for sla variables

* Updated build_dbt_task timeout value

* Changed variable names to match new naming reference

---------

Co-authored-by: chowbao <[email protected]>

* Update stellar-etl image (#381)

* Update stellar-etl image (#383)

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

* Help (#388)

* [PRODUCTION] Update production Airflow environment (#370)

* requirements updated

* sqlfluff

* Use the correct gcs bucket (#372)

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

* [PRODUCTION] Update production Airflow environment (#375)

* requirements updated

* Update READMEs

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

* [PRODUCTION] Update production Airflow environment (#378)

* HUBBLE-409 Update/delete airflow variable names (#377)

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

---------

Co-authored-by: chowbao <[email protected]>

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

* [PRODUCTION] Update production Airflow environment (#382)

* HUBBLE-409 Update/delete airflow variable names (#377)

* test time task without affinity

* delete affinity from repo

* dbt_

* testing config related to stellar_marts

* documented

* test without var dbt_dataset

* with dbt_dataset again

* dbt_dataset_for_test

* comment excluded

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

* Created alert_sla_miss callback funcion

* Defined test value for export task SLA

* Defined sla_miss_callback for DAG level parameter

* Increased time task timeout value

* Added dev Airflow variable for tasks SLA values

* Added slack notification logic for the SLA miss callback

* Added SLA parameter to the relavant task building functions

* Removed SLA miss callback function and related logic

* Added refactored logic for sla miss callback function

* Added sla miss callback to desired dags

* Adjusted callback logic for sentry integration

* Added task SLA values for prod Airflow variables

* Changed alert message to better distinguish SLA miss from task fail alerts

* Update dev tasks sla and timeout values

* Update prod tasks sla and timeout values

* Added default sla value to dag default args

* Added missing SLA param to build task functions

* Standardizing logic for fetching SLA param value

* Added  sla miss callback reference

* Remove default SLA from default args

* Set missing sla params to tasks

* Updated values for sla variables

* Updated build_dbt_task timeout value

* Changed variable names to match new naming reference

---------

Co-authored-by: chowbao <[email protected]>

* Update stellar-etl image (#381)

* Update stellar-etl image (#383)

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

* Patch/fix release conflicts (#390)

* [PRODUCTION] Update production Airflow environment (#370)

* requirements updated

* sqlfluff

* Use the correct gcs bucket (#372)

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

* [PRODUCTION] Update production Airflow environment (#375)

* requirements updated

* Update READMEs

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>

---------

Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

* [PRODUCTION] Update production Airflow environment (#378)

* HUBBLE-409 Update/delete airflow variable names (#377)

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

---------

Co-authored-by: chowbao <[email protected]>

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

* [PRODUCTION] Update production Airflow environment (#382)

* HUBBLE-409 Update/delete airflow variable names (#377)

* test time task without affinity

* delete affinity from repo

* dbt_

* testing config related to stellar_marts

* documented

* test without var dbt_dataset

* with dbt_dataset again

* dbt_dataset_for_test

* comment excluded

* Updates for testnet-reset (#380)

* Feature/add slas (#359)

* Created alert_sla_miss callback funcion

* Defined test value for export task SLA

* Defined sla_miss_callback for DAG level parameter

* Increased time task timeout value

* Added dev Airflow variable for tasks SLA values

* Added slack notification logic for the SLA miss callback

* Added SLA parameter to the relavant task building functions

* Removed SLA miss callback function and related logic

* Added refactored logic for sla miss callback function

* Added sla miss callback to desired dags

* Adjusted callback logic for sentry integration

* Added task SLA values for prod Airflow variables

* Changed alert message to better distinguish SLA miss from task fail alerts

* Update dev tasks sla and timeout values

* Update prod tasks sla and timeout values

* Added default sla value to dag default args

* Added missing SLA param to build task functions

* Standardizing logic for fetching SLA param value

* Added  sla miss callback reference

* Remove default SLA from default args

* Set missing sla params to tasks

* Updated values for sla variables

* Updated build_dbt_task timeout value

* Changed variable names to match new naming reference

---------

Co-authored-by: chowbao <[email protected]>

* Update stellar-etl image (#381)

* Update stellar-etl image (#383)

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>

---------

Co-authored-by: sydneynotthecity <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Laysa de Sousa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>

---------

Co-authored-by: Laysa Bitencourt <[email protected]>
Co-authored-by: chowbao <[email protected]>
Co-authored-by: Eduardo Alves <[email protected]>
Co-authored-by: sydneynotthecity <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

2 participants