Skip to content

Remove redundant airflowVersion from Helm Chart readme#9592

Merged
kaxil merged 1 commit intoapache:masterfrom
astronomer:remove-redundant
Jun 30, 2020
Merged

Remove redundant airflowVersion from Helm Chart readme#9592
kaxil merged 1 commit intoapache:masterfrom
astronomer:remove-redundant

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Jun 30, 2020

We no longer use airflowVersion , we instead use defaultAirflowRepository and defaultAirflowTag


Make sure to mark the boxes below before creating PR: [x]

  • Description above provides context of the change
  • Unit tests coverage for changes (not needed for documentation changes)
  • Target Github ISSUE in description if exists
  • Commits follow "How to write a good git commit message"
  • Relevant documentation is updated including usage instructions.
  • I will engage committers as explained in Contribution Workflow Example.

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.
Read the Pull Request Guidelines for more information.

We no longer use `airflowVersion` , we instead use `defaultAirflowRepository` and `defaultAirflowTag`
@kaxil kaxil requested a review from dimberman June 30, 2020 14:40
@boring-cyborg boring-cyborg bot added the area:helm-chart Airflow Helm Chart label Jun 30, 2020
@kaxil kaxil merged commit d6b323b into apache:master Jun 30, 2020
@kaxil kaxil deleted the remove-redundant branch June 30, 2020 16:03
potiuk pushed a commit that referenced this pull request Jul 1, 2020
We no longer use `airflowVersion` , we instead use `defaultAirflowRepository` and `defaultAirflowTag`

(cherry picked from commit d6b323b)
@potiuk potiuk added this to the Airflow 1.10.11 milestone Jul 1, 2020
kaxil added a commit that referenced this pull request Jul 2, 2020
We no longer use `airflowVersion` , we instead use `defaultAirflowRepository` and `defaultAirflowTag`

(cherry picked from commit d6b323b)
kaxil added a commit that referenced this pull request Jul 13, 2020
We no longer use `airflowVersion` , we instead use `defaultAirflowRepository` and `defaultAirflowTag`

(cherry picked from commit d6b323b)
cfei18 pushed a commit to cfei18/incubator-airflow that referenced this pull request Mar 5, 2021
We no longer use `airflowVersion` , we instead use `defaultAirflowRepository` and `defaultAirflowTag`

(cherry picked from commit d6b323b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:helm-chart Airflow Helm Chart

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants