Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented May 27, 2021

We have now separate apache/airflow-ci DockerHub repo and we
move all our images needed only during CI there.

The images from the main apache/airflow remaining are:

  • airflow tagged and latest tagged production images
  • images neded by the Helm Chart

^ 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 UPDATING.md.

@potiuk potiuk requested review from ashb and kaxil as code owners May 27, 2021 13:30
@boring-cyborg boring-cyborg bot added area:dev-tools provider:cncf-kubernetes Kubernetes (k8s) provider related issues labels May 27, 2021
We have now separate `apache/airflow-ci` DockerHub repo and we
move all our images needed only during CI there.

The images from the main `apache/airflow` remaining are:

* airflow tagged and latest tagged production images
* images neded by the Helm Chart
@potiuk potiuk force-pushed the move-to-airflow-ci-repo branch from 5b5d583 to c4184e5 Compare May 28, 2021 10:52
@potiuk
Copy link
Member Author

potiuk commented May 28, 2021

Random errors . Should be ok to merge @ashb and I can move to master -> main move then :)

Copy link
Member

@ashb ashb left a comment

Choose a reason for hiding this comment

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

Let infra team rename the branches, yes?

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label May 28, 2021
@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 master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@potiuk potiuk merged commit 595374a into apache:master May 28, 2021
@potiuk potiuk deleted the move-to-airflow-ci-repo branch May 28, 2021 17:38
@potiuk
Copy link
Member Author

potiuk commented May 28, 2021

Let infra team rename the branches, yes?

Yeah. Let's do it. And I will push a PR changing the rest. This will break everyone's PRs but I think we won't avoid disruption. Just thinking. The main branch will not be protected for a while, but It's good as we might need to push a change to `main' directly to fix something.

potiuk added a commit to potiuk/airflow that referenced this pull request Jun 22, 2021
…6116)

We have now separate `apache/airflow-ci` DockerHub repo and we
move all our images needed only during CI there.

The images from the main `apache/airflow` remaining are:

* airflow tagged and latest tagged production images
* images neded by the Helm Chart

(cherry picked from commit 595374a)
@potiuk potiuk restored the move-to-airflow-ci-repo branch April 26, 2022 20:47
@potiuk potiuk deleted the move-to-airflow-ci-repo branch July 29, 2022 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools full tests needed We need to run full set of tests for this PR to merge provider:cncf-kubernetes Kubernetes (k8s) provider related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants