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

nightly connector test: get results by 6AM UTC #23626

Merged
merged 7 commits into from
Mar 2, 2023

Conversation

alafanechere
Copy link
Contributor

@alafanechere alafanechere commented Mar 1, 2023

What

For organizational reasons, it's more convenient to get connector test reports in the Europe morning.

How

  • Change test run execution to 4PAM UTC
  • Change test report / QA engine time to 6AM UTC

@alafanechere alafanechere force-pushed the augustin/change-nightly-builds-hour branch from aa1e0d2 to 730457a Compare March 1, 2023 15:54
@alafanechere alafanechere requested review from a team and lazebnyi March 1, 2023 15:54
.github/workflows/build-report.yml Outdated Show resolved Hide resolved
.github/workflows/connector_integration_tests.yml Outdated Show resolved Hide resolved
.github/workflows/connector_integration_tests_alpha.yml Outdated Show resolved Hide resolved
.github/workflows/run-qa-engine.yml Outdated Show resolved Hide resolved
alafanechere and others added 4 commits March 1, 2023 16:58
Co-authored-by: Pedro S. Lopez <pedroslopez@me.com>
Co-authored-by: Pedro S. Lopez <pedroslopez@me.com>
Co-authored-by: Pedro S. Lopez <pedroslopez@me.com>
Co-authored-by: Pedro S. Lopez <pedroslopez@me.com>
@lazebnyi lazebnyi changed the title nightly connector test: get results by 8AM UTC nightly connector test: get results by 6AM UTC Mar 1, 2023
Copy link
Collaborator

@lazebnyi lazebnyi left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

Leave minor comment and updated name to changes 😁

@@ -6,8 +6,8 @@ name: Generate Build Report
on:
workflow_dispatch:
schedule:
# 1pm UTC is 6am PDT.
- cron: "0 13 * * *"
# 6AM UTC is 8AM EET, 7AM CET, 11PM PST.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think better have 3 hours before try get report, because we have some streams with timeout more than 7200. For example mixpanel has 9000 seconds timeout for basic read.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, lets push back the test start at 3AM UTC.

@alafanechere alafanechere enabled auto-merge (squash) March 2, 2023 13:11
@alafanechere alafanechere merged commit 3cf3cfc into master Mar 2, 2023
@alafanechere alafanechere deleted the augustin/change-nightly-builds-hour branch March 2, 2023 13:31
danielduckworth pushed a commit to danielduckworth/airbyte that referenced this pull request Mar 13, 2023
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.

4 participants