Skip to content

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Apr 30, 2025

Now that we have some bake-in time for folks to try this out. Instead of weekend, we extended it till Wednesday morning, so time to disable auto-merge -- discuss the learning and decide whether we want to go for it or not. The activities are back to normal since it is a weekday


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@kaxil kaxil requested a review from potiuk April 30, 2025 08:43
@potiuk potiuk marked this pull request as draft April 30, 2025 08:58
@kaxil
Copy link
Member Author

kaxil commented Apr 30, 2025

Jarek & I discussed it on Slack in #internal-airflow-ci-cd. Summary below:

Given that many countries will be on holiday on May 1 due to Labour Day, and some teething issues were fixed yesterday, we will let it run for a few more days so other committers and contributors can get a chance to try it out and share their experience after the experiment/trial is concluded.

If we find anything blocking/causing disruption in that period, we will merge this PR to disable auto-merge.

@kaxil kaxil force-pushed the remove-auto-merge branch from 7014d65 to a89b18d Compare April 30, 2025 08:59
Now that we have some bake-in time for folks to try this out. Instead of weekend, we extended it till Wednesday morning, so time to disable auto-merge -- discuss the learning and decide whether we want to go for it or not.
@kaxil kaxil force-pushed the remove-auto-merge branch from a89b18d to 81d5e18 Compare April 30, 2025 09:21
@potiuk potiuk marked this pull request as ready for review April 30, 2025 15:13
@potiuk
Copy link
Member

potiuk commented Apr 30, 2025

Following back @cmarteepants comments -> in breeze the example DAGs from standard provider will be visible - regardless of the final decision on what we do for the example dags for the users.

image

@kaxil
Copy link
Member Author

kaxil commented Apr 30, 2025

Following back @cmarteepants comments -> in breeze the example DAGs from standard provider will be visible - regardless of the final decision on what we do for the example dags for the users.

image

I think this comment is for #49978 :)

@kaxil
Copy link
Member Author

kaxil commented Apr 30, 2025

Given more discussions and Jarek's approval, merging it.

Related email: https://lists.apache.org/thread/x47xlh2wm0h8dvgzx9b55qbflb5rmldj

@kaxil kaxil merged commit 931ae23 into apache:main Apr 30, 2025
76 of 77 checks passed
@kaxil kaxil deleted the remove-auto-merge branch April 30, 2025 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants