Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@kaxil
Copy link
Collaborator

@kaxil kaxil commented May 25, 2022

The CI is failing intermittently as I initially suspected in #370

Example: https://github.com/astronomer/astro-sdk/runs/6601723455?check_suite_focus=true

This PR runs 2.2.5 tests after 2.3 tests are complete and also only runs a subset of the tests (Example DAGs and one of the integration tests) for 2.2.5 which should be enough for 2.2.5

@kaxil kaxil requested a review from tatiana May 25, 2022 23:39
@kaxil kaxil force-pushed the split-airflow-tests branch from 1e49365 to eb2fbb3 Compare May 26, 2022 00:04
The CI is failing intermittently as I initially suspected in #370

Example: https://github.com/astronomer/astro-sdk/runs/6601723455?check_suite_focus=true

This PR runs 2.2.5 tests after 2.3 tests are complete and also only runs a subset of the tests (Example DAGs and one of the integration tests) for 2.2.5 which should be enough for 2.2.5
@kaxil kaxil force-pushed the split-airflow-tests branch from eb2fbb3 to f46854e Compare May 26, 2022 00:14
Copy link
Collaborator

@tatiana tatiana left a comment

Choose a reason for hiding this comment

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

It makes sense to run a subset of the tests, for now, @kaxil ! Once the automated checks pass, please, feel free to merge this.

@kaxil
Copy link
Collaborator Author

kaxil commented May 26, 2022

I am going to merge this, the only failing test is due to the following "rate limits exceeded":

      raise self._exception

E google.api_core.exceptions.Forbidden: 403 Exceeded rate limits: too many table update operations for this table. For more information, see https://cloud.google.com/bigquery/docs/troubleshoot-quotas
E
E Location: US
E Job ID: airflow_1653524820919466_e4e6bf2ad111e68769e3b101768b8c5a

@kaxil kaxil merged commit e0a3077 into main May 26, 2022
@kaxil kaxil deleted the split-airflow-tests branch May 26, 2022 00:29
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