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

[AIRFLOW-6701] Rat is downloaded from stable backup/mirrors #7323

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Feb 1, 2020

Also curl options are now using long format and include --fail
to protect against some temporary errors (5xx). Also RAT download
uses now two possible sources of downloads and fallbacks to the
second if first is not available.


Issue link: AIRFLOW-6701

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

  • Description above provides context of the change
  • Commit message/PR title starts with [AIRFLOW-NNNN]. AIRFLOW-NNNN = JIRA ID*
  • Unit tests coverage for changes (not needed for documentation changes)
  • 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.

* For document-only changes commit message can start with [AIRFLOW-XXXX].


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.

@potiuk potiuk force-pushed the AIRFLOW-6701-download-rat-from-stable-mirrors branch from a2d2436 to 840be8b Compare February 1, 2020 14:45
@potiuk potiuk changed the title [AURFLOW-6701] Rat is downloaded from stable backup/mirrors [AIRFLOW-6701] Rat is downloaded from stable backup/mirrors Feb 1, 2020
@potiuk
Copy link
Member Author

potiuk commented Feb 1, 2020

We had quite a lot of failures in cron master job (where the Dockerfile is rebuilt) and the reason was RAT download failures - it was from maven repos and those repos were not stable, so I figured we need to do it "properly" using abache backup servers + apache mirrors. It should succeed (as it does not touch .py nor kubernetes related files, so we can merge it even before we find and fix the kubernetes/kerberos failures.

@potiuk potiuk force-pushed the AIRFLOW-6701-download-rat-from-stable-mirrors branch from 840be8b to 369223e Compare February 1, 2020 15:22
@potiuk
Copy link
Member Author

potiuk commented Feb 1, 2020

The sooner we get it - the better - the CRON job overnight will become much more stable.

@potiuk
Copy link
Member Author

potiuk commented Feb 1, 2020

All green ! pls take a look

Also curl options are now using long format and include --fail
to protect against some temporary errors (5xx). Also RAT download
uses now two possible sources of downloads and fallbacks to the
second if first is not available.
@potiuk potiuk force-pushed the AIRFLOW-6701-download-rat-from-stable-mirrors branch from 369223e to 6a46efe Compare February 2, 2020 07:29
@potiuk potiuk merged commit d7d2794 into apache:master Feb 2, 2020
potiuk added a commit that referenced this pull request Feb 2, 2020
Also curl options are now using long format and include --fail
to protect against some temporary errors (5xx). Also RAT download
uses now two possible sources of downloads and fallbacks to the
second if first is not available.

(cherry picked from commit d7d2794)
kaxil pushed a commit that referenced this pull request Feb 3, 2020
Also curl options are now using long format and include --fail
to protect against some temporary errors (5xx). Also RAT download
uses now two possible sources of downloads and fallbacks to the
second if first is not available.

(cherry picked from commit d7d2794)
galuszkak pushed a commit to FlyrInc/apache-airflow that referenced this pull request Mar 5, 2020
)

Also curl options are now using long format and include --fail
to protect against some temporary errors (5xx). Also RAT download
uses now two possible sources of downloads and fallbacks to the
second if first is not available.
kaxil pushed a commit to astronomer/airflow that referenced this pull request Mar 30, 2020
)

Also curl options are now using long format and include --fail
to protect against some temporary errors (5xx). Also RAT download
uses now two possible sources of downloads and fallbacks to the
second if first is not available.

(cherry picked from commit d7d2794)
(cherry picked from commit a2b8204)
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