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

Vendor in the bitnami chart #24395

Merged
merged 1 commit into from
Jun 14, 2022
Merged

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jun 11, 2022

After the Bitnami fiasco
bitnami/charts#10539

We lost trust in bitnami index being good and reliable source
of charts. That's why we vendored-in the postgres chart needed for
our Helm chart.

Fixes: #24037


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragement file, named {pr_number}.significant.rst, in newsfragments.

@potiuk potiuk force-pushed the vendor-in-postgres-chart branch 2 times, most recently from 638479c to 18ddf21 Compare June 11, 2022 21:32
After the Bitnami fiasco
bitnami/charts#10539

We lost trust in bitnami index being good and reliable source
of charts. That's why we vendored-in the postgres chart needed for
our Helm chart.

Fixes: apache#24037
@potiuk
Copy link
Member Author

potiuk commented Jun 13, 2022

That should also speed up K8S tests

@potiuk potiuk requested review from malthe and uranusjr June 13, 2022 07:05
@potiuk
Copy link
Member Author

potiuk commented Jun 13, 2022

:) @jedcunningham ?

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Jun 14, 2022
@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.

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 type:misc/internal Changelog: Misc changes that should appear in change log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Frequent failures of helm chart tests
3 participants