Skip to content

Conversation

@jedcunningham
Copy link
Member

@jedcunningham jedcunningham commented Sep 27, 2021

The helm tests are now regularly taking longer than 25 minutes on public
GitHub Actions workers, so we will increase the timeout.

The helm tests are now regularly taking longer than 25 minutes on public
GitHub Actions workers, so we will increase the timeout.
@jedcunningham jedcunningham force-pushed the chart-unit-test-failing branch from 0086348 to 471982a Compare September 27, 2021 20:20
@jedcunningham jedcunningham changed the title 🚧 Extra debugging for helm tests Chart: Fix unittests on plublic runners Sep 27, 2021
@jedcunningham jedcunningham marked this pull request as ready for review September 27, 2021 21:17
@jedcunningham jedcunningham changed the title Chart: Fix unittests on plublic runners Chart: Fix unittests on public runners Sep 27, 2021
@potiuk potiuk closed this Sep 28, 2021
@potiuk potiuk reopened this Sep 28, 2021
@potiuk
Copy link
Member

potiuk commented Sep 28, 2021

Reopened to rebuild

@github-actions
Copy link

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Sep 28, 2021
@potiuk
Copy link
Member

potiuk commented Sep 28, 2021

For one reason or another we seem to have NEW flaky tests ... It is a bit like Hydra. You cut one head and two more grow in it's place.

@jedcunningham
Copy link
Member Author

Merging now as these test failures are unrelated to this change.

@jedcunningham jedcunningham changed the title Chart: Fix unittests on public runners Fix helm chart unittests on public runners Sep 29, 2021
@jedcunningham jedcunningham merged commit 1763180 into apache:main Sep 29, 2021
@jedcunningham jedcunningham deleted the chart-unit-test-failing branch September 29, 2021 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools area:helm-chart Airflow Helm Chart full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants